cmake: Check dependencies after build option interaction #33974

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:251130-cmake-options changing 2 files +90 −73
  1. hebasto commented at 3:50 pm on November 30, 2025: member

    At present, CMakeLists.txt interleaves configuration-option handling with dependency discovery. As a result, unnecessary checks may be performed. For example:

     0$ cmake -B build --preset dev-mode -DBUILD_FOR_FUZZING=ON
     1<snip>
     2-- Found PkgConfig: /usr/bin/pkg-config (found version "2.3.0")
     3-- Found ZeroMQ: /usr/lib64 (found suitable version "4.3.5", minimum required is "4.0.0")
     4-- Performing Test HAVE_USDT_H
     5-- Performing Test HAVE_USDT_H - Success
     6-- Found USDT: /usr/include
     7-- Found QRencode: /usr/lib64/libqrencode.so (found version "4.1.1")
     8-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
     9-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
    10-- Found Threads: TRUE
    11-- Performing Test HAVE_STDATOMIC
    12-- Performing Test HAVE_STDATOMIC - Success
    13-- Found WrapAtomic: TRUE
    14-- Found OpenGL: /usr/lib64/libOpenGL.so
    15-- Found WrapOpenGL: TRUE
    16-- Found WrapVulkanHeaders: /usr/include
    17-- Found Qt: /usr/lib64/cmake/Qt6 (found suitable version "6.10.1", minimum required is "6.2")
    18CMake Warning at CMakeLists.txt:206 (message):
    19  BUILD_FOR_FUZZING=ON will disable all other targets and force
    20  BUILD_FUZZ_BINARY=ON.
    21
    22<snip>
    

    This PR restructures the code to ensure that all dependencies are checked only after the final build option interaction.

  2. hebasto added the label Build system on Nov 30, 2025
  3. DrahtBot commented at 3:51 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/33974.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    Stale ACK BrandonOdiwuor, sedited

    If your review is incorrectly listed, please copy-paste <!–meta-tag:bot-skip–> into the comment that the bot should ignore.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #34411 ([POC] Full Libevent removal by fanquake)
    • #33861 (build: Bump VS minimum supported version to 18.0 by hebasto)
    • #31723 (qa: Facilitate debugging bitcoind inside functional tests by hodlinator)
    • #31507 (build: Use clang-cl to build on Windows natively by hebasto)
    • #31425 (RFC: Riscv bare metal CI job by sedited)
    • #28792 (build: Embedded ASMap [3/3]: Build binary dump header file by fjahr)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. DrahtBot added the label CI failed on Nov 30, 2025
  5. hebasto force-pushed on Nov 30, 2025
  6. DrahtBot removed the label CI failed on Nov 30, 2025
  7. DrahtBot added the label Needs rebase on Dec 1, 2025
  8. hebasto force-pushed on Dec 1, 2025
  9. DrahtBot removed the label Needs rebase on Dec 1, 2025
  10. in CMakeLists.txt:256 in 56d0a0929d
    251+    find_package(QRencode MODULE REQUIRED)
    252+    set(USE_QRCODE TRUE)
    253+  endif()
    254+endif()
    255+
    256+
    


    sedited commented at 7:53 am on December 5, 2025:
    Nit: Extra line.

    hebasto commented at 12:15 pm on December 5, 2025:
    Thanks! Fixed.
  11. sedited approved
  12. sedited commented at 10:58 am on December 5, 2025: contributor

    ACK 56d0a0929d8974a490c57bf84a994ba7c9f19863

    Guix build:

     05bd1a78f590d48e5ceb5815b5ed346d06d738871c9ea26b754c9a851dcb7ee85  guix-build-56d0a0929d89/output/aarch64-linux-gnu/SHA256SUMS.part
     1cc0525310c1e1c1b1cb69c2256b8089b187830c27da9dd7ba9830884c35a0e95  guix-build-56d0a0929d89/output/aarch64-linux-gnu/bitcoin-56d0a0929d89-aarch64-linux-gnu-debug.tar.gz
     2bc891ef995772a5f09c48000b0b0e7d6dc142ae184490ef711c800f42c48cdeb  guix-build-56d0a0929d89/output/aarch64-linux-gnu/bitcoin-56d0a0929d89-aarch64-linux-gnu.tar.gz
     3c1da46ecfaec3c3e040a6504182faf5f114cdbb69926757b1b10192e11919c2c  guix-build-56d0a0929d89/output/arm-linux-gnueabihf/SHA256SUMS.part
     45d5e6287f0369f662526d3af08041e7544bed163c41421d8c81be97b522d0c89  guix-build-56d0a0929d89/output/arm-linux-gnueabihf/bitcoin-56d0a0929d89-arm-linux-gnueabihf-debug.tar.gz
     5b4752c479a4a26930d00eb7b775f70744d73a6adbb06ea6d1f5a9542841ab994  guix-build-56d0a0929d89/output/arm-linux-gnueabihf/bitcoin-56d0a0929d89-arm-linux-gnueabihf.tar.gz
     6f73e98957bad390793b82c9d0d60d1733723e57335bcd2535d942e3931297219  guix-build-56d0a0929d89/output/arm64-apple-darwin/SHA256SUMS.part
     7a82dea7564ffa2fe233d758778315aebcb1d9df4f0e5c637d2068512c5a8377a  guix-build-56d0a0929d89/output/arm64-apple-darwin/bitcoin-56d0a0929d89-arm64-apple-darwin-codesigning.tar.gz
     8f1efcb20c5c4316b54b29b3cbb7b13346df2429d21cb80cf539397b879ce490e  guix-build-56d0a0929d89/output/arm64-apple-darwin/bitcoin-56d0a0929d89-arm64-apple-darwin-unsigned.tar.gz
     94fb3b6f31c33b06cfa209235efa45129b4fc0b6914614f9760d697563d62e286  guix-build-56d0a0929d89/output/arm64-apple-darwin/bitcoin-56d0a0929d89-arm64-apple-darwin-unsigned.zip
    10f0403ed60ea3aa384c072fb669e42991336a1b0e55e469e31832a56ee39d77a3  guix-build-56d0a0929d89/output/dist-archive/bitcoin-56d0a0929d89.tar.gz
    113a14924b8009db324534962b571987a5cc739e79206193bddf486af4d4b1d311  guix-build-56d0a0929d89/output/powerpc64-linux-gnu/SHA256SUMS.part
    1214015dd634a12d7280f1a6566ee751985b94b6fc2977dcd145d1e3e7a19becd5  guix-build-56d0a0929d89/output/powerpc64-linux-gnu/bitcoin-56d0a0929d89-powerpc64-linux-gnu-debug.tar.gz
    13b20eab5ad318339ef0ef68246780836a3ea0a50a9d7f9ff321082bffcbe628b8  guix-build-56d0a0929d89/output/powerpc64-linux-gnu/bitcoin-56d0a0929d89-powerpc64-linux-gnu.tar.gz
    143c95edab2bf41192162392cd5fac69973974cbba4098e79da26abdc26ddf2a02  guix-build-56d0a0929d89/output/riscv64-linux-gnu/SHA256SUMS.part
    155897b2f595df0121e12075373343d6244375fb54067af692d6ed220b6925fcba  guix-build-56d0a0929d89/output/riscv64-linux-gnu/bitcoin-56d0a0929d89-riscv64-linux-gnu-debug.tar.gz
    169494532acad032a435b4cc0c6b0c2faef8e333a7b561515d236ed98acafff038  guix-build-56d0a0929d89/output/riscv64-linux-gnu/bitcoin-56d0a0929d89-riscv64-linux-gnu.tar.gz
    1758f758b7db06e5f76457f366b86a16b6d4994274c8dfedd3f6a4bb7c8938bc02  guix-build-56d0a0929d89/output/x86_64-apple-darwin/SHA256SUMS.part
    188720f905960ba7fc7e7625614236b587176380bb0b33cbe2c3ccc28ea1af7929  guix-build-56d0a0929d89/output/x86_64-apple-darwin/bitcoin-56d0a0929d89-x86_64-apple-darwin-codesigning.tar.gz
    19655b68f2ae7ef9cf4f54be47cda3fec98aa8b086bd616921a61904109da15108  guix-build-56d0a0929d89/output/x86_64-apple-darwin/bitcoin-56d0a0929d89-x86_64-apple-darwin-unsigned.tar.gz
    205020c9f35a2584cb7fcf210815b53f509b8da624ffe7ce953b22f1bfe0510d2a  guix-build-56d0a0929d89/output/x86_64-apple-darwin/bitcoin-56d0a0929d89-x86_64-apple-darwin-unsigned.zip
    2105fda737070cc821214064a2d5ef9bc15fc0b6b76d2df3b2b64c52d8c309c115  guix-build-56d0a0929d89/output/x86_64-linux-gnu/SHA256SUMS.part
    22a23759a540eb48c6695b0583fe30fbc994f9a89b70de996499e7304bd24e9b59  guix-build-56d0a0929d89/output/x86_64-linux-gnu/bitcoin-56d0a0929d89-x86_64-linux-gnu-debug.tar.gz
    231de1e23e698a6f23b9e69c96e65237f2a26a77d3da19b6fefe9386d7ada7f076  guix-build-56d0a0929d89/output/x86_64-linux-gnu/bitcoin-56d0a0929d89-x86_64-linux-gnu.tar.gz
    24df33c8b9e987fbeb0aa2fbfada49b24c596b63a4a9f6c6daf56168ade90bfb14  guix-build-56d0a0929d89/output/x86_64-w64-mingw32/SHA256SUMS.part
    2590fdb394150627d7dd85c407d56ae6a50cc7fb4e64ab95d916523515b4879724  guix-build-56d0a0929d89/output/x86_64-w64-mingw32/bitcoin-56d0a0929d89-win64-codesigning.tar.gz
    26b5d3df18b6f6de5809da5fdec697f2e4fcbc8b4635331d4214507b4c40b837a0  guix-build-56d0a0929d89/output/x86_64-w64-mingw32/bitcoin-56d0a0929d89-win64-debug.zip
    273902aa61740e4f74d4aa39b64849a1234c3bf28524298c382c4db556e1b5977f  guix-build-56d0a0929d89/output/x86_64-w64-mingw32/bitcoin-56d0a0929d89-win64-setup-unsigned.exe
    28eb3bccca9af379f2cee04fbe42c0441988ff558c514c04d9d7fa003f325a13ac  guix-build-56d0a0929d89/output/x86_64-w64-mingw32/bitcoin-56d0a0929d89-win64-unsigned.zip
    
  13. in CMakeLists.txt:158 in 56d0a0929d outdated
    153@@ -202,6 +154,9 @@ target_link_libraries(core_interface INTERFACE
    154   $<$<CONFIG:Debug>:core_interface_debug>
    155 )
    156 
    157+#=============================
    158+# Option interaction
    


    fanquake commented at 11:05 am on December 5, 2025:
    What is this comment meant to be?

    hebasto commented at 11:08 am on December 5, 2025:
    Some build options, such as BUILD_FOR_FUZZING, may alter other user-specified build option values depending on how they are set.

    fanquake commented at 11:11 am on December 5, 2025:
    Can the comment explain that?

    fanquake commented at 11:16 am on December 5, 2025:
    Also, is this meant to mark that no interaction happens after this point, or is the only place there is any interaction?

    hebasto commented at 12:14 pm on December 5, 2025:
    An explanatory comment has been added.
  14. hebasto force-pushed on Dec 5, 2025
  15. hebasto commented at 12:14 pm on December 5, 2025: member

    @sedited @fanquake

    Thank you for the review! Your feedback has been addressed.

  16. BrandonOdiwuor commented at 2:25 pm on December 21, 2025: contributor

    Tested ACK 05b92e1ffb0cfd2a388fcf05e3aaf2337c50781e

    Confirmed that dependency checks now occur after build option resolution, eliminating unnecessary scans (e.g., ZeroMQ, QRencode, QT) in specialized builds like fuzzing.

    Tested on Ubuntu 24.04 aarch64 cmake version 3.28.3

    Tested on master

    0$ cmake -B build --preset dev-mode -DBUILD_FOR_FUZZING=ON
    

    Tested on 05b92e1ffb0cfd2a388fcf05e3aaf2337c50781e

    0$ cmake -B build --preset dev-mode -DBUILD_FOR_FUZZING=ON
    
  17. DrahtBot requested review from sedited on Dec 21, 2025
  18. sedited approved
  19. sedited commented at 1:24 pm on January 25, 2026: contributor
    Re-ACK 05b92e1ffb0cfd2a388fcf05e3aaf2337c50781e
  20. DrahtBot added the label Needs rebase on Feb 5, 2026
  21. cmake: Check dependencies after build option interaction 00fc68fa0a
  22. hebasto force-pushed on Feb 14, 2026
  23. hebasto commented at 10:12 pm on February 14, 2026: member
    Rebased to resolve conflicts.
  24. DrahtBot removed the label Needs rebase on Feb 15, 2026
  25. hebasto commented at 9:18 am on February 15, 2026: member

    My Guix build:

     0aarch64
     124ac0893e16e7e8a1a0300c86c335c79a548a67b75835e794e53fbbf631e37d3  guix-build-00fc68fa0aa5/output/aarch64-linux-gnu/SHA256SUMS.part
     261ad90ac626226b0fa16a1b57dd05f3c3984369b1d83c68d96da1556bcee6a8d  guix-build-00fc68fa0aa5/output/aarch64-linux-gnu/bitcoin-00fc68fa0aa5-aarch64-linux-gnu-debug.tar.gz
     37aba7cec5d0e5f9fac539c4ccbfcab53cdc5b024fb71ac76c4646a3dacfd4aba  guix-build-00fc68fa0aa5/output/aarch64-linux-gnu/bitcoin-00fc68fa0aa5-aarch64-linux-gnu.tar.gz
     4e0d9ab3d8184953ae2c3a77739c92e78ffcf3205f48be05874e6562edc5d8467  guix-build-00fc68fa0aa5/output/arm-linux-gnueabihf/SHA256SUMS.part
     555c2450a08e99f04aa8c49fbaadc75b186c61a3ed1b84f79c1079db4e4a822f2  guix-build-00fc68fa0aa5/output/arm-linux-gnueabihf/bitcoin-00fc68fa0aa5-arm-linux-gnueabihf-debug.tar.gz
     6d5e5468a05329f9f9d8f6d4a7785c2e3f47de272c4d7855a348c2e7553986345  guix-build-00fc68fa0aa5/output/arm-linux-gnueabihf/bitcoin-00fc68fa0aa5-arm-linux-gnueabihf.tar.gz
     7cfe3deb18e02021fbeed156929ae8b4dedb8c011cc4bbe71bdb01ca5074840aa  guix-build-00fc68fa0aa5/output/arm64-apple-darwin/SHA256SUMS.part
     8ecf3c55a606620386a9b6fb5625148007b9e63aaa52da204e7d608e4e6f1825b  guix-build-00fc68fa0aa5/output/arm64-apple-darwin/bitcoin-00fc68fa0aa5-arm64-apple-darwin-codesigning.tar.gz
     91a48b33530473a156571a47e5047381530f61810ecc9625b784e751028f6ad6b  guix-build-00fc68fa0aa5/output/arm64-apple-darwin/bitcoin-00fc68fa0aa5-arm64-apple-darwin-unsigned.tar.gz
    107344c91c0b0867ba8fa42321041b69b9fc791b085d697fb831845a735d7e74bc  guix-build-00fc68fa0aa5/output/arm64-apple-darwin/bitcoin-00fc68fa0aa5-arm64-apple-darwin-unsigned.zip
    115d1d6755dc080661556556bcc005fe14559661934994cbf4763bd9f16ce02d31  guix-build-00fc68fa0aa5/output/dist-archive/bitcoin-00fc68fa0aa5.tar.gz
    12832e5f53eb8132f91470cc71d1543a42c44ef73476d4e8eaa319f4f3221a5065  guix-build-00fc68fa0aa5/output/powerpc64-linux-gnu/SHA256SUMS.part
    13a90c345cab8218658ed64d139fc91bd971a6a16e943394a6eee89d2d04f2278b  guix-build-00fc68fa0aa5/output/powerpc64-linux-gnu/bitcoin-00fc68fa0aa5-powerpc64-linux-gnu-debug.tar.gz
    14092ebe9a6d075d4e3901232dc6bac235b147ee3762b61ef3b25ce3014abd4b05  guix-build-00fc68fa0aa5/output/powerpc64-linux-gnu/bitcoin-00fc68fa0aa5-powerpc64-linux-gnu.tar.gz
    15f594fd95a5c56742f9a0aac9669a8a7b7c2638967257c8ade96d42c2afde6cbd  guix-build-00fc68fa0aa5/output/riscv64-linux-gnu/SHA256SUMS.part
    1662c6fb8192632d12da5aadb1c49a0068931d873d71bcf92d5c570032eb35df52  guix-build-00fc68fa0aa5/output/riscv64-linux-gnu/bitcoin-00fc68fa0aa5-riscv64-linux-gnu-debug.tar.gz
    17a33bad6d80e585c5a2f5100b40c151ed3458b38a601bd56f760b22353a95a614  guix-build-00fc68fa0aa5/output/riscv64-linux-gnu/bitcoin-00fc68fa0aa5-riscv64-linux-gnu.tar.gz
    187831630e9afa008b59092fcd3eb6ae1eda4b9b3bc92ee07eed3180c9b5897bda  guix-build-00fc68fa0aa5/output/x86_64-apple-darwin/SHA256SUMS.part
    19a2429e27d81a19ce3525f9453785f89e586f99f1fb6425d2fd76c1aa2ceee44d  guix-build-00fc68fa0aa5/output/x86_64-apple-darwin/bitcoin-00fc68fa0aa5-x86_64-apple-darwin-codesigning.tar.gz
    20d19d4d60f1a4b414d5f0131562857e314ff475fc14945b190e750441581cce8a  guix-build-00fc68fa0aa5/output/x86_64-apple-darwin/bitcoin-00fc68fa0aa5-x86_64-apple-darwin-unsigned.tar.gz
    21ecd856595f2fb9720502da5186a0cf7c78a2c10110d6af4e3f7483c37a924a79  guix-build-00fc68fa0aa5/output/x86_64-apple-darwin/bitcoin-00fc68fa0aa5-x86_64-apple-darwin-unsigned.zip
    22a37d5582f074684178f570decd795a0c4b14732d11e154c942b0f5f9db6a1fe9  guix-build-00fc68fa0aa5/output/x86_64-linux-gnu/SHA256SUMS.part
    23515fb95ff95f9dc99687e8ce568197d4bdbf7214d0d666597a6b58f03e0916fc  guix-build-00fc68fa0aa5/output/x86_64-linux-gnu/bitcoin-00fc68fa0aa5-x86_64-linux-gnu-debug.tar.gz
    249386eb06ac8892cc0f0317a990352024dd99757c156ab675e529a45cca508f55  guix-build-00fc68fa0aa5/output/x86_64-linux-gnu/bitcoin-00fc68fa0aa5-x86_64-linux-gnu.tar.gz
    25ad1a314b700192da81ced14cd7fd173e4158f860cecb76959b90b26b323da171  guix-build-00fc68fa0aa5/output/x86_64-w64-mingw32/SHA256SUMS.part
    268b72f09497fc4389cb2f6d14d163aa561e5f5dfe9d7950df8be620761c53b22b  guix-build-00fc68fa0aa5/output/x86_64-w64-mingw32/bitcoin-00fc68fa0aa5-win64-codesigning.tar.gz
    27f534a4c8acdf1d90acbc55184254c19e32fd32a446eeed0e697d4480a78890a2  guix-build-00fc68fa0aa5/output/x86_64-w64-mingw32/bitcoin-00fc68fa0aa5-win64-debug.zip
    2844311fc0d042462a3d099468c54a083526edfcafff4e5cc8a1d9a905714fdf49  guix-build-00fc68fa0aa5/output/x86_64-w64-mingw32/bitcoin-00fc68fa0aa5-win64-setup-unsigned.exe
    293d2c1e4a53cec5dec1836815ede9555d42da379321a1812ce5d338dda5b8ecbb  guix-build-00fc68fa0aa5/output/x86_64-w64-mingw32/bitcoin-00fc68fa0aa5-win64-unsigned.zip
    

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-02-18 06:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me