RFR: JDK-8230521: rename --output/-o option and add default value (".")

Andy Herrick andy.herrick at oracle.com
Fri Sep 13 12:10:44 UTC 2019


On 9/12/2019 9:34 PM, Alexander Matveev 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).
>
> - Renamed "--output" to "--dest" and made it optional with default to 
> ".".
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8230521
>
> [2] http://cr.openjdk.java.net/~almatvee/8230629/webrev.00/

I think this is wrong pointer to webrev, should be: 
http://cr.openjdk.java.net/~almatvee/8230521

Overall looks good - I think we need a little reworking of examples in 
help text.

When implementing other CLI changes, I didn't try to re-align the text 
of the examples because I knew this was coming.

If dest is optional, we should not use it in every example.  I was 
thinking of removing the option from all the examples except the first one.

This would also fix the excessively long line in the second example 
(caused by me adding "--package-type app-image".  check that it fits in 
80 columns after dest is removed.

(note - there is also an excessively long line in 
"MSG_Help_default_install_dir". can you re-layout this one to fit in 80 
columns ?)


/Andy




>
> Thanks,
> Alexander


More information about the core-libs-dev mailing list