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
Fix the Apple Silicon (macOS aarch64
/ arm64
) URLs for Julia nightly
#220
Fix the Apple Silicon (macOS aarch64
/ arm64
) URLs for Julia nightly
#220
Conversation
This is great, thank you @benlorenz! I'll take a look. @SaschaMann It would be great if you could take a look too. cc: @ViralBShah - once this is merged and a new release is made, we can use this in the SparseArrays.jl repo. |
aarch64
/ arm64
) URLs for Julia nightly
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!
@SaschaMann or @IanButterworth If you have time, can you do a post-merge review? |
Nice! Is using |
I don't think we need the added complexity of |
Testing with julia nightly on
macos-14
(aarch64) currently fails with:The filename is slightly wrong and should have
macaarch64
instead ofmac64
, this PR tries to fix this for linux and macOS.I also added
macOS-14
(for the new M1 runners) to the nightly tests.Note that these URLs are a bit of a guesswork. On the official nightly downloads page they have a different format using a slightly more homogeneous pattern (at least for linux and macos):
x86_64
,i686
andaarch64
in the baseurl instead ofx64
andx86
.macos
instead ofmac
in the baseurl.julia-latest-<osname>-<arch>
Adjusting to this pattern could simplify the code a bit even though it would still need a bunch of extra cases for windows.
But I don't really know anything about the stability of all those URLs.
Edit: Looks like the new tests on the M1 were successful: https://github.com/julia-actions/setup-julia/actions/runs/7855176535/job/21436628690?pr=220#step:5:15