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
build, macos: Drop unused osx_volname
target
#29733
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. |
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 eff19fa
984c548 fixup! cmake: Migrate Guix build scripts to CMake (Hennadii Stepanov) fb23c32 fixup! cmake: Add `Maintenance` module (Hennadii Stepanov) 6ac45b7 refactor! cmake: Redefine/adjust per-configuration flags (Hennadii Stepanov) bce7bea fixup! build: Generate `share/toolchain.cmake` in depends (Hennadii Stepanov) Pull request description: What to test: 1. `cmake --build build && cmake --build build -t deploydir` and the following `cmake --build build -t deploy` when cross-compiling for macOS and on macOS natively. 2. Guix builds. This PR includes changes from bitcoin#29733. UPD. Also related: bitcoin#29761. ACKs for top commit: TheCharlatan: ACK 984c548 Tree-SHA512: be056492ee7f5f210afaef7574bdcc56f01be80e7db888b7571e36a4be50f90337d0f9f1a623588f8c91898665ff06b22300dec9292016f05f2d2913a823d2eb
ACK eff19fa |
tACK eff19fa Tested that
|
The
osx_volname
makefile target was introduced in #7192 and was used to pass the package name to Gitian scripts as a content of theosx_volname
file.With the current Guix scripts, the
osx_volname
file is never read. Therefore, its creation might be omitted.My Guix builds: