]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Fix a check on numdiff. 8546/head
authorDavid Wells <drwells@email.unc.edu>
Sat, 10 Aug 2019 19:31:17 +0000 (13:31 -0600)
committerDavid Wells <drwells@email.unc.edu>
Mon, 2 Sep 2019 21:03:53 +0000 (17:03 -0400)
This check didn't work for two reasons:
1. We should expand the variable
2. FIND_PROGRAM indicates failure by setting the value of the variable
   to 'variable-NOTFOUND', not '-NOTFOUND'.

cmake/macros/macro_deal_ii_pickup_tests.cmake

index cd3cf6af9749168006a5d42f553ff986dbe747da..085ec637a5e52372b346a882afbce8afbbba0c83 100644 (file)
@@ -98,7 +98,7 @@ MACRO(DEAL_II_PICKUP_TESTS)
 
   MARK_AS_ADVANCED(NUMDIFF_EXECUTABLE)
 
-  IF( NUMDIFF_EXECUTABLE MATCHES "-NOTFOUND")
+  IF( "${NUMDIFF_EXECUTABLE}" MATCHES "NUMDIFF_EXECUTABLE-NOTFOUND")
     MESSAGE(FATAL_ERROR
       "Could not find numdiff, which is required for running the testsuite.\n"
       "Please specify NUMDIFF_DIR to a location containing the binary."
@@ -117,9 +117,9 @@ MACRO(DEAL_II_PICKUP_TESTS)
 
   IF(NOT "${_diff_program_status}" STREQUAL "0")
     MESSAGE(FATAL_ERROR
-      "\nThe command \"${_diff_program} -v\" did not run correctly: it either "
-      "failed to exit after a few seconds or returned a nonzero exit code. "
-      "The test suite cannot be set up without this program, so please "
+      "\nThe command \"${NUMDIFF_EXECUTABLE} -v\" did not run correctly: it "
+      "either failed to exit after a few seconds or returned a nonzero exit "
+      "code. The test suite cannot be set up without this program, so please "
       "reinstall it and then run the test suite setup command again.\n")
   ENDIF()
 

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.