]> https://gitweb.dealii.org/ - dealii.git/commitdiff
fix ctest makeopts 1719/head
authorTimo Heister <timo.heister@gmail.com>
Mon, 5 Oct 2015 17:45:05 +0000 (13:45 -0400)
committerTimo Heister <timo.heister@gmail.com>
Mon, 5 Oct 2015 17:45:05 +0000 (13:45 -0400)
The argument after TARGET is the target to build and we abused it to
specify build flags. This fails on windows. With cmake 3.3 there is an
argument FLAGS <flags>, but older versions grab CTEST_BUILD_FLAGS, so
this is where we put the flags now.

tests/run_testsuite.cmake

index 175cf52e8c3c59d361d36f1448bd7bcf3a31381d..007a62cbe8ec31c953d3f99c807670a1c1c52795 100644 (file)
@@ -472,7 +472,8 @@ IF("${_res}" STREQUAL "0")
   # Only run the build stage if configure was successful:
 
   MESSAGE("-- Running CTEST_BUILD()")
-  CTEST_BUILD(TARGET ${MAKEOPTS} NUMBER_ERRORS _res)
+  SET(CTEST_BUILD_FLAGS "${MAKEOPTS}")
+  CTEST_BUILD(NUMBER_ERRORS _res)
 
   IF("${_res}" STREQUAL "0")
     # Only run tests if the build was successful:

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.