Skip to content
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

ci, gha: Build macOS and Windows artifacts #365

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Sep 9, 2023

This PR reintroduces downloadable binary artifacts for macOS and Windows as it was on now decommissioned Cirrus CI.

Here are a few implementation details:


The artifacts are available for downloading from https://github.com/bitcoin-core/gui-qml/actions/workflows/artifacts.yml.

Also refer to GitHub's documentation: https://docs.github.com/en/actions/managing-workflow-runs/downloading-workflow-artifacts.

Links for macOS and Windows build artifacts:

macOS Apple Silicon
macOS Intel
Windows

@hebasto hebasto force-pushed the 230909-gha-cross branch 2 times, most recently from e7a71b6 to a9fd4f9 Compare September 9, 2023 18:17
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 734ea40

Tested grabbing the CI artifacts. You must be logged into GitHub in order to do this, we should leave a note on this in the README here or in a follow-up

diff --git a/src/qml/README.md b/src/qml/README.md
index 9b51e1b8a..0d24f3e5e 100644
--- a/src/qml/README.md
+++ b/src/qml/README.md
@@ -4,7 +4,7 @@
 
 This directory contains the source code for an experimental Bitcoin Core graphical user interface (GUI) built using the [Qt Quick](https://doc.qt.io/qt-5/qtquick-index.html) framework.
 
-Insecure CI artifacts are available for local testing of the master branch, avoiding the need to build:
+Unsecure CI artifacts are available for local testing of the master branch, avoiding the need to build (must be logged into github):
 - for Windows: [`insecure_win_gui.zip`](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/win64/insecure_win_gui.zip)
 - for Intel macOS: [`insecure_mac_gui.zip`](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/insecure_mac_gui.zip)
 - for Apple Silicon macOS: [`insecure_mac_arm64_gui.zip`](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos_arm64/insecure_mac_arm64_gui.zip)

Additional updates to the README would be:

  • removing the link for Android
  • Updating links for master CI artifacts

@hebasto
Copy link
Member Author

hebasto commented Sep 11, 2023

we should leave a note on this in the README here or in a follow-up

Leaving this for someone else :)

@hebasto hebasto merged commit ee4dc7d into bitcoin-core:main Sep 11, 2023
6 checks passed
@hebasto hebasto deleted the 230909-gha-cross branch September 15, 2023 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants