-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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: Replace macos-latest label with macos-13 #6729
Conversation
c464b7a
to
54295e9
Compare
Due to a change to the GitHub runner labels a few days ago (see: actions/runner#3256) our build workflows for macOS x64 didn't work anymore. According to https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories the macos-13 label is not using arm64 yet. Until a better solution is offered in the linked issue above, we'll keep using the macos-13 label which hopefully doesn't switch to arm64 soon.
54295e9
to
c324fa9
Compare
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 guess this is fine as a stop-gap, really odd that the default architecture just changed out of the blue.
I think so too! You'd think they'd at least announce it a few weeks before they roll it out. |
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.
lgtm, thanks.
Summary: 'macos-latest picks now automatically a mac14-arm' We are fixing here to use macos-13 that continue to use intel based CPU and will provide a way for us to keep a `x86_64` processor to compile our `x86_64` python wheels. We are using similar fix as other communities have done https://github.com/Ryujinx/Ryujinx/pull/6729/files See here for the discussion that happened on exactly the same issue to another GitHub project Ryujinx/Ryujinx#6729 Reviewed By: vijay609 Differential Revision: D56893802 fbshipit-source-id: 4f3163064f5c250dc7b0a86ecaf525cc93e06871
Due to a change to the GitHub runner labels a few days ago (see: actions/runner#3256) our build workflows for macOS x64 didn't work anymore. According to https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories the macos-13 label is not using arm64 yet.
Until a better solution is offered in the linked issue above, we'll keep using the macos-13 label which hopefully doesn't switch to arm64 soon.