cmake: Make `BUILD_KERNEL_TEST` depend on `BUILD_KERNEL_LIB` #33972

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:251130-kernel-test changing 1 files +1 −1

hebasto commented at 1:45 PM on November 30, 2025: member
The CMake script in the test/kernel subdirectory is already gated by BUILD_KERNEL_LIB:https://github.com/bitcoin/bitcoin/blob/f6acbef1084e34f126bf530df99e4ef6a11c38e8/src/CMakeLists.txt#L405-L409

As a result, the following configuration summary is misleading:
```
$ cmake -B build -DBUILD_KERNEL_LIB=OFF -DBUILD_KERNEL_TEST=ON
<snip>
  bitcoin-chainstate (experimental) ... OFF
  libbitcoinkernel (experimental) ..... OFF
  kernel-test (experimental) .......... ON
<snip>
```
This PR fixes the behaviour by making the BUILD_KERNEL_TEST option explicitly depend on BUILD_KERNEL_LIB.
cmake: Make `BUILD_KERNEL_TEST` depend on `BUILD_KERNEL_LIB` fe1815d48f
hebasto added the label Build system on Nov 30, 2025
DrahtBot commented at 1:46 PM on November 30, 2025: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33972.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK maflcko, sedited

If your review is incorrectly listed, please copy-paste <code></code> into the comment that the bot should ignore.
hebasto commented at 1:47 PM on November 30, 2025: member

cc @sedited
maflcko commented at 8:18 AM on December 1, 2025: member

lgtm ACK fe1815d48f0cee57d2f1af50b377c7f9e462369e
sedited approved
sedited commented at 8:44 AM on December 1, 2025: contributor

ACK fe1815d48f0cee57d2f1af50b377c7f9e462369e
fanquake merged this on Dec 1, 2025
fanquake closed this on Dec 1, 2025
hebasto deleted the branch on Dec 1, 2025

Type	Reviewers
ACK	maflcko, sedited

Contributors

Labels

cmake: Make `BUILD_KERNEL_TEST` depend on `BUILD_KERNEL_LIB` #33972

Code Coverage & Benchmarks

Reviews