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

build(ci): Publish MacOS universal binaries to github releases #568

Merged
merged 12 commits into from
Oct 13, 2021

Conversation

bkotsopoulossc
Copy link
Contributor

@bkotsopoulossc bkotsopoulossc commented Oct 12, 2021

Followup to #565. This now builds universal binaries and uploads to github releases. This is done in one job in the github workflow, with builds for each architecture running serially. Currently it doesn't seem like there is support in the rust build system to build a single multi-arch MacOS binary in one swoop, so this approach is to build each arch separately, and use lipo to combine them.

PTAL @jan-auer @Swatinem

@bkotsopoulossc bkotsopoulossc changed the title [WIP] -- build(ci): Publish MacOS M1 ARM binaries to github releases [WIP] -- build(ci): Publish MacOS universal binaries to github releases Oct 12, 2021
@bkotsopoulossc bkotsopoulossc changed the title [WIP] -- build(ci): Publish MacOS universal binaries to github releases build(ci): Publish MacOS universal binaries to github releases Oct 12, 2021
@bkotsopoulossc bkotsopoulossc marked this pull request as ready for review October 12, 2021 19:45
@bkotsopoulossc bkotsopoulossc requested a review from a team October 12, 2021 19:45
Copy link
Member

@Swatinem Swatinem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot to mention this in the first review.
This also needs to update the filenames here:

symbolicator/.craft.yml

Lines 31 to 35 in e67a0c1

- /^gh-pages.zip$/
- /^symbolicator-Linux-x86_64$/
- /^symbolicator-Linux-x86_64-debug.zip$/
- /^symsorter-Linux-x86_64$/
- /^wasm-split-Linux-x86_64$/

That is our release automation that uploads those built files to github releases, etc.

@Swatinem Swatinem merged commit 302c879 into getsentry:master Oct 13, 2021
@Swatinem
Copy link
Member

Thanks!

@bkotsopoulossc bkotsopoulossc deleted the macos_universal_ci_release branch October 15, 2021 02:35
@bkotsopoulossc
Copy link
Contributor Author

@Swatinem any word on when the next release will be cut?

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