On Tue, 27 Sep 2022 22:37:16 GMT, Doug Simon <dnsi...@openjdk.org> wrote:

>> This PR adds a new jlink plugin (`--save-jlink-argfiles=<filenames>`) to 
>> support persisting jlink options.
>> 
>> 
>>> echo "--add-modules jdk.internal.vm.ci --add-options=-Dfoo=xyzzy 
>>> --vendor-version="XyzzyVM 3.14.15" 
>>> --vendor-bug-url=https://bugs.xyzzy.com/"; > my_image.argfile
>>> export ALL_MODULES=$(java --list-modules | sed 's:@.*::g' | tr '\n' ',')
>>> jlink --keep-packaged-modules=my_image/jmods --add-modules $ALL_MODULES 
>>> --output=my_image --save-jlink-argfiles my_image.argfile
>>> my_image/bin/java -XshowSettings:properties --version 2>&1 | grep yzzy
>>> my_image/bin/jlink --add-modules=java.base --output=my_image2
>>> my_image2/bin/java -XshowSettings:properties --version 2>&1 | grep yzzy
>>     foo = xyzzy
>>     java.vendor.url.bug = https://bugs.xyzzy.com/
>>     java.vendor.version = XyzzyVM 3.14.15
>> OpenJDK Runtime Environment XyzzyVM 3.14.15 (build 
>> 20-internal-2022-09-22-0951036.dnsimon...)
>> OpenJDK 64-Bit Server VM XyzzyVM 3.14.15 (build 
>> 20-internal-2022-09-22-0951036.dnsimon..., mixed mode)
>>> my_image2/bin/java -d jdk.internal.vm.ci | head -1
>> jdk.internal.vm.ci@20-internal
>
> Doug Simon has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   SaveJlinkArgfilesPlugin should verify that jdk.jlink is in the output image

Looks good.

-------------

Marked as reviewed by mchung (Reviewer).

PR: https://git.openjdk.org/jdk/pull/10445

Reply via email to