Review Request for 8005428: Update jdeps to read the same profile information as by javac
Alan Bateman
Alan.Bateman at oracle.com
Thu Mar 14 15:20:38 UTC 2013
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).
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.
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.
-Alan.
More information about the core-libs-dev
mailing list