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

Build with Java 21 on Jenkins #767

Merged

Conversation

jonahgraham
Copy link
Member

Thanks to others work earlier the GitHub actions were switched to Java 21 builds. This change applies the same for JIPP (Jenkins) builds.

This is the state of the files as of cdt-infra
commit 497e7b2a643ff6ea12a56a21c17dd2d170e918c8 with
the Readme updated for the new locations
- Fix links for cdt-infra
- The cdt-infra git repo is now archived, so this commit updates
the links to the current values.
- Add new Docker related files to the gitattributes as text files
- Update DEFAULT_BUFFER_SIZE for Java 21's value when building JNI,
this also means the dll/exe need a rebuild
- add a toolchains.xml since we use Java 21 toolchain, but most bundles
are Java 17 still
@jonahgraham
Copy link
Member Author

This is running on Jenkins here: https://ci.eclipse.org/cdt/job/cdt-jonahgraham-fork/ to make sure that it builds as expected before merging. (Note, please merge without rebasing as one of the commits refers to another by SHA which would change in case of a rebase)

@jonahgraham jonahgraham merged commit f0ae450 into eclipse-cdt:main Apr 27, 2024
4 checks passed
@jonahgraham jonahgraham deleted the build-with-java21-on-jenkins branch April 27, 2024 10:18
@ruspl-afed
Copy link
Member

Looks very impressive, you have an unique set of skills @jonahgraham !

jonahgraham added a commit to jonahgraham/eclipse-cdt-lsp that referenced this pull request Apr 30, 2024
jonahgraham added a commit to jonahgraham/eclipse-cdt-lsp that referenced this pull request Apr 30, 2024
This synchronizes cdt-lsp to eclipse-cdt/cdt#767
and makes the JIPP behave the same as GitHub actions that was
changed in eclipse-cdt#304
jonahgraham added a commit to jonahgraham/cdt-infra that referenced this pull request Apr 30, 2024
The migration is complete now and everything that is still current
is in the main CDT repo, with the last major migration being
the Dockerfiles in eclipse-cdt/cdt#767
jonahgraham added a commit to eclipse-cdt/cdt-lsp that referenced this pull request Apr 30, 2024
This synchronizes cdt-lsp to eclipse-cdt/cdt#767
and makes the JIPP behave the same as GitHub actions that was
changed in #304
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