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

Compiling for Java 21 #1168

Merged
merged 1 commit into from
Aug 15, 2023
Merged

Compiling for Java 21 #1168

merged 1 commit into from
Aug 15, 2023

Conversation

mkarg
Copy link
Contributor

@mkarg mkarg commented Aug 12, 2023

Fixes #1167

This PR enables compilation of JDK 21 in the release-4.0 branch as proposed in the discussion of #1167.

As these are no API changes, this is a fast-track review period of just one day as per our committer rules.

@mkarg mkarg added the project label Aug 12, 2023
@mkarg mkarg added this to the 4.0 milestone Aug 12, 2023
@mkarg mkarg requested review from chkal and spericas August 12, 2023 20:15
@mkarg mkarg self-assigned this Aug 12, 2023
@mkarg mkarg marked this pull request as ready for review August 12, 2023 20:36
@mkarg mkarg force-pushed the jdk-1167 branch 2 times, most recently from 047aed0 to dafe7e9 Compare August 12, 2023 22:15
@arjantijms
Copy link
Contributor

Instead of source/target, the release attribute can be used as well.

@jamezp
Copy link
Contributor

jamezp commented Aug 14, 2023

Do we want to do this before Java 21 is released?

@mkarg
Copy link
Contributor Author

mkarg commented Aug 15, 2023

Do we want to do this before Java 21 is released?

This happens solely in the release-4.0 branch, where compiling against 21 is mandatory anyways according to the Jakarta EE 11 release plan. JDK 21 published RC 1 already; according to the JDK rules, the sole difference between now and later is more bug fixes. So why should we wait? What would break that could not get fixed within a single business day (and possibly doesn't care as this is not the master branch)?

@mkarg mkarg merged commit 6d803e5 into jakartaee:release-4.0 Aug 15, 2023
@mkarg mkarg deleted the jdk-1167 branch August 15, 2023 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants