-
Notifications
You must be signed in to change notification settings - Fork 84
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
add java-11-openjdk-jmods (#181) #198
add java-11-openjdk-jmods (#181) #198
Conversation
@gunnarmorling do you think this should be part of the base image? |
oh hang on while this works for Java 11 it produces this for the Java 8 image:
Unfortunately that message is tucked away in the build and hard to see - it ideally really should fail for unknown packages IMHO. |
|
I guess this can be ignored:
|
@gunnarmorling I like to do TDD even for something like this - can you propose a test command to run to verify if whatever |
@vorburger Yes, I think the jmods package should be part of the image for JDK 11 (but not 8, where it doesn't exist, as you found out). In order to test it, you could run the jlink tool. This creates a small variant of the JDK only containing selected modules (well, it will be really small if that bug has been resolved ;)). The jmods package contains the module files of the JDK which will be used by jlink for creating such "modular runtime images". The following shows an example. It produces a runtime image with simple main class and a launcher script:
Running this image via the created launcher script will print "Hello, jlink", if everything works as it should (note the image is fully self-contained at this point, i.e. it also could run on another system without a Java installation):
|
but do not install it on Java 8
3e95a00
to
a08555a
Compare
@gunnarmorling I've just merged this, so you now have Thanks a lot for the full example - to be picked up in # 181! |
Reverted this again, for now; see #181. |
No description provided.