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 on ubuntu-24.04-arm #86

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

reebalazs
Copy link

No description provided.

@reebalazs
Copy link
Author

reebalazs commented Jan 19, 2025

Arm64 linux builders are supported since Jan. 16, and here is a CI setup that utilizes it to build a native arm64 ppx (that does not need Rosetta 2).

Note, I had to change the builder image from alpine to ubuntu, because the action that installs NodeJs is currently broken on arm64. Once that's fixed, it can be switched back to alpine if desired.

@mununki
Copy link
Member

mununki commented Jan 20, 2025

Thanks for the PR!
I have one question: if we want to support apple silicon, can't we just use macos-latest (or macos-14) from github actions instead of linux-arm?

@reebalazs
Copy link
Author

reebalazs commented Jan 20, 2025

Thanks for the PR! I have one question: if we want to support apple silicon, can't we just use macos-latest (or macos-14) from github actions instead of linux-arm?

Yes, it could be done (and that was possible even before the linux-arm runners became available), but, it would be a more complicated setup.

The main problem is, that macos-latest is darwin, and because of some copyright issue with Docker, they cannot install docker in the runner by default. It's not trivial but then it can then be installed (from Homebrew). This is already a challenge (especially to figure it out), however the bigger problem is that this way we cannot use container and none of the Github actions will work. We have to run docker ourselves and the build has to be moved into a Dockerfile. So the configuration of this build will be uglier, and different from all the others.

So, I could do it, but I'm much happier with the current action configuration, which is only available just recently, since Github has made the linux-arm runners available.

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