Skip to content

Commit 011fc53

Browse files
committed
Merge #161: cmake: Avoid including CTest if not top level project
a7f0669 cmake: Avoid including CTest if not top level project (Ryan Ofsky) Pull request description: Avoid including CTest in if this is not a top-level cmake project because as reported #145 (comment) this adds a `BUILD_TESTING` option could be confused for Bitcoin core's `BUILD_TESTS` option. Top commit has no ACKs. Tree-SHA512: c28d8c9a43f23d7d1e91c914c777695fe422105a347133194b547e28217dd94be5418a1384b8c83b41fc173ee1b49fcd4656114f75d169d0cabc78ea2615d681
2 parents 26b9f3d + a7f0669 commit 011fc53

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

CMakeLists.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,13 @@ set(MPGEN_EXECUTABLE "" CACHE FILEPATH "If specified, should be full path to an
2828

2929
include("cmake/compat_config.cmake")
3030
include("cmake/pthread_checks.cmake")
31-
include(CTest)
3231
include(GNUInstallDirs)
3332

3433
# Set convenience variables for subdirectories.
3534
set(MP_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include")
3635
if(CMAKE_SOURCE_DIR STREQUAL CMAKE_CURRENT_SOURCE_DIR)
3736
set(MP_STANDALONE TRUE)
37+
include(CTest)
3838
else()
3939
# Set MP_INCLUDE_DIR for parent directories too, so target_capnp_sources calls
4040
# in parent directories can use it and not need to specify include directories

0 commit comments

Comments
 (0)