-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Export mac binaries in CI #2700
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
base: main
Are you sure you want to change the base?
Conversation
| config: [ | ||
| { | ||
| os: macos-14, | ||
| arch: arm64, |
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.
Very cool! This creates a binary for arm64 only, would it make sense to also create a binary for intel machines? We can have another config with a macos-13 runner.
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.
One problem I have is that the binary doesn't seem to work under arm64. The changes in this PR work under x86 on my 10 year old macbook pro. If you have some minutes to investigate what's going wrong under arm64, that would be great. I don't have access to an arm64 machine.
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.
I also don't have an arm64 machine at hand for now. Could we instead test the binary in a fresh arm64 runner in the CI?
No description provided.