Review request: 8003562: Provide a command-line tool to find static dependencies (original) (raw)
Mandy Chung mandy.chung at oracle.com
Fri Dec 14 21:41:09 UTC 2012
- Previous message: Review request: 8003562: Provide a command-line tool to find static dependencies
- Next message: Review request: 8003562: Provide a command-line tool to find static dependencies
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Ulf,
Thanks for the review.
On 12/14/2012 6:13 AM, Ulf Zibis wrote:
Am 14.12.2012 11:47, schrieb Ulf Zibis:
Am 14.12.2012 03:15, schrieb Mandy Chung:
JDepsTask:111 Did you mean --summary?
Yes will fix it. Why don't you remain consistent to all other existing java tools? E.g. javac uses: -cp path or -classpath path Double hyphen '--' is never used until today.
'--' is the GNU-style option. pack200 uses GNU-style options for some time. There was some discussion in jigsaw-dev [1] and the new jigsaw CLIs are moving to that direction.
So better: -P -profile Show profile or the file containing a package -R -recursive Traverse all dependencies recursively
Anyway, if you prefer to stick at '--', then you should consistently use it for '--version', '--classpath', '--all'
Renamed to --version.
Maybe I'm in error, but additionally it seems to me, that "-classpath=Foo" doesn't match by:
-classpath is the one inconsistent with others. We think that it would be better to live with "-classpath" as people are familiar with -classpath option.
Thanks Mandy [1] http://mail.openjdk.java.net/pipermail/jigsaw-dev/2009-March/000046.html
- Previous message: Review request: 8003562: Provide a command-line tool to find static dependencies
- Next message: Review request: 8003562: Provide a command-line tool to find static dependencies
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]