Merge pull request #1259 from ACSimon33/fix_python_test_summary

Fix Python test summary

Author: langou

CMakeLists.txt

@@ -123,13 +123,6 @@ message(STATUS "Build Index-64 API as extended API with _64 suffix: ${BUILD_INDE
 
 include(GNUInstallDirs)
 
-# Configure the warning and code coverage suppression file
-configure_file(
-  "${LAPACK_SOURCE_DIR}/CTestCustom.cmake.in"
-  "${LAPACK_BINARY_DIR}/CTestCustom.cmake"
-  @ONLY
-)
-
 include(PreventInSourceBuilds)
 include(PreventInBuildInstalls)
 
@@ -186,6 +179,13 @@ if(BUILD_TESTING)
   else()
     message(STATUS "${_msg} - not found (skipping summary tests)")
   endif()
+
+  # Configure the warning and code coverage suppression file
+  configure_file(
+    "${LAPACK_SOURCE_DIR}/CTestCustom.cmake.in"
+    "${LAPACK_BINARY_DIR}/CTestCustom.cmake"
+    @ONLY
+  )
 endif()
 
 # --------------------------------------------------

CTestCustom.cmake.in

@@ -47,9 +47,9 @@ set(CTEST_CUSTOM_WARNING_EXCEPTION
 )
 
 # Only run post test if suitable python interpreter was found
-set(PYTHON_EXECUTABLE  @PYTHON_EXECUTABLE@)
+set(Python3_EXECUTABLE "@Python3_EXECUTABLE@")
 set(LAPACK_TESTING_USE_PYTHON  @LAPACK_TESTING_USE_PYTHON@)
-if(PYTHON_EXECUTABLE AND LAPACK_TESTING_USE_PYTHON)
-  set(CTEST_CUSTOM_POST_TEST "${PYTHON_EXECUTABLE} ./lapack_testing.py -s -d TESTING")
+if(Python3_EXECUTABLE AND LAPACK_TESTING_USE_PYTHON)
+  set(CTEST_CUSTOM_POST_TEST "\"${Python3_EXECUTABLE}\" ./lapack_testing.py -s -d TESTING")
 endif()
 

TESTING/CMakeLists.txt

@@ -15,13 +15,13 @@ add_subdirectory(EIG)
 
 
 # Only run this test if python 3 is found
-if(PYTHON_EXECUTABLE AND LAPACK_TESTING_USE_PYTHON)
+if(Python3_EXECUTABLE AND LAPACK_TESTING_USE_PYTHON)
   message(STATUS "Enabling LAPACK test summary (see TESTING/testing_results.txt)")
   file(COPY ${LAPACK_SOURCE_DIR}/lapack_testing.py DESTINATION ${LAPACK_BINARY_DIR})
   add_test(
     NAME LAPACK_Test_Summary
     WORKING_DIRECTORY ${LAPACK_BINARY_DIR}
-    COMMAND ${PYTHON_EXECUTABLE} "lapack_testing.py"
+    COMMAND ${Python3_EXECUTABLE} "lapack_testing.py"
     )
 endif()
 
@@ -38,7 +38,7 @@ function(add_lapack_test output input target)
       -DINTDIR=${CMAKE_CFG_INTDIR}
       -P "${LAPACK_SOURCE_DIR}/TESTING/runtest.cmake")
 
-    if(PYTHON_EXECUTABLE AND LAPACK_TESTING_USE_PYTHON)
+    if(Python3_EXECUTABLE AND LAPACK_TESTING_USE_PYTHON)
       set_property(
         TEST LAPACK_Test_Summary
         APPEND PROPERTY DEPENDS LAPACK-${testName}