From b8ec4e3fa8060a1a4edcb15d8d7eb98c27136945 Mon Sep 17 00:00:00 2001 From: Timo Heister Date: Mon, 5 Oct 2015 13:45:05 -0400 Subject: [PATCH] fix ctest makeopts 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 , but older versions grab CTEST_BUILD_FLAGS, so this is where we put the flags now. --- tests/run_testsuite.cmake | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/run_testsuite.cmake b/tests/run_testsuite.cmake index 175cf52e8c..007a62cbe8 100644 --- a/tests/run_testsuite.cmake +++ b/tests/run_testsuite.cmake @@ -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: -- 2.39.5