Re: RFR: JDK-823359: Reorder jpackage help text to focus on package

2019-11-08 Thread Kevin Rushforth
It seems fine, but it might be even better to have both the non-modular and modular application examples be for the default package case and have a single example of --app-image. Also, I presume `--package-type` will be replaced with `--type` (or `-t`) to match the change in command line option

Re: RFR: JDK-823359: Reorder jpackage help text to focus on package

2019-11-08 Thread Alexey Semenyuk
Looks good On 11/8/2019 11:03 AM, Andy Herrick wrote: Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). This fix reorders the sample usage in the help text. [1] https://bugs.openjdk.java.net/browse/J

RFR: JDK-823359: Reorder jpackage help text to focus on package

2019-11-08 Thread Andy Herrick
Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). This fix reorders the sample usage in the help text. [1] https://bugs.openjdk.java.net/browse/JDK-8233591 [2] https://cr.openjdk.java.net/~herrick/823