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

Use M1 runner to build arm64 macOS builds #6868

Merged
merged 6 commits into from
May 8, 2024
Merged

Conversation

SiarheiFedartsou
Copy link
Member

@SiarheiFedartsou SiarheiFedartsou commented May 7, 2024

Issue

GitHub now provides M1 based runner, so we can use it actions/runner-images#9254

Besides that I had to update Boost to version 1.85.0, because otherwise we had compilation errors on this new macOS.

Tasklist

Requirements / Relations

Link any requirements here. Other pull requests this PR is based on?

@SiarheiFedartsou SiarheiFedartsou changed the title Try to use M1 runner to build arm64 macOS builds Use M1 runner to build arm64 macOS builds May 7, 2024
@DennisOSRM
Copy link
Collaborator

Nice, looks like this is much cleaner than #6864.

@SiarheiFedartsou SiarheiFedartsou marked this pull request as ready for review May 7, 2024 20:11
@SiarheiFedartsou
Copy link
Member Author

Nice, looks like this is much cleaner than #6864.

Ah sorry, I haven’t noticed your PR when started working on this…

@@ -331,7 +331,7 @@ jobs:
build_node_package: true
continue-on-error: true
node: 20
runs-on: macos-11
runs-on: macos-13 # x86_64
Copy link
Member Author

Choose a reason for hiding this comment

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

Looking at #6864 it seems we can use macos-14-large here, right @DennisOSRM ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, that should probably work.

@DennisOSRM
Copy link
Collaborator

Nice, looks like this is much cleaner than #6864.

Ah sorry, I haven’t noticed your PR when started working on this…

No worries. Great minds think alike. 😉

Copy link
Collaborator

@DennisOSRM DennisOSRM left a comment

Choose a reason for hiding this comment

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

👋🏼

@SiarheiFedartsou SiarheiFedartsou merged commit 3254686 into master May 8, 2024
23 checks passed
@SiarheiFedartsou SiarheiFedartsou deleted the sf-macos-arm64 branch May 8, 2024 19:08
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