RFR: 8293667: Align jlink's --compress option with jmod's --compress option [v12]
Mandy Chung
mchung at openjdk.org
Thu Feb 23 18:31:15 UTC 2023
On Thu, 23 Feb 2023 17:31:02 GMT, Ian Graves <igraves at openjdk.org> wrote:
>> This is an approach to adding a flag to jlink that will allow --compress to take the same types of arguments as jmod, thus bringing the two into alignment. This likely requires a CSR and a discussion on whether we should deprecate or simply remove the original numeric compression arguments.
>
> Ian Graves has updated the pull request incrementally with one additional commit since the last revision:
>
> Tweaking properties
Looks good in general. Minor comments.
src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/DefaultCompressPlugin.java line 96:
> 94: switch (level) {
> 95: case LEVEL_0:
> 96: System.err.println(getMessage("warn.prefix") + " " +
Suggestion:
System.err.println(getMessage("compress.warn.argumentdeprecated", level));
It can be simpler to out the warning prefix in the message. Same for line 102 and 107.
src/jdk.jlink/share/classes/jdk/tools/jlink/resources/plugins.properties line 74:
> 72:
> 73: compress.warn.argumentdeprecated=\
> 74: The {0} argument for --compress has been deprecated and will be removed from a future release
Suggestion:
Warning: The "{0}" argument for --compress is deprecated and may be removed from a future release.
-------------
Marked as reviewed by mchung (Reviewer).
PR: https://git.openjdk.org/jdk/pull/11617
More information about the core-libs-dev
mailing list