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

Cannot access Adoptium on Mac M2 #135

Closed
buchholzmd opened this issue Jul 16, 2023 · 3 comments
Closed

Cannot access Adoptium on Mac M2 #135

buchholzmd opened this issue Jul 16, 2023 · 3 comments

Comments

@buchholzmd
Copy link

Hello, I am trying to install jdk on my new mac and recieve a 404 error when trying to run jdk.install('17')).

I can run jdk.install('17', vendor='Azul')) as well as access that link directly in web browser.

However, for Adoptium, when running jdk.get_download_url('17') I get the following link for mac arm, which cannot be accessed from a web browser:
https://api.adoptium.net/v3/binary/latest/17/ga/mac/arm/jdk/hotspot/normal/eclipse

However, I changed to this architecture and this url works:
https://api.adoptium.net/v3/binary/latest/17/ga/mac/aarch64/jdk/hotspot/normal/eclipse

Any help for quickest way to setup on Mac M2 would be great and thanks in advance

@buchholzmd
Copy link
Author

Following up on this, I looked at the different Adoptium releases, and there is aarch64 and arm available for the linux releases, and only aarch64 for the MacOS (arm based, of course) releases. This makes sense to me, but as a side note am confused in the difference between linux 'aarch64' and 'arm' as I thought they were the same architectures.

Still, it seems with the M2, install-jdk is generating the url with arm as the architecture, and as such the url won't exist without one explicitly changing the url or including the arch parameter.

@jsnow-taconic
Copy link

As a temporary workaround jdk.install() has a named parameter called arch. You can pass in the architecture of your choice with arch=Architecure.AARCH64. I will need to update the Adoptium architecture detection to account for this.

@jyksnw jyksnw mentioned this issue Jul 21, 2023
@jyksnw jyksnw closed this as completed in 527ba6a Jul 21, 2023
jyksnw added a commit that referenced this issue Jul 21, 2023
@jyksnw
Copy link
Owner

jyksnw commented Jul 21, 2023

@buchholzmd this has been fixed in version 1.1.0

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

No branches or pull requests

3 participants