cmake: Improve Python robustness and test usability

hebasto commented at 1:45 pm on November 6, 2024: member

This PR:

Switches to a modern CMake approach by using the Python3::Interpreter imported target, which is more robust than using variables.
Disables the util_rpcauth_test test explicitly instead of silently ignoring it.

A build and test log for the case when Python is unavailable is provided below:

 0$ cmake -B build
 1$ cmake --build build -j 16
 2$ ctest --test-dir build -j $(nproc) -R "^util"
 3Internal ctest changing into directory: /bitcoin/build
 4Test project /bitcoin/build
 5    Start 115: util_tests
 6    Start 117: util_trace_tests
 7    Start 114: util_string_tests
 8    Start 116: util_threadnames_tests
 9    Start   1: util_rpcauth_test
101/5 Test   [#1](/bitcoin-bitcoin/1/): util_rpcauth_test ................***Not Run (Disabled)   0.00 sec
112/5 Test [#114](/bitcoin-bitcoin/114/): util_string_tests ................   Passed    0.11 sec
123/5 Test [#117](/bitcoin-bitcoin/117/): util_trace_tests .................   Passed    0.11 sec
134/5 Test [#116](/bitcoin-bitcoin/116/): util_threadnames_tests ...........   Passed    0.11 sec
145/5 Test [#115](/bitcoin-bitcoin/115/): util_tests .......................   Passed    0.13 sec
15
16100% tests passed, 0 tests failed out of 4
17
18Total Test time (real) =   0.13 sec
19
20The following tests did not run:
21	  1 - util_rpcauth_test (Disabled)

hebasto added the label Build system on Nov 6, 2024

hebasto added the label Tests on Nov 6, 2024

DrahtBot commented at 1:45 pm on November 6, 2024: 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/31233.

Reviews

See the guideline for information on the review process.

Type	Reviewers
ACK	purpleKarrot, janb84
Stale ACK	lucasbalieiro, BrandonOdiwuor

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

Conflicts

No conflicts as of last run.

hebasto commented at 3:35 pm on November 6, 2024: member

My Guix build:

 0aarch64
 1b4bd41285ec597b50489a3e3bf8ffde5e557d514ad5860fa3cfdd04da2144839  guix-build-4b6a842c2801/output/aarch64-linux-gnu/SHA256SUMS.part
 23222cf16fe93c3df73efc360150b384bf968f18ce929875bf384c9833e6411e7  guix-build-4b6a842c2801/output/aarch64-linux-gnu/bitcoin-4b6a842c2801-aarch64-linux-gnu-debug.tar.gz
 368cb1589be0fe45c8d9bd2dfa96b2610d49fcec5621b678a952db857493fdae9  guix-build-4b6a842c2801/output/aarch64-linux-gnu/bitcoin-4b6a842c2801-aarch64-linux-gnu.tar.gz
 49e289ec70fff76b6f6079ea852bb192bb97976618460404d46570992e6416964  guix-build-4b6a842c2801/output/arm-linux-gnueabihf/SHA256SUMS.part
 55c26ca52d1b93ac1356ee002fcdc772262dd8d7611da0b4295a286d512702e24  guix-build-4b6a842c2801/output/arm-linux-gnueabihf/bitcoin-4b6a842c2801-arm-linux-gnueabihf-debug.tar.gz
 66b2c340d82b641c58b956e27b8b034a72e791435cbc6c9de6551d6d359495cc7  guix-build-4b6a842c2801/output/arm-linux-gnueabihf/bitcoin-4b6a842c2801-arm-linux-gnueabihf.tar.gz
 7351619f4e964f8d2b7c8e22d1ddd6b1d0db0cf27c3e1be36c489a9962388a8b2  guix-build-4b6a842c2801/output/arm64-apple-darwin/SHA256SUMS.part
 8ea99f4078880f5233dd0e46666611839d52a9ffaf35fa4d85471f09e320da519  guix-build-4b6a842c2801/output/arm64-apple-darwin/bitcoin-4b6a842c2801-arm64-apple-darwin-unsigned.tar.gz
 98b78963b310762d2e0068bc8ba0b643ba4cae110d2942d659387fea807297718  guix-build-4b6a842c2801/output/arm64-apple-darwin/bitcoin-4b6a842c2801-arm64-apple-darwin-unsigned.zip
1092ac17a362ec939c33498179371224d60daeb0338ea482edffb7ce0cbea551d0  guix-build-4b6a842c2801/output/arm64-apple-darwin/bitcoin-4b6a842c2801-arm64-apple-darwin.tar.gz
11fece3efc27e98b0e163e79adf8e965cbff67037a6e73c217fb58bc40d19f7f1d  guix-build-4b6a842c2801/output/dist-archive/bitcoin-4b6a842c2801.tar.gz
12f9383706e867737e69cc6853487d88dcf92c5a1b1e350e27e69115a0853e8d4f  guix-build-4b6a842c2801/output/powerpc64-linux-gnu/SHA256SUMS.part
13c3a3a578d64a2721e1be9b92dd42d787c270a0fe5a16c9c4821abf833f2020e4  guix-build-4b6a842c2801/output/powerpc64-linux-gnu/bitcoin-4b6a842c2801-powerpc64-linux-gnu-debug.tar.gz
14fa2df5e6902c846657aa57d75d900010e2cc3f00ad946785fb6afb961b72b132  guix-build-4b6a842c2801/output/powerpc64-linux-gnu/bitcoin-4b6a842c2801-powerpc64-linux-gnu.tar.gz
15df2f59f8d71094fa4d2aa5eab39dce5136f004813746c5773bb793460d94cb45  guix-build-4b6a842c2801/output/riscv64-linux-gnu/SHA256SUMS.part
1692f62e9650540ac8cdb44b7cd24998e038dfcac69d9db17bfe6d09cd8df02c71  guix-build-4b6a842c2801/output/riscv64-linux-gnu/bitcoin-4b6a842c2801-riscv64-linux-gnu-debug.tar.gz
17ba982ed663b82f143b82807c873697eeea90aa99812d4663cd36d80dd66d2664  guix-build-4b6a842c2801/output/riscv64-linux-gnu/bitcoin-4b6a842c2801-riscv64-linux-gnu.tar.gz
18cf98209feab437849d8bf48839ae303310cd4b89fe4117556ff303a3f119471c  guix-build-4b6a842c2801/output/x86_64-apple-darwin/SHA256SUMS.part
19923859ff858ae7c521aea4a211752623133d52a38b475e335b66241aa97f5193  guix-build-4b6a842c2801/output/x86_64-apple-darwin/bitcoin-4b6a842c2801-x86_64-apple-darwin-unsigned.tar.gz
206398753ab5dc7a7976a7a109be9b5ed71259b4c9fc68d8312e936009f5c9ace8  guix-build-4b6a842c2801/output/x86_64-apple-darwin/bitcoin-4b6a842c2801-x86_64-apple-darwin-unsigned.zip
2113c6f9050ee33e1324b3cf65f97d5ab2540644953a1c7ce94c053e6395c9508a  guix-build-4b6a842c2801/output/x86_64-apple-darwin/bitcoin-4b6a842c2801-x86_64-apple-darwin.tar.gz
22fee8c5bf60d56acfd6cdaed763ac841e132eea3cfc839b0bc8d824bc5d554cb6  guix-build-4b6a842c2801/output/x86_64-linux-gnu/SHA256SUMS.part
23f92482e905e2949c1f34d60d4a895b2fafe09825ed00e5aa5401ae30d0976757  guix-build-4b6a842c2801/output/x86_64-linux-gnu/bitcoin-4b6a842c2801-x86_64-linux-gnu-debug.tar.gz
24d958174725702e4be7162c2b8ad93afd600ad10e9e82598e8584914bbea9fe5d  guix-build-4b6a842c2801/output/x86_64-linux-gnu/bitcoin-4b6a842c2801-x86_64-linux-gnu.tar.gz
2549a7b22039c44c4ed1c6324811d9043944665320560bc12dcbc7d3ae1321807a  guix-build-4b6a842c2801/output/x86_64-w64-mingw32/SHA256SUMS.part
2633c90eb5a0393e35418dcb1ddf333f90df35804c6a2ae1352aacdf5908fc1e3f  guix-build-4b6a842c2801/output/x86_64-w64-mingw32/bitcoin-4b6a842c2801-win64-debug.zip
274a24314797582e87ffdb7c36d248cd248682ceec62e7f419b39beb464ce6200a  guix-build-4b6a842c2801/output/x86_64-w64-mingw32/bitcoin-4b6a842c2801-win64-setup-unsigned.exe
287f1c314c88d858976f7c0c5f0fb49540d758ef585c41b8dcd884fd5e981dd941  guix-build-4b6a842c2801/output/x86_64-w64-mingw32/bitcoin-4b6a842c2801-win64-unsigned.tar.gz
294e7c8a0c6770b18072d7dd641cd4d1904e03728196e5c3c2a9a84a82fc27cdf1  guix-build-4b6a842c2801/output/x86_64-w64-mingw32/bitcoin-4b6a842c2801-win64.zip

lucasbalieiro approved

lucasbalieiro commented at 3:48 am on November 19, 2024: none

Hi, @hebasto!

tACK Commit 4b6a842

I have successfully run the build modifications on my machine:

Distributor: Ubuntu
Version: 22.04.5 LTS
Release: 22.04
Codename: Jammy

The build completed without any warnings or errors. I also ran the tests both with and without Python, and the console responses were as expected. Please see the attached images for further details.

Without Python
With Python
With -DBUILD_TX=OFF

For the “no Python” simulation, I commented out the following line in CMakeLists.txt:

0#find_package(Python3 3.10 COMPONENTS Interpreter)

Minor Recommendations (Really minor):

Verbosity for Missing Python:
Consider adding more verbosity when Python is not found. For example, in tests.cmake:
```
0if(NOT TARGET Python3::Interpreter)
1  message(WARNING "Python3 interpreter not found. Some tests will be disabled.")
2endif()
```
This could help clarify the situation when Python is missing. Example output:
Improving Readability with Variables:
For better readability, it might be beneficial to replace TARGET_EXISTS checks with variables. For example:
```
0set(PYTHON_AVAILABLE $<TARGET_EXISTS:Python3::Interpreter>)
1set(UTIL_AND_TX_AVAILABLE $<AND:$<TARGET_EXISTS:bitcoin-util>,$<TARGET_EXISTS:bitcoin-tx>>)
```
This would enhance clarity and make the conditions easier to follow.

This is my first PR review, and I want to ensure I’m doing it correctly. If there’s anything I missed or could improve in my testing approach, feel free to let me know.
Thanks for the contribution!

BrandonOdiwuor approved

BrandonOdiwuor commented at 12:28 pm on November 19, 2024: contributor

tACK 4b6a842c28010a00e121fd36cc0b4e1fa658d249 on Ubuntu 24.04

Was able to successfully test that util_test_runner and util_rpcauth_test are correctly disabled when Python3 interpreter is not found (screenshots below)

Overally this is an improvement, using the Python3::Interpreter target defined directly by find_package(…) is cleaner than relying on the ${PYTHON_COMMAND} variable we define

without Python3 Screenshot from 2024-11-19 15-24-13

With Python3 Screenshot from 2024-11-19 15-26-39

DrahtBot added the label Needs rebase on Feb 12, 2025

hebasto force-pushed on Feb 12, 2025

hebasto commented at 12:41 pm on February 12, 2025: member

Rebased due to the conflict with the merged bitcoin/bitcoin#31818.

DrahtBot removed the label Needs rebase on Feb 12, 2025

DrahtBot added the label Needs rebase on Feb 18, 2025

hebasto force-pushed on Feb 18, 2025

hebasto commented at 4:10 pm on February 18, 2025: member

Rebased due to the conflict with the merged bitcoin/bitcoin#31892.

DrahtBot removed the label Needs rebase on Feb 18, 2025

DrahtBot added the label Needs rebase on Apr 2, 2025

hebasto force-pushed on Apr 2, 2025

hebasto commented at 2:58 pm on April 2, 2025: member

Rebased due to a conflict with the merged bitcoin/bitcoin#30997.

DrahtBot removed the label Needs rebase on Apr 2, 2025

in cmake/tests.cmake:10 in 0f64a9d968 outdated

15+    BITCOINUTIL=set:$<$<TARGET_EXISTS:bitcoin-util>:$<TARGET_FILE:bitcoin-util>>
16+    BITCOINTX=set:$<$<TARGET_EXISTS:bitcoin-tx>:$<TARGET_FILE:bitcoin-tx>>
17+)
18+set_tests_properties(util_test_runner PROPERTIES
19+  DISABLED $<NOT:$<AND:$<TARGET_EXISTS:Python3::Interpreter>,$<TARGET_EXISTS:bitcoin-util>,$<TARGET_EXISTS:bitcoin-tx>>>
20+)

maflcko commented at 10:00 am on April 3, 2025:

I’ve asked this previously in #32183 (review), but the question also applies here. Especially, given that this is moving code around, which could just be deleted?

At least locally, on current master, this is passing:

 0diff --git a/cmake/tests.cmake b/cmake/tests.cmake
 1index 2791329800..c1d9bb31cf 100644
 2--- a/cmake/tests.cmake
 3+++ b/cmake/tests.cmake
 4@@ -4,7 +4,7 @@
 5 
 6 if(TARGET bitcoin-util AND TARGET bitcoin-tx AND PYTHON_COMMAND)
 7   add_test(NAME util_test_runner
 8-    COMMAND ${CMAKE_COMMAND} -E env BITCOINUTIL=$<TARGET_FILE:bitcoin-util> BITCOINTX=$<TARGET_FILE:bitcoin-tx> ${PYTHON_COMMAND} ${PROJECT_BINARY_DIR}/test/util/test_runner.py
 9+    COMMAND ${PYTHON_COMMAND} ${PROJECT_BINARY_DIR}/test/util/test_runner.py
10   )
11 endif()
12

hebasto commented at 10:27 am on April 22, 2025:

Answered in #32183 (review).

DrahtBot added the label CI failed on Apr 11, 2025

DrahtBot removed the label CI failed on Apr 11, 2025

hebasto force-pushed on Apr 22, 2025

hebasto commented at 12:54 pm on April 22, 2025: member

Rebased on top of #32324, which addresses #31233 (review), and drafted.

hebasto marked this as a draft on Apr 22, 2025

DrahtBot added the label Needs rebase on May 12, 2025

fanquake commented at 4:04 pm on June 18, 2025: member

Rebased on top of #32324,

That PR has now been closed, in favour of #32697.

Is this PR still an alternative to #31669.

in cmake/module/Maintenance.cmake:42 in fee2ccfb67 outdated

39   add_custom_target(check-security
40     COMMAND ${CMAKE_COMMAND} -E echo "Checking binary security..."
41-    COMMAND ${PYTHON_COMMAND} ${PROJECT_SOURCE_DIR}/contrib/devtools/security-check.py ${executables}
42+    COMMAND Python3::Interpreter ${PROJECT_SOURCE_DIR}/contrib/devtools/security-check.py ${executables}
43     VERBATIM
44   )

purpleKarrot commented at 11:27 am on June 28, 2025:

Maybe those two custom commands should also be turned into tests and disabled when python is not available?

hebasto commented at 10:12 am on July 1, 2025:

These custom targets are meaningful and run only within the context of the Guix environment. Currently, no tests are run there.

in cmake/tests.cmake:7 in 350210774e outdated

3@@ -4,7 +4,7 @@
4 
5 if(TARGET bitcoin-util AND TARGET bitcoin-tx AND PYTHON_COMMAND)
6   add_test(NAME util_test_runner
7-    COMMAND ${CMAKE_COMMAND} -E env BITCOINUTIL=$<TARGET_FILE:bitcoin-util> BITCOINTX=$<TARGET_FILE:bitcoin-tx> ${PYTHON_COMMAND} ${PROJECT_BINARY_DIR}/test/util/test_runner.py

purpleKarrot commented at 11:51 am on June 28, 2025:

If the test requires environment variables to be set, setting them in the github workflow is not a good approach. How are developers expected to reproduce that locally?

Better use the ENVIRONMENT property:

0set_property(TEST util_test_runner PROPERTY ENVIRONMENT
1  BITCOINUTIL=$<TARGET_FILE:bitcoin-util>
2  BITCOINTX=$<TARGET_FILE:bitcoin-tx>
3)

hebasto commented at 10:14 am on July 1, 2025:

Sure. However, this is no longer relevant following the recent branch update.

cmake, refactor: Switch to `Python3::Interpreter` imported target

Imported targets are more robust than using variables.

bb9157db5d

cmake, test: Disable tests instead of ignoring them 67dc7523f3

hebasto force-pushed on Jul 1, 2025

hebasto marked this as ready for review on Jul 1, 2025

hebasto commented at 10:43 am on July 1, 2025: member

Rebased and undrafted. The PR description has been updated as well.

Friendly ping @maflcko and @purpleKarrot :)

hebasto commented at 10:46 am on July 1, 2025: member

Is this PR still an alternative to #31669.

It’s related, but I wouldn’t consider it an alternative.

purpleKarrot commented at 11:02 am on July 1, 2025: contributor

ACK 67dc7523f3e103c8359b546d38f28c1feb2b9b34

The issue is specific to python. I leave it up to you and other reviewers whether that should be reworded or not.

DrahtBot requested review from BrandonOdiwuor on Jul 1, 2025

DrahtBot requested review from lucasbalieiro on Jul 1, 2025

janb84 commented at 11:52 am on July 1, 2025: contributor

tACK 67dc7523f3e103c8359b546d38f28c1feb2b9b34

The PR changes the usage of the Python3_EXECUTABLE to the (indeed) more modern target-based approach of TARGET Python3::Interpreter

test outcome:

no python installed:

cmake -S . -B build output gives warning:

0CMake Warning at CMakeLists.txt:710 (message):
1  Minimum required Python not found.  Rpcauth tests are disabled.

ctest --test-dir build -j 8 skips / disables util_rpcauth_test (as intended):

 0Start   1: util_rpcauth_test
 1  1/139 Test   [#1](/bitcoin-bitcoin/1/): util_rpcauth_test ....................***Not Run (Disabled)   0.00 sec
 2
 3137/139 Test   [#4](/bitcoin-bitcoin/4/): secp256k1_noverify_tests .............   Passed   15.44 sec
 4138/139 Test [#126](/bitcoin-bitcoin/126/): coinselector_tests ...................   Passed   11.16 sec
 5139/139 Test   [#5](/bitcoin-bitcoin/5/): secp256k1_tests ......................   Passed   29.23 sec
 6
 7100% tests passed, 0 tests failed out of 138
 8
 9Total Test time (real) =  29.24 sec
10
11The following tests did not run:
12          1 - util_rpcauth_test (Disabled)

(small) NIT maybe include python in the pr title because the PR has morphed to more a python robustness and usability PR. But no strong feelings.

DrahtBot removed the label Needs rebase on Jul 1, 2025

hebasto renamed this:
~~cmake: Improve robustness and usability~~
cmake: Improve Python robustness and test usability
on Jul 1, 2025

hebasto commented at 11:57 am on July 1, 2025: member

Updated the PR title.

fanquake commented at 2:27 pm on July 2, 2025: member

Guix Build:

 0cc67459424a62c706895f382cff677c1e0c0ecd7ee7f43429d64d1f5d872f151  guix-build-67dc7523f3e1/output/aarch64-linux-gnu/SHA256SUMS.part
 13c1eb6f629cc3b00d7e3803fc33383aea1f7cf6646d4de3e5c96d30923a1a795  guix-build-67dc7523f3e1/output/aarch64-linux-gnu/bitcoin-67dc7523f3e1-aarch64-linux-gnu-debug.tar.gz
 2494c55c6843e0d1f8aa133ee4c7685f06f04ca4c3dcb8ccee02aa11686a64350  guix-build-67dc7523f3e1/output/aarch64-linux-gnu/bitcoin-67dc7523f3e1-aarch64-linux-gnu.tar.gz
 3e12c0167012af21d9b49f2c9593f49eaf75f2f1a9dea30df9b498a75562ac00d  guix-build-67dc7523f3e1/output/arm-linux-gnueabihf/SHA256SUMS.part
 446c4a276bcb676a853e40d80e40495068333cef6389dec3b8fb324e62633c169  guix-build-67dc7523f3e1/output/arm-linux-gnueabihf/bitcoin-67dc7523f3e1-arm-linux-gnueabihf-debug.tar.gz
 531a3fd642a393b91030e2a6ff329091eb9cb8698a954fec335ea8a955bcb4968  guix-build-67dc7523f3e1/output/arm-linux-gnueabihf/bitcoin-67dc7523f3e1-arm-linux-gnueabihf.tar.gz
 618c2aec9b8ef2de41e517850782ce6fff47c9a27a208cfa233f957899330d825  guix-build-67dc7523f3e1/output/arm64-apple-darwin/SHA256SUMS.part
 793cf0d273ababea20ff1891e5523c34aa49db99fafa700c36d1384f3014b1aa7  guix-build-67dc7523f3e1/output/arm64-apple-darwin/bitcoin-67dc7523f3e1-arm64-apple-darwin-codesigning.tar.gz
 8a96ce467e74222633f9d2dc1ce03f7a71f56545fe9618fc5ccfb26fdbe6bfc0c  guix-build-67dc7523f3e1/output/arm64-apple-darwin/bitcoin-67dc7523f3e1-arm64-apple-darwin-unsigned.tar.gz
 96bafa4ceee017a00235aeb1af5e470900d0a657d7f16d85f2d65bf84e55099db  guix-build-67dc7523f3e1/output/arm64-apple-darwin/bitcoin-67dc7523f3e1-arm64-apple-darwin-unsigned.zip
10c974c8f7989adbf894b833e32d4441cca8b0edaa317ce851b6f511fe37377b3f  guix-build-67dc7523f3e1/output/dist-archive/bitcoin-67dc7523f3e1.tar.gz
1187dc02ed9c5b2a3e139afcc73b192dc21c1f9f275f65265c0158e01690aece43  guix-build-67dc7523f3e1/output/powerpc64-linux-gnu/SHA256SUMS.part
1288565d48b79941d1932783466620947c042a4f74449b607494a99c4306b1ddd0  guix-build-67dc7523f3e1/output/powerpc64-linux-gnu/bitcoin-67dc7523f3e1-powerpc64-linux-gnu-debug.tar.gz
13d04dffc416cd3cb617086a0c48361ccc2407d1d92520c836362fa9a051d0f779  guix-build-67dc7523f3e1/output/powerpc64-linux-gnu/bitcoin-67dc7523f3e1-powerpc64-linux-gnu.tar.gz
1489f171e0cb62956a53dfa09d0a30de600d1982455358348c4626a8ef5d09951a  guix-build-67dc7523f3e1/output/riscv64-linux-gnu/SHA256SUMS.part
15f281e4e30db28d38c19b60e4e4e6331c9efaabebe9deb5400042359c1d24bc7d  guix-build-67dc7523f3e1/output/riscv64-linux-gnu/bitcoin-67dc7523f3e1-riscv64-linux-gnu-debug.tar.gz
168e01bfff50f5b5e3c3149171d9a29c971dfeb29ce4462aaa64e8c3020e8162ae  guix-build-67dc7523f3e1/output/riscv64-linux-gnu/bitcoin-67dc7523f3e1-riscv64-linux-gnu.tar.gz
17751f807f7b5836d09a9533b4da5d1ae5e51d4aeba044ed617a2ac5a8bbb76f31  guix-build-67dc7523f3e1/output/x86_64-apple-darwin/SHA256SUMS.part
18300b8bce42ed4b3ff7306ffba671c1ebf753d6bfb0a3dcca3c9c78162ab84d87  guix-build-67dc7523f3e1/output/x86_64-apple-darwin/bitcoin-67dc7523f3e1-x86_64-apple-darwin-codesigning.tar.gz
19eff432177d074dfb45042cb04a9e2eb73f7b7ae556c61ab3ce86cbe90125435c  guix-build-67dc7523f3e1/output/x86_64-apple-darwin/bitcoin-67dc7523f3e1-x86_64-apple-darwin-unsigned.tar.gz
205c792873cffc31b6f0b43b7a010e166748f77fca2a0bc5c8060d7d858dbbad19  guix-build-67dc7523f3e1/output/x86_64-apple-darwin/bitcoin-67dc7523f3e1-x86_64-apple-darwin-unsigned.zip
21e791bc57cb6a8d64085309eb9a11505e547140129e771666ee25e2fc5010f3b4  guix-build-67dc7523f3e1/output/x86_64-linux-gnu/SHA256SUMS.part
22f2ca0521a9a241cce0d150999015f7379c42c80466bc50186eba0afca49fa280  guix-build-67dc7523f3e1/output/x86_64-linux-gnu/bitcoin-67dc7523f3e1-x86_64-linux-gnu-debug.tar.gz
23d5130d4215cd48a3a30c2ad65868edbdb715cc6526a9fe2cebca92da2a129660  guix-build-67dc7523f3e1/output/x86_64-linux-gnu/bitcoin-67dc7523f3e1-x86_64-linux-gnu.tar.gz
2479270f33123495325ea59b67f8ada022a593bce8402ce41cef7b99c219c405a6  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/SHA256SUMS.part
25efd18ffb494c2eb32f4deaf326d76fbee88d3f5d77f84939d3e48e3a67f41970  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/bitcoin-67dc7523f3e1-win64-codesigning.tar.gz
26a4523b2b33c552ac9104664ecb4a503f39eeef55bcbd3d2933e317ea2de8cb40  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/bitcoin-67dc7523f3e1-win64-debug.zip
274ddd9f94695e6f5590fa3f36a402e586b079facdf4ace7fd6b0ef9b3c1200a8d  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/bitcoin-67dc7523f3e1-win64-setup-unsigned.exe
283431db0566364e7074d0b0982ba998ba2dd8d008de27f334bc6ae891a1d1fc15  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/bitcoin-67dc7523f3e1-win64-unsigned.zip

hebasto commented at 4:13 pm on July 2, 2025: member

My Guix build:

 0aarch64
 1cc67459424a62c706895f382cff677c1e0c0ecd7ee7f43429d64d1f5d872f151  guix-build-67dc7523f3e1/output/aarch64-linux-gnu/SHA256SUMS.part
 23c1eb6f629cc3b00d7e3803fc33383aea1f7cf6646d4de3e5c96d30923a1a795  guix-build-67dc7523f3e1/output/aarch64-linux-gnu/bitcoin-67dc7523f3e1-aarch64-linux-gnu-debug.tar.gz
 3494c55c6843e0d1f8aa133ee4c7685f06f04ca4c3dcb8ccee02aa11686a64350  guix-build-67dc7523f3e1/output/aarch64-linux-gnu/bitcoin-67dc7523f3e1-aarch64-linux-gnu.tar.gz
 4e12c0167012af21d9b49f2c9593f49eaf75f2f1a9dea30df9b498a75562ac00d  guix-build-67dc7523f3e1/output/arm-linux-gnueabihf/SHA256SUMS.part
 546c4a276bcb676a853e40d80e40495068333cef6389dec3b8fb324e62633c169  guix-build-67dc7523f3e1/output/arm-linux-gnueabihf/bitcoin-67dc7523f3e1-arm-linux-gnueabihf-debug.tar.gz
 631a3fd642a393b91030e2a6ff329091eb9cb8698a954fec335ea8a955bcb4968  guix-build-67dc7523f3e1/output/arm-linux-gnueabihf/bitcoin-67dc7523f3e1-arm-linux-gnueabihf.tar.gz
 718c2aec9b8ef2de41e517850782ce6fff47c9a27a208cfa233f957899330d825  guix-build-67dc7523f3e1/output/arm64-apple-darwin/SHA256SUMS.part
 893cf0d273ababea20ff1891e5523c34aa49db99fafa700c36d1384f3014b1aa7  guix-build-67dc7523f3e1/output/arm64-apple-darwin/bitcoin-67dc7523f3e1-arm64-apple-darwin-codesigning.tar.gz
 9a96ce467e74222633f9d2dc1ce03f7a71f56545fe9618fc5ccfb26fdbe6bfc0c  guix-build-67dc7523f3e1/output/arm64-apple-darwin/bitcoin-67dc7523f3e1-arm64-apple-darwin-unsigned.tar.gz
106bafa4ceee017a00235aeb1af5e470900d0a657d7f16d85f2d65bf84e55099db  guix-build-67dc7523f3e1/output/arm64-apple-darwin/bitcoin-67dc7523f3e1-arm64-apple-darwin-unsigned.zip
11c974c8f7989adbf894b833e32d4441cca8b0edaa317ce851b6f511fe37377b3f  guix-build-67dc7523f3e1/output/dist-archive/bitcoin-67dc7523f3e1.tar.gz
1287dc02ed9c5b2a3e139afcc73b192dc21c1f9f275f65265c0158e01690aece43  guix-build-67dc7523f3e1/output/powerpc64-linux-gnu/SHA256SUMS.part
1388565d48b79941d1932783466620947c042a4f74449b607494a99c4306b1ddd0  guix-build-67dc7523f3e1/output/powerpc64-linux-gnu/bitcoin-67dc7523f3e1-powerpc64-linux-gnu-debug.tar.gz
14d04dffc416cd3cb617086a0c48361ccc2407d1d92520c836362fa9a051d0f779  guix-build-67dc7523f3e1/output/powerpc64-linux-gnu/bitcoin-67dc7523f3e1-powerpc64-linux-gnu.tar.gz
1589f171e0cb62956a53dfa09d0a30de600d1982455358348c4626a8ef5d09951a  guix-build-67dc7523f3e1/output/riscv64-linux-gnu/SHA256SUMS.part
16f281e4e30db28d38c19b60e4e4e6331c9efaabebe9deb5400042359c1d24bc7d  guix-build-67dc7523f3e1/output/riscv64-linux-gnu/bitcoin-67dc7523f3e1-riscv64-linux-gnu-debug.tar.gz
178e01bfff50f5b5e3c3149171d9a29c971dfeb29ce4462aaa64e8c3020e8162ae  guix-build-67dc7523f3e1/output/riscv64-linux-gnu/bitcoin-67dc7523f3e1-riscv64-linux-gnu.tar.gz
18751f807f7b5836d09a9533b4da5d1ae5e51d4aeba044ed617a2ac5a8bbb76f31  guix-build-67dc7523f3e1/output/x86_64-apple-darwin/SHA256SUMS.part
19300b8bce42ed4b3ff7306ffba671c1ebf753d6bfb0a3dcca3c9c78162ab84d87  guix-build-67dc7523f3e1/output/x86_64-apple-darwin/bitcoin-67dc7523f3e1-x86_64-apple-darwin-codesigning.tar.gz
20eff432177d074dfb45042cb04a9e2eb73f7b7ae556c61ab3ce86cbe90125435c  guix-build-67dc7523f3e1/output/x86_64-apple-darwin/bitcoin-67dc7523f3e1-x86_64-apple-darwin-unsigned.tar.gz
215c792873cffc31b6f0b43b7a010e166748f77fca2a0bc5c8060d7d858dbbad19  guix-build-67dc7523f3e1/output/x86_64-apple-darwin/bitcoin-67dc7523f3e1-x86_64-apple-darwin-unsigned.zip
22e791bc57cb6a8d64085309eb9a11505e547140129e771666ee25e2fc5010f3b4  guix-build-67dc7523f3e1/output/x86_64-linux-gnu/SHA256SUMS.part
23f2ca0521a9a241cce0d150999015f7379c42c80466bc50186eba0afca49fa280  guix-build-67dc7523f3e1/output/x86_64-linux-gnu/bitcoin-67dc7523f3e1-x86_64-linux-gnu-debug.tar.gz
24d5130d4215cd48a3a30c2ad65868edbdb715cc6526a9fe2cebca92da2a129660  guix-build-67dc7523f3e1/output/x86_64-linux-gnu/bitcoin-67dc7523f3e1-x86_64-linux-gnu.tar.gz
2579270f33123495325ea59b67f8ada022a593bce8402ce41cef7b99c219c405a6  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/SHA256SUMS.part
26efd18ffb494c2eb32f4deaf326d76fbee88d3f5d77f84939d3e48e3a67f41970  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/bitcoin-67dc7523f3e1-win64-codesigning.tar.gz
27a4523b2b33c552ac9104664ecb4a503f39eeef55bcbd3d2933e317ea2de8cb40  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/bitcoin-67dc7523f3e1-win64-debug.zip
284ddd9f94695e6f5590fa3f36a402e586b079facdf4ace7fd6b0ef9b3c1200a8d  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/bitcoin-67dc7523f3e1-win64-setup-unsigned.exe
293431db0566364e7074d0b0982ba998ba2dd8d008de27f334bc6ae891a1d1fc15  guix-build-67dc7523f3e1/output/x86_64-w64-mingw32/bitcoin-67dc7523f3e1-win64-unsigned.zip

fanquake merged this on Jul 3, 2025

fanquake closed this on Jul 3, 2025

hebasto deleted the branch on Jul 3, 2025

cmake: Improve Python robustness and test usability #31233

Code Coverage & Benchmarks

Reviews

Conflicts

Minor Recommendations (Really minor):