Merge branch 'devel' of https://cta-gitlab.irap.omp.eu/ctools/ctools into 1982-a…

…dd-phase-info-ctphase

Merge branch 'devel' of https://cta-gitlab.irap.omp.eu/ctools/ctools into 1982-a…
…dd-phase-info-ctphase
Josh Cardenzana
2 parents bb9410c7 e77d88fa
Showing 4 changed files with 51 additions and 17 deletions
ChangeLog
NEWS
configure.ac
cscripts/Makefile.am
-2017-04-04
+2017-04-05
  
         * Version 1.3.0 released
         ========================
  
+        Copy cscripts into bin directory instead of creating a symbolic links (#1987)
+        If code originates from Git, always compile the Python bindings (#1985)
         Raise exception instead of exiting with return code in Python tests (#1983)
         Support of map cubes lying outside RoI in ctobssim 
         Support energy boundary FITS file in ctmapcube (#1978)
 New Features and Important Changes in ctools 1.3.0.dev1
  
-4 April 2017
+5 April 2017
  
  
 Introduction
@@ -15,6 +15,16 @@ The Python unit tests now raise exceptions instead of exiting with a non-
 zero return code so that the user is not thrown out of Python after calling
 ctools.test() and cscripts.test() (#1983).
  
+If the code originates from the Git repository the configuration now switches
+to developer mode which implies that the Python bindings are always built.
+This solves the problem that occured for developers that could not get rid
+of existing Python wrappers, even if they modified the Python interface
+(#1985).
+
+The cscripts are now copied into the bin directory upon installation instead
+of creating symbolic links. This avoids problems on some VM file systems on
+which creation of symbolic links does not work (#1987).
+
  
 Python interface
 ----------------
 #############################################################################
 # Configuration for ctools - tools for science analysis of CTA data         #
 # ------------------------------------------------------------------------- #
-# Copyright (C) 2011-2016 Juergen Knoedlseder                               #
+# Copyright (C) 2011-2017 Juergen Knoedlseder                               #
 # ------------------------------------------------------------------------- #
 #                                                                           #
 #  This program is free software: you can redistribute it and/or modify     #
@@ -608,9 +608,10 @@ AC_C_INLINE
 # Python binding requires Python and either the files tools_wrap.cpp and    #
 # tools.py or swig to create these files.  The following conditionals are   #
 # set:                                                                      #
-# WITH_PYTHON   - Python bindings will be built                             #
-# HAVE_SWIG     - swig is available                                         #
-# HAVE_WRAPPERS - wrappers are available                                    #
+# WITH_PYTHON    - Python bindings will be built                            #
+# HAVE_SWIG      - swig is available                                        #
+# HAVE_WRAPPERS  - wrappers are available                                   #
+# BUILD_WRAPPERS - build the Python wrappers                                #
 #############################################################################
 # Initialise flags
 enable_python="no"
@@ -620,6 +621,12 @@ has_swig=&quot;no&quot;
 has_wrappers="no"
 build_wrappers="no"
  
+# Check if the code is a git clone. In that case the systems switches to
+# developer mode which forces building of the Python interface with SWIG.
+AC_CHECK_FILE([.git/HEAD],
+              [from_git=yes;build_wrappers="yes"],
+              [from_git=no])
+
 # Python binding enabled?
 AC_ARG_ENABLE([python-binding],
               AS_HELP_STRING([--enable-python-binding],
@@ -1065,21 +1072,36 @@ else
   echo "  - GammaLib support             (no)    no gammalib library found"
 fi
  
+# Dump developer configuration
+if test "x$from_git" = "xyes"; then
+  echo "  * Developer configuration      (yes)"
+else
+  echo "  * End user configuration       (yes)"
+fi
+
 # Dump Python binding information
 if test "x$ac_enable_python_binding" = "xyes"; then
   if test "x$has_python" = "xyes"; then
     if test "x$has_python_h" = "xyes"; then
-      if test "x$has_wrappers" = "xyes"; then
-        if test "x$has_swig" = "xyes"; then
-          echo "  * Make Python binding          (yes)   use swig for updates"
+      if test "x$build_wrappers" = "xyes"; then
+        if test "x$has_wrappers" = "xyes"; then
+          if test "x$has_swig" = "xyes"; then
+            echo "  * Make Python binding          (yes)   use swig for updates"
+          else
+            echo "  * Make Python binding          (yes)   use wrappers"
+          fi
         else
-          echo "  * Make Python binding          (yes)   use wrappers"
+          if test "x$has_swig" = "xyes"; then
+            echo "  * Make Python binding          (yes)   use swig for building"
+          else
+            echo "  - Make Python binding          (no)    swig required to build wrappers"
+          fi
         fi
       else
-        if test "x$has_swig" = "xyes"; then
-          echo "  * Make Python binding          (yes)   use swig for building"
+        if test "x$has_wrappers" = "xyes"; then
+          echo "  - Make Python binding          (yes)   use existing wrappers"
         else
-          echo "  - Make Python binding          (no)    swig required to build wrappers"
+          echo "  - Make Python binding          (no)    wrappers not found"
         fi
       fi
     else
@@ -223,7 +223,7 @@ EXTRA_DIST = $(binscripts) $(pythonscripts) $(parfiles) \
              $(testscripts) $(testdata) $(testiactdata) \
              $(testiactevents)
  
-# Add install hook rule to create symbolic links for cscripts
+# Add install hook rule to copy cscripts without .py into the bindir
 install-data-hook:
 	@binlist='$(binscripts)';\
 	for p in $$binlist; do \
@@ -232,11 +232,11 @@ install-data-hook:
 	@linklist='$(linkscripts)';\
 	for p in $$linklist; do \
 		rm -rf "$(DESTDIR)$(bindir)/$$p"; \
-		ln -s "$(DESTDIR)$(pyexecdir)/cscripts/$$p.py" "$(DESTDIR)$(bindir)/$$p"; \
-		chmod +x "$(DESTDIR)$(pyexecdir)/cscripts/$$p.py"; \
+		cp "$(DESTDIR)$(pyexecdir)/cscripts/$$p.py" "$(DESTDIR)$(bindir)/$$p"; \
+		chmod +x "$(DESTDIR)$(bindir)/$$p"; \
 	done
  
-# Uninstall hook rule to remove the symbolic links for cscripts
+# Uninstall hook rule to remove the cscripts from the bindir
 uninstall-hook:
 	@linklist='$(linkscripts)';\
 	for p in $$linklist; do \