Skip to content

Conversation

@ahojnnes
Copy link
Contributor

No description provided.

@ahojnnes ahojnnes requested review from B1ueber2y and sarlinpe August 10, 2024 17:08
config: [
{
os: macos-14,
arch: arm64,
Copy link
Member

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.

Copy link
Contributor Author

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.

Copy link
Member

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?

@sarlinpe sarlinpe marked this pull request as draft November 27, 2024 14:22
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.

3 participants