-
Notifications
You must be signed in to change notification settings - Fork 37.7k
build: swap cctools otool for llvm-objdump #29739
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. Code CoverageFor detailed information about the code coverage, see the test coverage report. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. ConflictsNo conflicts as of last run. |
0ab1866
to
33a6392
Compare
Tested 33a6392 depends build on Intel macOS 13.6.6 as well as 14.4.1, including I did run into a confusing, but unrelated (happens on 28f2ca6 too, see #29792), failure with bdb, only on macOS 13.6.6:
The folder On that machine I proceeded with CI seems unhappy for macOS cross:
|
33a6392
to
77eaee0
Compare
Also tested the guix build (made on AMD Ubuntu) for 77eaee0 on macOS 14.4.1.
|
Guix Build (aarch64): c39f93edbd44b97256834f1173f23c311a7f7fdf467ecebd4ec930e0691732ad guix-build-77eaee0ba46a/output/aarch64-linux-gnu/SHA256SUMS.part
4e88994692636055e105346229ad87fcb40569a984fddb278e92b78498a983ef guix-build-77eaee0ba46a/output/aarch64-linux-gnu/bitcoin-77eaee0ba46a-aarch64-linux-gnu-debug.tar.gz
f2d7c80d039ecfe9889eada5be4f701f413b22eb679511634a20545528b61a15 guix-build-77eaee0ba46a/output/aarch64-linux-gnu/bitcoin-77eaee0ba46a-aarch64-linux-gnu.tar.gz
df23e2877a33f7c7c2745949e64901c56322ec036407f9aac9525e06b2f7e9eb guix-build-77eaee0ba46a/output/arm-linux-gnueabihf/SHA256SUMS.part
850d8edfba1b36b5e246300adfa8f725f6202db033078da6e30d38ec9da78c77 guix-build-77eaee0ba46a/output/arm-linux-gnueabihf/bitcoin-77eaee0ba46a-arm-linux-gnueabihf-debug.tar.gz
c4b27197411da1126837e8b25bf074f70c36caf1f9b54432d1e5eaa588a9df02 guix-build-77eaee0ba46a/output/arm-linux-gnueabihf/bitcoin-77eaee0ba46a-arm-linux-gnueabihf.tar.gz
4fd8566ccc8229c4784eac5cc21d077b179b4165b457911fe142ea2e3b0069e7 guix-build-77eaee0ba46a/output/arm64-apple-darwin/SHA256SUMS.part
6cca0ae159eddbf88dd1858eec51625816346487fc5eb9e5260037c686e8e312 guix-build-77eaee0ba46a/output/arm64-apple-darwin/bitcoin-77eaee0ba46a-arm64-apple-darwin-unsigned.tar.gz
c967afbfd76efb0a16e54524734396e56afe249273620f59904253a8608d1416 guix-build-77eaee0ba46a/output/arm64-apple-darwin/bitcoin-77eaee0ba46a-arm64-apple-darwin-unsigned.zip
3d28e921e71106996348e8a483adc1e9e24441fa21455a74323e836a92b01588 guix-build-77eaee0ba46a/output/arm64-apple-darwin/bitcoin-77eaee0ba46a-arm64-apple-darwin.tar.gz
ea71bc724ce8214e97a226279b0c6eb4c392d401b1f3b8a126140467160e379a guix-build-77eaee0ba46a/output/dist-archive/bitcoin-77eaee0ba46a.tar.gz
4ba4b49a414cddf6da484762a91969a6e6ad9a055365118ac5cd2c4045b34bd8 guix-build-77eaee0ba46a/output/powerpc64-linux-gnu/SHA256SUMS.part
79ea63ee2018b020851f089d6c34d70a18c112af125ad12496df4ccf8942b235 guix-build-77eaee0ba46a/output/powerpc64-linux-gnu/bitcoin-77eaee0ba46a-powerpc64-linux-gnu-debug.tar.gz
a6d8ba0e924a743ae1a1b834afdee238957487f6cb5d56a67bd0c4452bf5c468 guix-build-77eaee0ba46a/output/powerpc64-linux-gnu/bitcoin-77eaee0ba46a-powerpc64-linux-gnu.tar.gz
db34798d4529aadd80ba8ccefbb70be637ba9a8ab8fbbb9485e205d3d6945c96 guix-build-77eaee0ba46a/output/riscv64-linux-gnu/SHA256SUMS.part
2127f321726156fa44022384bea4ca899f09b9c3986ff17910e306844ed77270 guix-build-77eaee0ba46a/output/riscv64-linux-gnu/bitcoin-77eaee0ba46a-riscv64-linux-gnu-debug.tar.gz
73d7f7b9db8eb35754463513448e5be6fbdea0b3dcbb9c213ebea0556fd3a3e4 guix-build-77eaee0ba46a/output/riscv64-linux-gnu/bitcoin-77eaee0ba46a-riscv64-linux-gnu.tar.gz
abe2a8abf88b59982b46f8e7012fa981adbacaf1b424877456754f1c1d331d23 guix-build-77eaee0ba46a/output/x86_64-apple-darwin/SHA256SUMS.part
172b34356fde06ee17a09218c169ae2e4a3c223534a123f9aa67a5f86bded8d2 guix-build-77eaee0ba46a/output/x86_64-apple-darwin/bitcoin-77eaee0ba46a-x86_64-apple-darwin-unsigned.tar.gz
8c3f82577227c0b7133873896c360faf194a4523bb8d9d1602bd97fef2a6beb9 guix-build-77eaee0ba46a/output/x86_64-apple-darwin/bitcoin-77eaee0ba46a-x86_64-apple-darwin-unsigned.zip
bae74b14820d06c7667278f320937e2f86a25ae0efc2d94b2a2c905c26c15b2e guix-build-77eaee0ba46a/output/x86_64-apple-darwin/bitcoin-77eaee0ba46a-x86_64-apple-darwin.tar.gz
6a42ab303c741779c501e85bf7428a3588c16c9eafbda3657de4cc3fcf52753c guix-build-77eaee0ba46a/output/x86_64-linux-gnu/SHA256SUMS.part
24a368dd727c900adae144967d27ae58893edb6f8821a48d506c9a4ec80d40a9 guix-build-77eaee0ba46a/output/x86_64-linux-gnu/bitcoin-77eaee0ba46a-x86_64-linux-gnu-debug.tar.gz
8430b7ca91b94da095add1db6d46c74a09c791351c81dfeb552c9732cdd54dc2 guix-build-77eaee0ba46a/output/x86_64-linux-gnu/bitcoin-77eaee0ba46a-x86_64-linux-gnu.tar.gz
77485baa9d8edb6ee240dbcd17e6b8ec0613144dc1b3765cb6f1e8142745bc82 guix-build-77eaee0ba46a/output/x86_64-w64-mingw32/SHA256SUMS.part
aba6e456fabfce78727f912fa6be21f880e4e2922f33221eefb26356f5984545 guix-build-77eaee0ba46a/output/x86_64-w64-mingw32/bitcoin-77eaee0ba46a-win64-debug.zip
1ad214989983c5c54a15d1af7aa25e9ea50d2b7d4ba6aeefecd7b7aee7774db3 guix-build-77eaee0ba46a/output/x86_64-w64-mingw32/bitcoin-77eaee0ba46a-win64-setup-unsigned.exe
e02826473de7db61094d71f59084d0e002393ef58c45c420b08ddc3c41a7211f guix-build-77eaee0ba46a/output/x86_64-w64-mingw32/bitcoin-77eaee0ba46a-win64-unsigned.tar.gz
6b682937c9895475851f20ececee42bef44ffa9fa55d4eecc68b7de75407efad guix-build-77eaee0ba46a/output/x86_64-w64-mingw32/bitcoin-77eaee0ba46a-win64.zip |
Tested the cross-compiled binaries and deployed bundle on macos 12.7.4. |
Guix builds (x86_64):
|
6623a33
to
f4a3bd7
Compare
This was actually just masking a bug. Dropped and rebased on #29890, which has simplified things here. |
🚧 At least one of the CI tasks failed. Make sure to run all tests locally, according to the Possibly this is due to a silent merge conflict (the changes in this pull request being Leave a comment here, if you need help tracking down a confusing failure. |
1a9aa8d build: better scope usage of -Wl,-headerpad_max_install_names (fanquake) 3bee514 build: don't use install_name_tool for macOS deploy when cross-compiling (fanquake) 78b6b5c build: don't pass strip to macOS deploy if cross-compiling (fanquake) Pull request description: Neither of these tools are actually used when we are cross-compiling for macOS. They are used when we have to adjust non-static libs during a deploy after building on a macOS machine. Simplies #29739 (will be rebased on top). Guix (aarch64): ```bash 8f29bce75d7f574306a0e38d793e0e4e145b547a4b9e5a755a54976121d8ac41 guix-build-5afd3c302051/output/arm64-apple-darwin/SHA256SUMS.part 9ba01fe46be715adcbe80f39dc7dbe449f32ca9d9b660da698f933aef3e6d80b guix-build-5afd3c302051/output/arm64-apple-darwin/bitcoin-5afd3c302051-arm64-apple-darwin-unsigned.tar.gz 37719437e951449341d0e10dcc4afe93e955d59de5312ce6351e1fa01b4927ac guix-build-5afd3c302051/output/arm64-apple-darwin/bitcoin-5afd3c302051-arm64-apple-darwin-unsigned.zip 06a79fc871dcd4290f5f7e7e9de19a5a535203d20279f4555d1c319d07abe2d0 guix-build-5afd3c302051/output/arm64-apple-darwin/bitcoin-5afd3c302051-arm64-apple-darwin.tar.gz 98d2b8b37197dcad36a04eb2f3ff2130b859220a17b83a4186a78dcf0af4eafd guix-build-5afd3c302051/output/dist-archive/bitcoin-5afd3c302051.tar.gz df63ff44ef41565ff13ce6dde5485173a18d5866ebc316df86f9ebd91fda18f5 guix-build-5afd3c302051/output/x86_64-apple-darwin/SHA256SUMS.part 28362ce9e80d5e78db198efa5f89434fbe76ca91df5fde7455da4d50ceb8523a guix-build-5afd3c302051/output/x86_64-apple-darwin/bitcoin-5afd3c302051-x86_64-apple-darwin-unsigned.tar.gz 534745b679eb9e8e408dd251a6bf0829e62e12f7a41772b8a57a044ded14208c guix-build-5afd3c302051/output/x86_64-apple-darwin/bitcoin-5afd3c302051-x86_64-apple-darwin-unsigned.zip f53d0c9a1bb83d548c7d274c7d39653a3989fb1b4efec49e73dd1cac7c92074c guix-build-5afd3c302051/output/x86_64-apple-darwin/bitcoin-5afd3c302051-x86_64-apple-darwin.tar.gz ``` ACKs for top commit: TheCharlatan: ACK 1a9aa8d Tree-SHA512: 0aa77ea4d6dc45c226806bb1758b6aa7e8ca17f91045bab4fc6891af7b9de476211cd5692c11cb9d5bcf59744fd86a2534812a77fe304ae10c3518e08fc412be
f4a3bd7
to
7a49b7f
Compare
7990c72
to
f93bc3d
Compare
@fanquake I get the same hashes on AMD Ubuntu 24.04 |
Guix builds (on x86_64) [untrusted test-only build, possibly unsafe, not for production use]
|
Guix build (aarch64):
|
Similar to libtool, (llvm-)otool only exists with a version suffix on some systems (Ubuntu), which makes it annoying to use/find. Avoid this, by switching to objdump. Which is a drop-in replacement. This is related to bitcoin#21778, and the switchover to using vanilla LLVM for macOS.
4313feb
to
7f5ac45
Compare
Dropped the last commit just to make this simpler to review. The swap to objdump is the most complex of the cctools -> llvm swaps, because of the associated Python changes, all the of the rest are easier, and can be done in 1 go in a followup. |
Guix build (aarch64): 67f4db93b4f985590948dc344306042841ab747b40ad73bf97ba62d77eb7bfd4 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/SHA256SUMS.part
62e019f164d3c8cf8a1a649b2caa500dd85b00a319910feaf97658020b8cd1e3 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/bitcoin-7f5ac4520d15-aarch64-linux-gnu-debug.tar.gz
c2227a058cf91c0bc3eb244cb254c3c6fcecddc21ba619bca8839d9d79bb1961 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/bitcoin-7f5ac4520d15-aarch64-linux-gnu.tar.gz
e794a50c8aeebefa61531469208080bbba7dc1d3a4658b0aedeb9988428c9feb guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/SHA256SUMS.part
c5681536ffe85b27a078540db918a3113351b5a5da75f9d77709a422e0e22129 guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/bitcoin-7f5ac4520d15-arm-linux-gnueabihf-debug.tar.gz
fe9cb7ae413614baffedb7c4fd16254aed816d4a79b0d267d5482dc3000b6dad guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/bitcoin-7f5ac4520d15-arm-linux-gnueabihf.tar.gz
1d1a5b78a5cae8f0fa34a88168a34e0bf0aadaaaf29360109a089222824c11dc guix-build-7f5ac4520d15/output/arm64-apple-darwin/SHA256SUMS.part
64bef8bef26d24d3276f4eee20161e22f645f55fb1cdff39dd0c6a9df434f633 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin-unsigned.tar.gz
dea89a35fd4bec7945e9e735e6057ab462b8ec3914fb2b5265bc72cca5ced616 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin-unsigned.zip
25fafdc462c06c59cbb4ac0ac12a77fa33a0b742ac23584258a4558f1c3fe797 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin.tar.gz
e307d1b0d3b39aadb8cc794e00f0acbc4289794feb266f22c8d9013ab9869444 guix-build-7f5ac4520d15/output/dist-archive/bitcoin-7f5ac4520d15.tar.gz
9b5a40d7838bfe268443cf594a6798b6be3b9e2b875ae6e0084f6894d592ac3c guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/SHA256SUMS.part
fa95ecef8e32b7e4bea3d6c4a6910a3d13ac87c46324f9769fe1aeb990d2830d guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/bitcoin-7f5ac4520d15-powerpc64-linux-gnu-debug.tar.gz
b9697fa2633d080b14db8abd2f7219e3aa41171aefb76eeb99090027cb8bb8b7 guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/bitcoin-7f5ac4520d15-powerpc64-linux-gnu.tar.gz
b288387bf7959fd91aa676a0be643496f47118fcbbf6ce36f8d7dc4e7cef08e8 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/SHA256SUMS.part
0795fc420ad52e3f599d8b7e1c86c09fb7ca53a6171008cf8a6d205ddd273357 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/bitcoin-7f5ac4520d15-riscv64-linux-gnu-debug.tar.gz
4cc70dd8da7b48ad810f14190a19e1475a58f753922f177943449806af550387 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/bitcoin-7f5ac4520d15-riscv64-linux-gnu.tar.gz
af2d4af34b801209a85b8ef6d149f5fa152348134ea9abe7e38cb50d5e5f4b85 guix-build-7f5ac4520d15/output/x86_64-apple-darwin/SHA256SUMS.part
7066c328aac9138479a174e5b62fad4dce68a2a225ca9639020a9567a0e3022a guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin-unsigned.tar.gz
fd2301f76637d62912130996bd6f9d6162f934bf8ba7cea32b8508f90bc59e75 guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin-unsigned.zip
cbbfa6d6b2e4530071164f87389a0e55104417f0123d8bff29d519f72a992f7a guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin.tar.gz
33c2e40a65b0ad415028ff6d01a568bd566bd677ae078dc0aac6b347e72ac7f7 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/SHA256SUMS.part
b666cf8de8a98019b7d18511cb4ed3bc85b0a0e44af669754c0d3306e8925341 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/bitcoin-7f5ac4520d15-x86_64-linux-gnu-debug.tar.gz
570e71ac049b8093a4b4afdb45a24e694585d8587c09db53687630da97b3b1c9 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/bitcoin-7f5ac4520d15-x86_64-linux-gnu.tar.gz
6313e18382824d8ab6b2db58f6f12d5a23df71e419dfcc5fca6f8fab3b2e1dc4 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/SHA256SUMS.part
074b9d6c52920f86ae161010d819ae9e80a1fa814fb148eb7726b211a9827541 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-debug.zip
0748410dd75c764c443fe519398ede89b70857f6f6a58cbb12e1d9cf7ce42d20 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-setup-unsigned.exe
bd9edbc9dd51a147175bef1443e1c2b419b804252c2b0ef14ae49639381586bc guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-unsigned.tar.gz
8d74c85baaedaf9dadec59cf68babd56cc6869eca6eae3fce494453dd80c99d4 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64.zip |
Guix builds (aarch64):
There is a mismatch on the windows debug zip :/ |
AMD Ubuntu:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 7f5ac45. Tested make deploy
on native macOS. Looks good.
I've rebuilt, and see matching now (you and Sjors): 67f4db93b4f985590948dc344306042841ab747b40ad73bf97ba62d77eb7bfd4 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/SHA256SUMS.part
62e019f164d3c8cf8a1a649b2caa500dd85b00a319910feaf97658020b8cd1e3 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/bitcoin-7f5ac4520d15-aarch64-linux-gnu-debug.tar.gz
c2227a058cf91c0bc3eb244cb254c3c6fcecddc21ba619bca8839d9d79bb1961 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/bitcoin-7f5ac4520d15-aarch64-linux-gnu.tar.gz
e794a50c8aeebefa61531469208080bbba7dc1d3a4658b0aedeb9988428c9feb guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/SHA256SUMS.part
c5681536ffe85b27a078540db918a3113351b5a5da75f9d77709a422e0e22129 guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/bitcoin-7f5ac4520d15-arm-linux-gnueabihf-debug.tar.gz
fe9cb7ae413614baffedb7c4fd16254aed816d4a79b0d267d5482dc3000b6dad guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/bitcoin-7f5ac4520d15-arm-linux-gnueabihf.tar.gz
1d1a5b78a5cae8f0fa34a88168a34e0bf0aadaaaf29360109a089222824c11dc guix-build-7f5ac4520d15/output/arm64-apple-darwin/SHA256SUMS.part
64bef8bef26d24d3276f4eee20161e22f645f55fb1cdff39dd0c6a9df434f633 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin-unsigned.tar.gz
dea89a35fd4bec7945e9e735e6057ab462b8ec3914fb2b5265bc72cca5ced616 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin-unsigned.zip
25fafdc462c06c59cbb4ac0ac12a77fa33a0b742ac23584258a4558f1c3fe797 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin.tar.gz
e307d1b0d3b39aadb8cc794e00f0acbc4289794feb266f22c8d9013ab9869444 guix-build-7f5ac4520d15/output/dist-archive/bitcoin-7f5ac4520d15.tar.gz
9b5a40d7838bfe268443cf594a6798b6be3b9e2b875ae6e0084f6894d592ac3c guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/SHA256SUMS.part
fa95ecef8e32b7e4bea3d6c4a6910a3d13ac87c46324f9769fe1aeb990d2830d guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/bitcoin-7f5ac4520d15-powerpc64-linux-gnu-debug.tar.gz
b9697fa2633d080b14db8abd2f7219e3aa41171aefb76eeb99090027cb8bb8b7 guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/bitcoin-7f5ac4520d15-powerpc64-linux-gnu.tar.gz
b288387bf7959fd91aa676a0be643496f47118fcbbf6ce36f8d7dc4e7cef08e8 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/SHA256SUMS.part
0795fc420ad52e3f599d8b7e1c86c09fb7ca53a6171008cf8a6d205ddd273357 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/bitcoin-7f5ac4520d15-riscv64-linux-gnu-debug.tar.gz
4cc70dd8da7b48ad810f14190a19e1475a58f753922f177943449806af550387 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/bitcoin-7f5ac4520d15-riscv64-linux-gnu.tar.gz
af2d4af34b801209a85b8ef6d149f5fa152348134ea9abe7e38cb50d5e5f4b85 guix-build-7f5ac4520d15/output/x86_64-apple-darwin/SHA256SUMS.part
7066c328aac9138479a174e5b62fad4dce68a2a225ca9639020a9567a0e3022a guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin-unsigned.tar.gz
fd2301f76637d62912130996bd6f9d6162f934bf8ba7cea32b8508f90bc59e75 guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin-unsigned.zip
cbbfa6d6b2e4530071164f87389a0e55104417f0123d8bff29d519f72a992f7a guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin.tar.gz
33c2e40a65b0ad415028ff6d01a568bd566bd677ae078dc0aac6b347e72ac7f7 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/SHA256SUMS.part
b666cf8de8a98019b7d18511cb4ed3bc85b0a0e44af669754c0d3306e8925341 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/bitcoin-7f5ac4520d15-x86_64-linux-gnu-debug.tar.gz
570e71ac049b8093a4b4afdb45a24e694585d8587c09db53687630da97b3b1c9 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/bitcoin-7f5ac4520d15-x86_64-linux-gnu.tar.gz
c120be9d2aa8b5883f0ef8762b5e3093a8d0a2f034b456750085fe1f50dca455 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/SHA256SUMS.part
91433dd01de315488753edc3e13f7300c53231eb3e7dab55d233631b95a33e9f guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-debug.zip
0748410dd75c764c443fe519398ede89b70857f6f6a58cbb12e1d9cf7ce42d20 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-setup-unsigned.exe
bd9edbc9dd51a147175bef1443e1c2b419b804252c2b0ef14ae49639381586bc guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-unsigned.tar.gz
8d74c85baaedaf9dadec59cf68babd56cc6869eca6eae3fce494453dd80c99d4 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64.zip |
Also refired and got the same hashes again. Might have been something else unrelated to this PR? ACK 7f5ac45 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My Guix build:
67f4db93b4f985590948dc344306042841ab747b40ad73bf97ba62d77eb7bfd4 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/SHA256SUMS.part
62e019f164d3c8cf8a1a649b2caa500dd85b00a319910feaf97658020b8cd1e3 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/bitcoin-7f5ac4520d15-aarch64-linux-gnu-debug.tar.gz
c2227a058cf91c0bc3eb244cb254c3c6fcecddc21ba619bca8839d9d79bb1961 guix-build-7f5ac4520d15/output/aarch64-linux-gnu/bitcoin-7f5ac4520d15-aarch64-linux-gnu.tar.gz
e794a50c8aeebefa61531469208080bbba7dc1d3a4658b0aedeb9988428c9feb guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/SHA256SUMS.part
c5681536ffe85b27a078540db918a3113351b5a5da75f9d77709a422e0e22129 guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/bitcoin-7f5ac4520d15-arm-linux-gnueabihf-debug.tar.gz
fe9cb7ae413614baffedb7c4fd16254aed816d4a79b0d267d5482dc3000b6dad guix-build-7f5ac4520d15/output/arm-linux-gnueabihf/bitcoin-7f5ac4520d15-arm-linux-gnueabihf.tar.gz
1d1a5b78a5cae8f0fa34a88168a34e0bf0aadaaaf29360109a089222824c11dc guix-build-7f5ac4520d15/output/arm64-apple-darwin/SHA256SUMS.part
64bef8bef26d24d3276f4eee20161e22f645f55fb1cdff39dd0c6a9df434f633 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin-unsigned.tar.gz
dea89a35fd4bec7945e9e735e6057ab462b8ec3914fb2b5265bc72cca5ced616 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin-unsigned.zip
25fafdc462c06c59cbb4ac0ac12a77fa33a0b742ac23584258a4558f1c3fe797 guix-build-7f5ac4520d15/output/arm64-apple-darwin/bitcoin-7f5ac4520d15-arm64-apple-darwin.tar.gz
e307d1b0d3b39aadb8cc794e00f0acbc4289794feb266f22c8d9013ab9869444 guix-build-7f5ac4520d15/output/dist-archive/bitcoin-7f5ac4520d15.tar.gz
9b5a40d7838bfe268443cf594a6798b6be3b9e2b875ae6e0084f6894d592ac3c guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/SHA256SUMS.part
fa95ecef8e32b7e4bea3d6c4a6910a3d13ac87c46324f9769fe1aeb990d2830d guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/bitcoin-7f5ac4520d15-powerpc64-linux-gnu-debug.tar.gz
b9697fa2633d080b14db8abd2f7219e3aa41171aefb76eeb99090027cb8bb8b7 guix-build-7f5ac4520d15/output/powerpc64-linux-gnu/bitcoin-7f5ac4520d15-powerpc64-linux-gnu.tar.gz
b288387bf7959fd91aa676a0be643496f47118fcbbf6ce36f8d7dc4e7cef08e8 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/SHA256SUMS.part
0795fc420ad52e3f599d8b7e1c86c09fb7ca53a6171008cf8a6d205ddd273357 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/bitcoin-7f5ac4520d15-riscv64-linux-gnu-debug.tar.gz
4cc70dd8da7b48ad810f14190a19e1475a58f753922f177943449806af550387 guix-build-7f5ac4520d15/output/riscv64-linux-gnu/bitcoin-7f5ac4520d15-riscv64-linux-gnu.tar.gz
af2d4af34b801209a85b8ef6d149f5fa152348134ea9abe7e38cb50d5e5f4b85 guix-build-7f5ac4520d15/output/x86_64-apple-darwin/SHA256SUMS.part
7066c328aac9138479a174e5b62fad4dce68a2a225ca9639020a9567a0e3022a guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin-unsigned.tar.gz
fd2301f76637d62912130996bd6f9d6162f934bf8ba7cea32b8508f90bc59e75 guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin-unsigned.zip
cbbfa6d6b2e4530071164f87389a0e55104417f0123d8bff29d519f72a992f7a guix-build-7f5ac4520d15/output/x86_64-apple-darwin/bitcoin-7f5ac4520d15-x86_64-apple-darwin.tar.gz
33c2e40a65b0ad415028ff6d01a568bd566bd677ae078dc0aac6b347e72ac7f7 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/SHA256SUMS.part
b666cf8de8a98019b7d18511cb4ed3bc85b0a0e44af669754c0d3306e8925341 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/bitcoin-7f5ac4520d15-x86_64-linux-gnu-debug.tar.gz
570e71ac049b8093a4b4afdb45a24e694585d8587c09db53687630da97b3b1c9 guix-build-7f5ac4520d15/output/x86_64-linux-gnu/bitcoin-7f5ac4520d15-x86_64-linux-gnu.tar.gz
c120be9d2aa8b5883f0ef8762b5e3093a8d0a2f034b456750085fe1f50dca455 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/SHA256SUMS.part
91433dd01de315488753edc3e13f7300c53231eb3e7dab55d233631b95a33e9f guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-debug.zip
0748410dd75c764c443fe519398ede89b70857f6f6a58cbb12e1d9cf7ce42d20 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-setup-unsigned.exe
bd9edbc9dd51a147175bef1443e1c2b419b804252c2b0ef14ae49639381586bc guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64-unsigned.tar.gz
8d74c85baaedaf9dadec59cf68babd56cc6869eca6eae3fce494453dd80c99d4 guix-build-7f5ac4520d15/output/x86_64-w64-mingw32/bitcoin-7f5ac4520d15-win64.zip
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 7f5ac45.
After migration to CMake, there is a possibility that CMake will expect to find otool
for the "Darwin" systems. But we can deal with this in due time.
Ported to the CMake-based build system in hebasto#202. |
, bitcoin#28622, bitcoin#28880, bitcoin#29185, bitcoin#29170, bitcoin#29233, bitcoin#29298, bitcoin#29598, bitcoin#29732, bitcoin#29890, bitcoin#29739, bitcoin#30074, bitcoin#30198, bitcoin#29072 (toolchain backports: part 2) 1506d9d merge bitcoin#29072: use `-no_exported_symbols` on macOS (Kittywhiskers Van Gogh) 9247960 merge bitcoin#30198: qt 5.15.14 and fix macOS build with Clang 18 (Kittywhiskers Van Gogh) 5585e7a merge bitcoin#30074: use ENV flags in get_arch (Kittywhiskers Van Gogh) decd420 merge bitcoin#29739: swap cctools otool for llvm-objdump (Kittywhiskers Van Gogh) 0f8c420 merge bitcoin#29890: remove some tools when cross-compiling for macOS (Kittywhiskers Van Gogh) 936da1a merge bitcoin#29732: qt 5.15.13 (Kittywhiskers Van Gogh) c294b47 revert: patch qt to make placeholders differ from actual text (Kittywhiskers Van Gogh) af7090c merge bitcoin#29598: don't use -h with touch on OpenBSD (Kittywhiskers Van Gogh) ebf8ff2 merge bitcoin#29298: patch libtool out of libnatpmp/miniupnpc (Kittywhiskers Van Gogh) 070b876 merge bitcoin#29233: depends move macOS C(XX) FLAGS out of C & CXX (Kittywhiskers Van Gogh) d838481 revert dash#2398: Force fvisibility=hidden when compiling on macos (Kittywhiskers Van Gogh) 59a18f9 merge bitcoin#29170: add macho branch protection check (Kittywhiskers Van Gogh) cb024d9 merge bitcoin#29185: remove `--enable-lto` (Kittywhiskers Van Gogh) 6d75a81 merge bitcoin#28880: switch to using LLVM 17.x for macOS builds (Kittywhiskers Van Gogh) 7b0a1f2 merge bitcoin#28622: use macOS 14 SDK (Xcode 15.0) (Kittywhiskers Van Gogh) 02eb735 merge bitcoin#24948: fix typo in permissions (Kittywhiskers Van Gogh) 2739107 merge bitcoin#24534: make gen-sdk deterministic (Kittywhiskers Van Gogh) ab10bf9 merge bitcoin#24241: cleanup doc on need of Developer Account to obtain macOS SDK (Kittywhiskers Van Gogh) Pull request description: ## Additional Information * Dependent on #6384 * Dependency for #6389 * The Qt patch introduced in [dash#5596](#5596), `fix_qt_placeholders.patch`, was a portion of a suggested workaround for QTBUG-92199 ([source](https://bugreports.qt.io/browse/QTBUG-92199?focusedId=669719&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-669719)) but since then, a fix ([here](https://codereview.qt-project.org/c/qt/qtbase/+/434310)) has made its way to 5.15.12 and we are upgrading to 5.15.14 from 5.15.11. So we can safely remove this patch. ## Breaking Changes None expected ## Checklist - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas **(note: N/A)** - [x] I have added or updated relevant unit/integration/functional/e2e tests - [x] I have made corresponding changes to the documentation **(note: N/A)** - [x] I have assigned this pull request to a milestone _(for repository code-owners and collaborators only)_ ACKs for top commit: UdjinM6: utACK 1506d9d PastaPastaPasta: utACK 1506d9d Tree-SHA512: df8e4ea0ce9e7b269d248518698f0566b5eca1a54cdfb53f5b213b90fb5177e5a5df44eaeb6f3fc014cd93351c9245736bb2fd52bc2af4ae274d8fa93e601b07
Summary: ``` Similar to libtool, (llvm-)otool only exists with a version suffix on some systems (Ubuntu), which makes it annoying to use/find. Avoid this, by switching to objdump. Which is a drop-in replacement. This is related to #21778, and the switchover to using vanilla LLVM for macOS. ``` Backport of [[bitcoin/bitcoin#29739 | core#29739]]. Depends on D17258. Test Plan: Run the GUIX osx build Reviewers: #bitcoin_abc, PiRK Reviewed By: #bitcoin_abc, PiRK Subscribers: PiRK Differential Revision: https://reviews.bitcoinabc.org/D17259
Summary: ``` Similar to libtool, (llvm-)otool only exists with a version suffix on some systems (Ubuntu), which makes it annoying to use/find. Avoid this, by switching to objdump. Which is a drop-in replacement. This is related to #21778, and the switchover to using vanilla LLVM for macOS. ``` Backport of [[bitcoin/bitcoin#29739 | core#29739]]. Depends on D17258. Test Plan: Run the GUIX osx build Reviewers: #bitcoin_abc, PiRK Reviewed By: #bitcoin_abc, PiRK Subscribers: PiRK Differential Revision: https://reviews.bitcoinabc.org/D17259
This tool is used in GUI packaging on macOS, and also somewhat of a blocker for #21778. The main issue is that some distros don't really ship this tool in a standard ways, i.e Ubuntu only ships
llvm-otool
with a version suffix, i.ellvm-otool-17
, which makes it hard to find and use. Rather than trying to deal with that mess, switch to using the equivalent LLVM tool (objdump), which is a drop-in replacement.