Review Request for 8027481: jdeps to handle classes with the same package name ...

Mandy Chung mandy.chung at oracle.com
Wed Oct 30 15:28:26 UTC 2013


On 10/30/2013 7:38 AM, Alan Bateman wrote:
> On 30/10/2013 02:06, Mandy Chung wrote:
>> I have added the test cases to verify dependency on javax.crypto and 
>> also a class in a SE package (javax.activity in the test case).  The 
>> updated webrev:
>> http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8027481/webrev.01/
>>
>> Mandy
> I've looked through the updated webrev and the changes look okay to me 
> (I didn't see anything obviously wrong). It is bit annoying to have to 
> special case javax.crypto, I've often wondered if it really necessary 
> to continue to put these classes in jce.jar.
>

I also have the same question and plan to follow up.  When we have 
modules in jdk9, this workaround will no longer be needed.

> The -showLabel looks useful but I could imagine the output being very 
> cluttered (time will tell).
>

Yup and will be useful for clean graphs.

> The new test classes have 2012 in the header, otherwise looks okay.

Will fix it before the push.

Thanks
Mandy




More information about the core-libs-dev mailing list