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

Support arm64 on darwin #211

Closed
domoritz opened this issue May 15, 2021 · 14 comments · Fixed by #215
Closed

Support arm64 on darwin #211

domoritz opened this issue May 15, 2021 · 14 comments · Fixed by #215

Comments

@domoritz
Copy link
Contributor

It would be great if you could support ARM64 (Apple M1) for the OSX bundles.

@ChadKillingsworth
Copy link
Collaborator

Will be quite a while before that is possible. Graal has to support building in that environment and we would need an automated build system that supports M1 macs.

@domoritz
Copy link
Contributor Author

Is that oracle/graal#2666?

@ChadKillingsworth
Copy link
Collaborator

That looks correct for the Graal support. But we would still need a build environment. GitHub Actions doesn't support M1 macs right now.

@domoritz
Copy link
Contributor Author

The relevant issue for that is actions/runner-images#2187, I believe.

@ChadKillingsworth
Copy link
Collaborator

We can probably add arm64 to the architecture though and let Rosetta handle the translation for now. Would you be able to test that the current image actually works? I don't have an M1 Mac to try this on.

@domoritz
Copy link
Contributor Author

domoritz commented May 18, 2021

Yes, I'd be happy to give it a try. Would you make a prerelease that I can install or a branch?

@ChadKillingsworth
Copy link
Collaborator

You can download the MacOS image binary from any of the CI build runs such as https://github.com/google/closure-compiler-npm/actions/runs/860306976

You just need to see if the binary will execute. So compiler --help should do it.

@domoritz
Copy link
Contributor Author

I downloaded the binary, marked it as executable, allowed it in the macOS security settings and ran ./compiler --help without problems.

$ arch
arm64
$ lipo -info compiler
Non-fat file: compiler is architecture: x86_64

So looks like it should just work.

domoritz added a commit to domoritz/closure-compiler-npm that referenced this issue May 28, 2021
@domoritz
Copy link
Contributor Author

domoritz commented Jun 3, 2021

@ChadKillingsworth anything else I can test before you make a release?

@ChadKillingsworth
Copy link
Collaborator

I created the pr.

@domoritz
Copy link
Contributor Author

domoritz commented Jun 3, 2021

Thank you. I made a similar pull request in #215.

@ChadKillingsworth
Copy link
Collaborator

Awe sorry - I missed that one.

@ChadKillingsworth
Copy link
Collaborator

You should be able to test this tomorrow with the nightly build

@domoritz
Copy link
Contributor Author

domoritz commented Jun 3, 2021

Thanks for merging. I totally forgot to mention it here. Looking forward to trying the new version.

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