Skip to content

Conversation

@cortlepp
Copy link
Member

@cortlepp cortlepp commented Sep 24, 2025

Add JDK 25 to the list of supported and tested Java versions. To achieve this, I

  • added JDK 25 to our CI matrix (and refactored our configuration for easier extensibility).
  • shielded SecurityManager usages by a runtime check, so that we only use it on supported JVMs (the current solution works, but the JDK documentation discourages just delegating to the deprecated methods link).
  • removed some build warnings by no longer including the ant runtime in several tasks.
  • removed the hermetic-maven-plugin, as it depends on the SecurityManager. I don't think it was ever required, but maybe @veithen can give some background on this and whether additional steps have to be taken.
  • removed the animal-sniffer-maven-plugin as it is no longer used anywhere.
  • updated some plugins to make them compatible with JDK 25

@cortlepp cortlepp force-pushed the ci/add-jdk-25-lts branch 2 times, most recently from dff0929 to 183f315 Compare September 24, 2025 06:42
@cortlepp cortlepp marked this pull request as draft September 24, 2025 06:45
@cortlepp cortlepp force-pushed the ci/add-jdk-25-lts branch 2 times, most recently from c2fa950 to e79e310 Compare November 17, 2025 14:43
@cortlepp cortlepp force-pushed the ci/add-jdk-25-lts branch 2 times, most recently from 5dbdb34 to ab23e4a Compare November 17, 2025 14:55
@cortlepp cortlepp marked this pull request as ready for review November 17, 2025 14:55
@cortlepp
Copy link
Member Author

@robertlazarski do you still want to look over this or can I just merge it?

@robertlazarski robertlazarski merged commit 4e9de1f into master Nov 25, 2025
5 checks passed
@robertlazarski
Copy link
Contributor

@cortlepp this looks great and thanks for reminding me, I was traveling the week you posted this and meant to revisit it but never did.

If there are any other outstanding questions you have and/or reviews that have been neglected, feel free to reach out like you did today.

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