Skip to content

Commit

Permalink
CMake: use enable_testing() instead of include(CTest)
Browse files Browse the repository at this point in the history
According to Craig Scott in https://discourse.cmake.org/t/is-there-any-reason-to-prefer-include-ctest-or-enable-testing-over-the-other/1905/2 ,
using include(CTest) adds unnecessary clutter that is only needed for
dashboard submission. enable_testing() is enough otherwise

"Port" of OSGeo/shapelib#162
  • Loading branch information
rouault committed Oct 25, 2024
1 parent 793e19a commit 21ccce8
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -382,9 +382,12 @@ add_subdirectory(capi)
#-----------------------------------------------------------------------------
# Tests
#-----------------------------------------------------------------------------

option(BUILD_TESTING "Build the testing tree." ON)

if(PROJECT_IS_TOP_LEVEL)
include(CTest)
if(BUILD_TESTING)
enable_testing()
add_subdirectory(tests)
endif()
endif()
Expand Down

0 comments on commit 21ccce8

Please sign in to comment.