-
Notifications
You must be signed in to change notification settings - Fork 50
ci: Make Win64 static bitcoin-qt.exe available from CI task #71
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
Is it what you have been expected for? |
ACK, very nice! I was able to download and run the artifact. |
ACK. That is fantastic. |
ACK d228a5c. For how long are the artifacts available? |
It is not clear documented 🤷♂️ |
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 d228a5c
Tested by downloading and running the resulting binary on Windows 10. Will add the motivation behind providing artifacts along with how to get the link to them and run them in a productivity doc.
b1baa79 ci: Add artifacts instruction to macOS task (Hennadii Stepanov) Pull request description: This PR makes `bitcoin-qt` available for downloading from CI for each PR. Being downloaded it is required to make it executable with `chmod +x bitcoin-qt`. Similar to #71. A link to download: https://api.cirrus-ci.com/v1/artifact/build/5223626531143680/macos_bitcoin_qt.zip ACKs for top commit: promag: Tested ACK b1baa79. jarolrod: ACK b1baa79 Tree-SHA512: eb339ca7fae0686fda924365f5ce03f282eeb55b50dc35936b75fdaba89e4ad2e85eb7c6ee166985de548c5246dadf1b8f76039b518a7d833609fe11e0b1286d
@@ -173,6 +173,8 @@ task: | |||
env: | |||
<< : *CIRRUS_EPHEMERAL_WORKER_TEMPLATE_ENV | |||
FILE_ENV: "./ci/test/00_setup_env_win64.sh" | |||
bitcoin_qt_artifacts: | |||
path: "ci/scratch/build/bitcoin-x86_64-w64-mingw32/release/bitcoin-qt.exe" |
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.
Same feedback here as #73.
Github-Pull: bitcoin-core#71 Rebased-From: 500a6d2
Github-Pull: bitcoin-core#71 Rebased-From: 500a6d2
Github-Pull: bitcoin-core#71 Rebased-From: 500a6d2
Github-Pull: bitcoin-core#71 Rebased-From: 500a6d2
Github-Pull: bitcoin-core#71 Rebased-From: 500a6d2
Github-Pull: bitcoin-core#71 Rebased-From: 500a6d2
Github-Pull: bitcoin-core#71 Rebased-From: 500a6d2
This PR makes
bitcoin-qt.exe
available for downloading from CI for each PR.This should make testing PRs by designers a bit easier.
If this idea will work for Windows, the similar approach can be expected for macOS binaries.
A link to download: https://api.cirrus-ci.com/v1/artifact/build/4640743399620608/bitcoin_qt.zip
To get the unique
4640743399620608
identifier, one can follow the links as depicted on screenshots: