RFR: 8304691: Remove jlink --post-process-path option [v2]

Mandy Chung mchung at openjdk.org
Wed Mar 22 20:02:45 UTC 2023


On Wed, 22 Mar 2023 16:25:51 GMT, Ian Graves <igraves at openjdk.org> wrote:

>> Removing the hidden `jlink --post-process-path` option.
>
> Ian Graves has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Updates per feedback

`test/jdk/tools/lib/tests/JImageGenerator` references this option and needs update.

src/jdk.jlink/share/classes/jdk/tools/jlink/internal/JlinkTask.java line 392:

> 390:         ExecutableImage img = DefaultImageBuilder.getExecutableImage(existingImage);
> 391:         if (img == null) {
> 392:             throw taskHelper.newBadArgs("err.existing.image.invalid");

"err.existing.image.invalid" can be removed from `jlink.properties.`

src/jdk.jlink/share/classes/jdk/tools/jlink/internal/TaskHelper.java line 265:

> 263:                 Path path = Paths.get(arg);
> 264:                 if (!Files.exists(path) || !Files.isDirectory(path)) {
> 265:                     throw newBadArgs("err.image.must.exist", path);

"err.image.must.exist" can be removed from `jlink.properties`

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

PR Review: https://git.openjdk.org/jdk/pull/13126#pullrequestreview-1353383619
PR Review Comment: https://git.openjdk.org/jdk/pull/13126#discussion_r1145341533
PR Review Comment: https://git.openjdk.org/jdk/pull/13126#discussion_r1145339835


More information about the core-libs-dev mailing list