Review request: 8003562: Provide a command-line tool to find static dependencies

Mandy Chung mandy.chung at oracle.com
Fri Dec 14 21:41:09 UTC 2012


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



More information about the core-libs-dev mailing list