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

[GR-30433] Make native-image driver buildable via --module-path. #3382

Merged
merged 24 commits into from
Jun 2, 2021

Conversation

olpaw
Copy link
Member

@olpaw olpaw commented Apr 28, 2021

The changes on this PR further refine native-image module support to a point where we can now build the native-image driver into and image if svm-driver.jar is specified on the --module-path. I.e. The following works now:

cd graal/substratevm
export USE_NATIVE_IMAGE_JAVA_PLATFORM_MODULE_SYSTEM=true
mx build
mx native-image -g -ea --module-path svmbuild/vm/lib/graalvm/svm-driver.jar --module org.graalvm.nativeimage.driver/com.oracle.svm.driver.NativeImage -H:Path=svmbuild/vm/lib/svm/bin -H:Name=native-image

and results in a working driver image. Which can easily tested with e.g.

svmbuild/vm/lib/svm/bin/native-image -g -ea --module-path svmbuild/vm/lib/graalvm/svm-driver.jar --module org.graalvm.nativeimage.driver/com.oracle.svm.driver.NativeImage -H:Path=svmbuild/vm/lib/svm/bin -H:Name=native-image

@olpaw olpaw changed the title [GR-30433] Make native-image driver buildable with --module-path. [GR-30433] Make native-image driver buildable via --module-path. Apr 28, 2021
@olpaw olpaw force-pushed the paw/GR-30433-M2 branch 3 times, most recently from 453abf9 to 867bfd1 Compare May 3, 2021 15:05
@olpaw
Copy link
Member Author

olpaw commented May 5, 2021

Heads up: The remains of the previously to experimental demoted native-image build server mode will be removed soon.
With all the changes for module-support it is impossible to keep the server specific sources in a consistent state. See:

8130b75

olpaw added 21 commits May 26, 2021 09:53
and enable it for the native-image-launcher
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants