-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add arm64 macOS (Apple Silicon) build #96
Comments
open for contributions here, unless we get active complaints from schools I don't think I can make time for this anytime soon |
Apple Silicon GitHub runners are in public beta now. So this should be much easier. https://github.blog/2023-10-02-introducing-the-new-apple-silicon-powered-m1-macos-larger-runner-for-github-actions/ |
Yes, unfortunately they aren't free though... |
That shouldn't be a problem. @xeruf can you add a macOS runner for the build? |
xlarge or large? |
We need to cofngireu payment for that:
|
That should be fixed. I guess "large"is sufficient. |
It was implemented in d4f26f4 but I realized the releases never contained an arm image? Should check this since we pay for it. |
Warum ist der aktuelle Stand nicht akzeptabel?
The current macOS binaries produced by GitHub Releases don't run on ARM Macs as they bundle native x86_64 JavaFX libraries. Strangely running
java
under Rosetta doesn't work either, but since Java is largely cross-platform we should probably avoid emulation entirely.Was wäre dein Lösungsansatz?
Provide a build for the
macos-aarch64
(aka.osx-aarch64
) platform as described in the upstream Gradle-JFX plugin.While having GitHub-provisioned ARM runners would make this easy (since we could just add a new OS to the matrix), these aren't available yet. Still, making cross-builds should be doable since the JVM code is cross-platform anyway, so we would only have to ensure that Gradle bundles the native libraries for the specified target architecture. Still, that might require some fiddling, as evidenced by this thread:
The text was updated successfully, but these errors were encountered: