Merge pull request #162 from thbeu/remove-ctest

Update CMake build dependencies: Remove include(CTest) (/w enable_testing() still set) and fix Google Benchmark
OSGeo · Oct 25, 2024 · 0d55f04 · 0d55f04
2 parents a279d13 + a768b99
commit 0d55f04
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -292,8 +292,6 @@ if(BUILD_TESTING)
   )
   declare_test_executable(test3 "dbfadd;dbfcreate;dbfdump;shpadd;shpcreate;shpdump")
 
-  include(CTest)
-
   add_subdirectory(tests)
 endif()
 

diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
@@ -1,4 +1,4 @@
-# CMake configuration for SHP C++ unit tests
+# CMake configuration for C++ unit tests
 
 project(${CMAKE_PROJECT_NAME}Tests CXX)
 
@@ -9,7 +9,6 @@ FetchContent_Declare(
   benchmark
   GIT_REPOSITORY https://github.com/google/benchmark.git
   GIT_TAG c19cfee61e136effb05a7fc8a037b0db3b13bd4c
-  GIT_SHALLOW TRUE
 )
 
 FetchContent_Declare(
-Original file line number
+Diff line change
@@ Expand Up / @@ -292,8 +292,6 @@ if(BUILD_TESTING) @@
       )
       declare_test_executable(test3 "dbfadd;dbfcreate;dbfdump;shpadd;shpcreate;shpdump")
-      include(CTest)
       add_subdirectory(tests)
     endif()
@@ Expand Down @@