Merge #145: CTest: Module must be included at the top level

ryanofsky · ryanofsky · commit c978878a3cd8 · 2025-02-04T22:39:58.000-05:00
63ac092 CTest: Module must be included at the top level (Daniel Pfeifer) Pull request description: In order to be able to run `ctest` from the top level build directory, it is necessary that `enable_testing()` is invoked in the top level CMakeLists.txt file. The `CTest` module invokes `enable_testing()` based on the value of `BUILD_TESTING`. ACKs for top commit: hebasto: ACK 63ac092, tested on Ubuntu 24.04: Tree-SHA512: 042b3b4575e90db0c4c181d70407577ae373bc35faf9d4cccaf2816250746b76d6457f05bbd0e949cf6bc0bdb3beaf0d7f2c71e1b44c8aecc0215bdafa4a3837
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -28,6 +28,7 @@ set(Libmultiprocess_EXTERNAL_MPGEN "" CACHE FILEPATH "If specified, should be fu
 
 include("cmake/compat_config.cmake")
 include("cmake/pthread_checks.cmake")
+include(CTest)
 include(GNUInstallDirs)
 
 # Set convenience variables for subdirectories.
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -2,8 +2,6 @@
 # Distributed under the MIT software license, see the accompanying
 # file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
-include(CTest)
-
 # Custom test targets for convenience, based on
 # https://gitlab.kitware.com/cmake/community/-/wikis/doc/tutorials/EmulateMakeCheck.
 #

Original file line number	Diff line number	Diff line change
`@@ -2,8 +2,6 @@`
`2`	`2`	`# Distributed under the MIT software license, see the accompanying`
`3`	`3`	`# file COPYING or http://www.opensource.org/licenses/mit-license.php.`
`4`	`4`
`5`		`-include(CTest)`
`6`		`-`
`7`	`5`	`# Custom test targets for convenience, based on`
`8`	`6`	`# https://gitlab.kitware.com/cmake/community/-/wikis/doc/tutorials/EmulateMakeCheck.`
`9`	`7`	`#`