On Tue, 13 Oct 2026 11:26:47 GMT, Alan Bateman <[email protected]> wrote:
>> Sean Mullan has updated the pull request incrementally with eight additional >> commits since the last revision: >> >> - Remove whitespace, fix continuation char. >> - Test property with EOF immediately after continuation character >> - Disallow include statement in java.security file. >> - Check for duplicates in java.security file. >> - Remove redundant prop. >> - Remove extra back-tick. >> - Test more properties with different escape characters. >> - Store properties as specified by Properties.store(OutputStream, String). > > src/jdk.jlink/share/classes/jdk/tools/jlink/resources/plugins.properties line > 181: > >> 179: properties that may have been added. The include filename does not need >> to\n\ >> 180: exist at jlink time. If the properties file contains duplicate >> properties,\n\ >> 181: the value of the last property is used. > > The plugins listed by `jlink --list-plugins` are mostly a short description > of the plugin. I think we should trim this down, the plugin's usage message > has the verbose description. Actually, `jlink --list-plugins` prints `security-properties.usage`. The `security-properties.description` is only printed if you don't overload `getUsage()` in the plugin (which the sec props plugin does because it extends `AbstractPlugin`). There doesn't seem to be a `jlink` option to show one or the other. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31884#discussion_r4260514490
