Skip to content

Adding JDK 25 build compatibility#174

Open
svanteschubert wants to merge 2 commits intoitplr-kosit:mainfrom
svanteschubert:jdk-25-compatibility
Open

Adding JDK 25 build compatibility#174
svanteschubert wants to merge 2 commits intoitplr-kosit:mainfrom
svanteschubert:jdk-25-compatibility

Conversation

@svanteschubert
Copy link

Allowing to build with JDK 25 and keeping the ability to build with 11+
In addition, redirecting a test output file to ./target (sorry, no time for own PR)

Only strange is the signature error of a test package:
validator/libs/de/kosit/validationtool/packaged-test-scenarios/1.0.2/packaged-test-scenarios-1.0.2.pom
org.eclipse.aether.transfer.ChecksumFailureException: Checksum validation failed, no checksums available

But I have not built the repo for years, so this error might existed earlier.

@phax
Copy link
Collaborator

phax commented Feb 17, 2026

Moin Svante, damit verlieren wir aber leider Java 11-Backwards-Kompatibilität, die uns derzeit noch sehr wichtig ist. Daher wird der PR wohl leider erstmal länger liegen...

@svanteschubert
Copy link
Author

Moin Philip,

Habe den PR mit JDK 11 auf Windows und MacOS getestet. Build successful, könntest Du Dir das bitte einmal ansehen, denn da sind Exceptions, aber dann sollte auch der Build failen, wenn die nicht erwartet sind, oder?

LG aus Berlin

@phax
Copy link
Collaborator

phax commented Feb 17, 2026

Sorry, ich nochmal. Irgendwas ist mit deiner Quelle nicht astrein. Du hast z.B. <version.jacoco>0.8.5</version.jacoco> drin, während das pom.xml hier vom main branch schon <version.jacoco>0.8.13</version.jacoco> hat etc.

Außerdem haben wir in unserem Gitlab Repo die Pipelines gegen Java 11, 17, 21 und 25 erfolgreich laufen.

Also schau bitte nochmal, ob du nicht von einem veralteten Source ausgehst

@svanteschubert
Copy link
Author

svanteschubert commented Feb 17, 2026

Ich hatte auf einen alten Fork aufgesetzt (glaube, der Rebase hatte beim 2. Versuch geklappt), sollte jetzt gefixt sein - sorry for the noise!

@svanteschubert svanteschubert marked this pull request as draft February 17, 2026 19:41
@svanteschubert svanteschubert marked this pull request as ready for review February 17, 2026 20:22
@phax
Copy link
Collaborator

phax commented Feb 17, 2026

Danke, jetzt schauts schon besser aus. Das mit Jacoco und Java 25 schau ich mir an

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