Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mkurz
Copy link
Contributor

@mkurz mkurz commented Dec 23, 2022

Fixes #2636, makes coursier run on Linux running on bar metal Apple silicon and also in Linux docker containers within macOS on M1/M2 machines.

Same like we did for scala-ci here: VirtusLab/scala-cli#1726

@mkurz
Copy link
Contributor Author

mkurz commented Dec 23, 2022

Can someone approve the workflows to run? Thanks!

@mkurz mkurz force-pushed the aarch64_linux_pagesize_64k branch from 411908f to 59e9c15 Compare December 23, 2022 16:43
@mkurz
Copy link
Contributor Author

mkurz commented Dec 23, 2022

@alexarchambault Can you approve once more? I force pushed because formatting was wrong.

Copy link
Member

@alexarchambault alexarchambault left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mkurz! I just hope this won't have unintended consequences (performance-wise, ...)

@mkurz
Copy link
Contributor Author

mkurz commented Dec 27, 2022

@alexarchambault Besides more memory usage, which can be neglected IMHO, there should hopefully not be a performance penalty. Also this changes really only effects the Linux aarch64 binaries, so nothing changes for the "average" Linux, macOS or Windows users 😉

Would really love to see this in RC5, thanks!

@mkurz
Copy link
Contributor Author

mkurz commented Jan 3, 2023

Just to let you know, sbt 1.8.1 just got released, which contains the same page size setting like in this pull request and like scala-cli has now as well: sbt/sbt#7108

@mkurz
Copy link
Contributor Author

mkurz commented Jan 11, 2023

Any reason not to merge this?

@tgodzik
Copy link
Collaborator

tgodzik commented Jan 12, 2023

@alexarchambault should we merge?

@alexarchambault alexarchambault merged commit 3464529 into coursier:main Jan 13, 2023
@mkurz mkurz deleted the aarch64_linux_pagesize_64k branch January 13, 2023 09:30
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.

Error when running coursier on Linux aarch64

3 participants