-
Notifications
You must be signed in to change notification settings - Fork 957
[1.8.x] Prepare aarch64 support for sbtn binary #7108
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
Conversation
|
Hi @mkurz, Thank you for your contribution! We really value the time you've taken to put this together. We see that you have signed the Lightbend Contributors License Agreement before, however, the CLA has changed since you last signed it. |
| ), | ||
| ) ++ (if (isLinux && isArmArchitecture) | ||
| Seq("-H:PageSize=65536") // Make sure binary runs on kernels with page size set to 4k, 16 and 64k | ||
| else Nil), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to set the page size to 64k for aarch64 binaries to make sure it runs on all kind of kernels.
I did lots of research, collected here: coursier/coursier#2636 (comment)
scala-ci already merged the same fix: VirtusLab/scala-cli#1726
coursier should follow soon: coursier/coursier#2641
|
@eed3si9n If you could take a look, that would be nice. More pull requests are coming afterwards. (Also, I just signed the CLA). Thanks! |
eed3si9n
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
If possible, could you target the default branch (currently 1.9.x branch), and backport next time?
Sure 👍 |
In https://github.com/mkurz/sbtn-dist/commits/aarch64-support I am setting up a GitHub actions job to build a
sbtnbinary for Linux aarch64. To make this happen, the changes in this PR are necessary. I am almost done, binary can be downloaded here already: https://github.com/mkurz/sbtn-dist/actions/runs/3825258120