On Tue, 6 Apr 2021 14:45:26 GMT, Andy Herrick <herr...@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