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

Update nb-javac to jdk-22.0.1 #7366

Merged
merged 1 commit into from May 13, 2024
Merged

Conversation

mbien
Copy link
Member

@mbien mbien commented May 7, 2024

upgrades nb-javac from jdk-22+33 to jdk-22.0.1+8

currently uses a staged build

fixes #6826 due to https://bugs.openjdk.org/browse/JDK-8322159

@mbien mbien added do not merge Don't merge this PR, it is not ready or just demonstration purposes. Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) Upgrade Library Library (Dependency) Upgrade ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels May 7, 2024
@mbien mbien added this to the NB22 milestone May 7, 2024
@mbien mbien linked an issue May 7, 2024 that may be closed by this pull request
@mbien mbien requested review from jtulach, dbalek and lahodaj May 7, 2024 20:56
@mbien mbien force-pushed the nb-javac-2201 branch 2 times, most recently from 9ce1e9c to b8bed75 Compare May 7, 2024 21:36
Comment on lines 21 to 22
release.external/nb-javac-jdk-22+33-api.jar=modules/ext/nb-javac-jdk-22-33-api.jar
release.external/nb-javac-jdk-22+33.jar=modules/ext/nb-javac-jdk-22-33.jar
release.external/nb-javac-jdk-22.0.1+8-api.jar=modules/ext/nb-javac-jdk-22.0.1_8-api.jar
release.external/nb-javac-jdk-22.0.1+8.jar=modules/ext/nb-javac-jdk-22.0.1_8.jar
Copy link
Member Author

@mbien mbien May 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note for future me: JarClassLoader doesn't like the + likely due to url path encoding.

@mbien mbien force-pushed the nb-javac-2201 branch 3 times, most recently from d81e535 to 278f222 Compare May 8, 2024 00:50
@mbien
Copy link
Member Author

mbien commented May 8, 2024

used the dev-build from this PR and the test project @matthiasblaesing provided from #6826 (comment) and everything was working fine (opening the project in NB 21 would cause a CCE).

@mbien
Copy link
Member Author

mbien commented May 8, 2024

switched from staging to maven central

@mbien mbien removed the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label May 8, 2024
@mbien mbien marked this pull request as ready for review May 8, 2024 15:08
Copy link
Contributor

@lahodaj lahodaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me.

Copy link
Contributor

@dbalek dbalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine.

@ebarboni
Copy link
Contributor

let's merge this

@ebarboni ebarboni merged commit 9cff074 into apache:delivery May 13, 2024
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) Upgrade Library Library (Dependency) Upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

-Xlint:all breaks nb-javac 21
5 participants