RFR: 8236127: Use value of --icon CLI option to set icon for exe installers

Alexey Semenyuk asemenyuk at openjdk.java.net
Tue Apr 6 17:14:39 UTC 2021


On Tue, 6 Apr 2021 14:45:26 GMT, Andy Herrick <herrick at openjdk.org> wrote:

>> Marked as reviewed by herrick (Reviewer).
>
> One question -
> ICON is only a valid option for creating app launcher.  If creating an installer using --app-image option it would not be a vlid option.
> Does this change imply we should change ValidOptions to change:
>          options.put(CLIOptions.ICON.getId(), USE.LAUNCHER);
> to
>          options.put(CLIOptions.ICON.getId(), USE.ALL);

Good point! I'll add corresponding test case and update ValidOptions accordingly.

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

PR: https://git.openjdk.java.net/jdk/pull/3347


More information about the core-libs-dev mailing list