Skip to content

Conversation

@kkriske
Copy link
Contributor

@kkriske kkriske commented Nov 5, 2025

  • update required native-image arguments
  • bump ci to GraalVM 25 to run against LTS

Changes seems to mostly be related to this I think: graalvm/native-build-tools#693
I've added some comments inline on the PR to clarify.
Interestingly, everything seemed to run fine without any changes on JDK 25 (on Windows at least)

<fallback>false</fallback>
<verbose>true</verbose>
<buildArgs>
<arg>-H:+AllowDeprecatedBuilderClassesOnImageClasspath</arg>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is only necessary for GraalVM JDK 17. On later versions this option is deprecated so at some point this might cause failures if the option is removed. At that point I would recommend just dropping builds for GraalVM JDK 17.

<arg>--initialize-at-build-time=org.junit.jupiter.api.parallel.ResourceLock</arg>
<arg>--initialize-at-build-time=org.junit.jupiter.api.parallel.ResourceLockTarget</arg>
<arg>--initialize-at-build-time=org.junit.jupiter.api.parallel.ResourceAccessMode</arg>
<arg>--initialize-at-build-time=org.junit.platform.commons.support.scanning.DefaultClasspathScanner</arg>
Copy link
Contributor Author

@kkriske kkriske Nov 5, 2025

Choose a reason for hiding this comment

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

Most of these initialization options seem to no longer be required on JDK 25. They're only here for 17 and 21.

@gotson
Copy link
Collaborator

gotson commented Nov 6, 2025

Thanks a lot. Do you think you could put the comments you made on github inside the pom.xml for future reference ?

…o 0.11.2

- update required native-image arguments
- bump ci to GraalVM 25 to run against LTS
@kkriske kkriske force-pushed the native-maven-plugin-0.11.2 branch from 6a2ee6d to 87fbac4 Compare November 6, 2025 19:01
@gotson gotson merged commit 4f126e5 into xerial:master Nov 7, 2025
31 checks passed
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