From ab5992ad0e4d796828df9dd64288217999cfa2e2 Mon Sep 17 00:00:00 2001 From: Hennadii Stepanov <32963518+hebasto@users.noreply.github.com> Date: Mon, 22 Apr 2024 05:32:44 +0100 Subject: [PATCH] cmake [KILL 3-STATE]: Switch `WITH_USDT` to boolean w/ default OFF --- CMakeLists.txt | 10 +++++----- cmake/optional.cmake | 11 ----------- depends/Makefile | 2 +- depends/toolchain.cmake.in | 6 ++++-- 4 files changed, 10 insertions(+), 19 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 2f861f415fa52..a0278ada5018d 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -108,11 +108,11 @@ if(WITH_ZMQ) endif() endif() -tristate_option(WITH_USDT - "Enable tracepoints for Userspace, Statically Defined Tracing." - "if sys/sdt.h is found." - AUTO -) +option(WITH_USDT "Enable tracepoints for Userspace, Statically Defined Tracing." OFF) +if(WITH_USDT) + find_package(USDT MODULE REQUIRED) +endif() + cmake_dependent_option(WITH_EXTERNAL_SIGNER "Enable external signer support." ON "NOT WIN32" OFF) set(ENABLE_EXTERNAL_SIGNER ${WITH_EXTERNAL_SIGNER}) tristate_option(WITH_QRENCODE "Enable QR code support." "if libqrencode is found." AUTO) diff --git a/cmake/optional.cmake b/cmake/optional.cmake index 20ec0a9a9f9ca..78dfe5f291daa 100644 --- a/cmake/optional.cmake +++ b/cmake/optional.cmake @@ -50,17 +50,6 @@ if(CCACHE) mark_as_advanced(CCACHE_COMMAND) endif() -if(WITH_USDT) - find_package(USDT MODULE) - if(USDT_FOUND) - set(WITH_USDT ON) - elseif(WITH_USDT STREQUAL "AUTO") - set(WITH_USDT OFF) - else() - message(FATAL_ERROR "sys/sdt.h requested, but not found.") - endif() -endif() - if(ENABLE_WALLET) if(WITH_SQLITE) if(VCPKG_TARGET_TRIPLET) diff --git a/depends/Makefile b/depends/Makefile index 32f0fdd67646e..512351470e393 100644 --- a/depends/Makefile +++ b/depends/Makefile @@ -291,7 +291,7 @@ $(host_prefix)/toolchain.cmake : toolchain.cmake.in $(host_prefix)/.stamp_$(fina -e 's|@no_sqlite@|$(NO_SQLITE)|' \ -e 's|@no_upnp@|$(NO_UPNP)|' \ -e 's|@no_natpmp@|$(NO_NATPMP)|' \ - -e 's|@no_usdt@|$(NO_USDT)|' \ + -e 's|@no_usdt@|$(NO_USDT)|' -e 's|@usdt_packages@|$(usdt_packages_)|' \ -e 's|@no_harden@|$(NO_HARDEN)|' \ -e 's|@multiprocess@|$(MULTIPROCESS)|' \ $< > $@ diff --git a/depends/toolchain.cmake.in b/depends/toolchain.cmake.in index 1a6e6d9b99c9b..6e6eb30b53c45 100644 --- a/depends/toolchain.cmake.in +++ b/depends/toolchain.cmake.in @@ -158,8 +158,10 @@ else() set(WITH_NATPMP ON CACHE BOOL "") endif() -if(NOT WITH_USDT AND "@no_usdt@" STREQUAL "1") - set(WITH_USDT OFF CACHE STRING "Enable tracepoints for Userspace, Statically Defined Tracing.") +if("@no_usdt@" OR NOT "@usdt_packages@") + set(WITH_USDT OFF CACHE BOOL "") +else() + set(WITH_USDT ON CACHE BOOL "") endif() if("@no_harden@")