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

Allow building on Linux ARM64 #777

Merged
merged 63 commits into from
Feb 10, 2023
Merged

Allow building on Linux ARM64 #777

merged 63 commits into from
Feb 10, 2023

Conversation

tfausak
Copy link
Contributor

@tfausak tfausak commented Sep 28, 2022

This would address #714 and #765. It's similar to #769.

These changes allow elm-format to be built on an ARM64 (AArch64) Linux machine. I have tested them with AMD64 (x86-64) as well, which continues to work as expected.

@ismaelga
Copy link

Bumped into the same issue. This PR seems legit.
Thanks! Hope it gets merged

@avh4
Copy link
Owner

avh4 commented Feb 10, 2023

Huh, github seems confused about the list of commits after I fixed the merge conflicts here. 🤷

Merging.

@avh4 avh4 merged commit 5463c19 into avh4:main Feb 10, 2023
@avh4
Copy link
Owner

avh4 commented Feb 10, 2023

Thanks! This is on main branch now. I haven't set up linux aarch64 CI yet, so lmk if there's still any problems.

@jsynacek
Copy link

Awesome, thanks a lot! Would you mind releasing a new version so that the builds show in the release and also in the npm archives?

@avh4
Copy link
Owner

avh4 commented Feb 14, 2023

The release is tracked here: https://github.com/avh4/elm-format/milestone/31

Basically I have to do a test run of the new npm publishing process, then get linux-aarch64 binaries to build on CI, then take a look at building musl binaries, and then do the release. 🤞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants