Skip to content

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

Merged
merged 2 commits into from
Oct 28, 2021

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Oct 27, 2021

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:

DeepinScreenshot_select-area_20211027182527

DeepinScreenshot_select-area_20211027182558

DeepinScreenshot_select-area_20211027182648

DeepinScreenshot_select-area_20211027182725

@hebasto
Copy link
Member Author

hebasto commented Oct 27, 2021

@Bosch-0

Is it what you have been expected for?

@hebasto hebasto added the UX Designers' opinions are required label Oct 27, 2021
@promag
Copy link
Contributor

promag commented Oct 27, 2021

ACK, very nice! I was able to download and run the artifact.

@GBKS
Copy link
Contributor

GBKS commented Oct 27, 2021

ACK. That is fantastic.

@hebasto
Copy link
Member Author

hebasto commented Oct 27, 2021

@promag @GBKS

Many thanks for your warm words!

To make your "ACK"s pickable by the github-merge.py script, could you update your comments by adding the top commit hash (d228a5c) to your "ACK"s, please?

@promag
Copy link
Contributor

promag commented Oct 27, 2021

ACK d228a5c.

For how long are the artifacts available?

@hebasto
Copy link
Member Author

hebasto commented Oct 27, 2021

For how long are the artifacts available?

It is not clear documented 🤷‍♂️

Copy link
Member

@jarolrod jarolrod left a 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.

@Bosch-0
Copy link
Contributor

Bosch-0 commented Oct 28, 2021

ACK from me on windows, looks great! This will speed up testing from designers a lot :)
Screenshot 2021-10-28 101016

@hebasto hebasto merged commit fa9abdf into bitcoin-core:main Oct 28, 2021
@hebasto hebasto deleted the 211027-win-art branch October 28, 2021 03:53
hebasto added a commit that referenced this pull request Oct 28, 2021
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"
Copy link
Contributor

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.

hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 6, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 7, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 7, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 8, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 8, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UX Designers' opinions are required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants