p/tdv
1
1
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix the test target and unargumented ctest

Browse Source 
CMake does know how to be tough on users, but we've won at last.
This commit is contained in:
Přemysl Eric Janouch
2021-10-27 00:16:42 +02:00
parent 0f45b9bf3b
commit 451859e976
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
CMakeLists.txt
View File

@@ -72,9 +72,11 @@ if (USE_SYSTEM_TERMO)
message (FATAL_ERROR "System termo library not found") message (FATAL_ERROR "System termo library not found")
endif () endif ()
else () else ()
# We don't want the library to install, even though EXCLUDE_FROM_ALL # We don't want the library to install, but EXCLUDE_FROM_ALL ignores tests
# sabotages CTest -- those unbuilt tests need to be excluded in CTest runs
add_subdirectory (termo EXCLUDE_FROM_ALL) add_subdirectory (termo EXCLUDE_FROM_ALL)
file (WRITE ${PROJECT_BINARY_DIR}/CTestCustom.cmake
"execute_process (COMMAND ${CMAKE_COMMAND} --build termo)")
# We don't have many good choices; this is a relatively clean approach # We don't have many good choices; this is a relatively clean approach
# (other possibilities: setting a variable in the parent scope, using # (other possibilities: setting a variable in the parent scope, using
# a cache variable, writing a special config file with build paths in it # a cache variable, writing a special config file with build paths in it