Review Request for 8005428: Update jdeps to read the same profile information as by javac
Mandy Chung
mandy.chung at oracle.com
Thu Mar 14 16:56:16 UTC 2013
On 3/14/13 8:20 AM, Alan Bateman wrote:
> On 13/03/2013 20:23, Mandy Chung wrote:
>> This fix removes the temporary workaround for jdeps to print the
>> profile information. The profiles information is embedded in ct.sym
>> generated during the jdk build. jdeps will read ct.sym and parse the
>> annotation to determine which profile a class belongs to. I have
>> extended the test to test the profile information.
>>
>> Webrev at:
>> http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8005428/webrev.00/
> I went through the webrev and it looks good to me (educational too as
> I only had a superficial understanding of the ct.sym before this).
Thanks for the review. Updated webrev:
http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8005428/webrev.01/
>
> In initProfiles then this might be less code to get the right path:
>
> Path home = Paths.get(System.getProperty("java.home"));
> if (home.endsWith("jre"))
> home = home.getParent();
> Path ctsym = home.resolve("lib").resolve("ct.sym");
>
> For consistency then the constants with the annotation types should
> probably be uppercase.
>
I have made these changes.
> You've moved profiles.properties into the test tree to be used by the
> test. I think that is okay for the types that are tested but one thing
> to mention is that these properties file is out of date in a few
> places now.
>
Yes I didn't bother to update it as it's only used by the test. I added
a comment to avoid any confusion.
Thanks
Mandy
More information about the core-libs-dev
mailing list