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

AdoptOpenJDK 17 Support #80

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

Redstoneguy129
Copy link

This adds support for building JDK 17 for the EV3.
A few issues must be known:

  • The builds themself fail.
  • The issue i believe has to do with sources that have been moved or discontinued.
  • The jdk source i have put links to an old Adoptium CI job that worked in the past. All newer jobs fail relating to my 2nd issue.
  • The .patch file contribution i believe i have not worked on it well as there isn't a way to test it.

Currently to get the JDK 17 onto the EV3 i run this via ssh.

wget https://ci.adoptium.net/job/eljbuild/job/${DEBIAN_DISTRO}-bleeding/84/artifact/build/jdk-ev3.tar.gz

Unpackage it, then set java path.

This will give you a working JDK 17-ev3-unreleased.
It would be great it we can get this build working again allowing us to support even newer versions of Java.

@Redstoneguy129
Copy link
Author

@Redstoneguy129
Copy link
Author

Last successful build being 2 years, 9 months ago.
https://ci.adoptium.net/job/eljbuild/

@Redstoneguy129 Redstoneguy129 marked this pull request as draft March 11, 2024 19:50
@Redstoneguy129
Copy link
Author

Turned into Draft as this needs contributions made before it's production ready

@Redstoneguy129
Copy link
Author

If you wish to contact me for easier communication my email is displayed on my github, if you would prefer my Discord the username is same as my Github username.

@jabrena
Copy link
Member

jabrena commented Mar 11, 2024

I will reach you tomorrow for sure!

@jabrena jabrena self-requested a review March 12, 2024 09:41
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.

2 participants