RFR: JDK-8065673: Makefile enable JAXP tests running
Alan Bateman
Alan.Bateman at oracle.com
Fri Dec 12 10:45:16 UTC 2014
On 11/12/2014 21:10, Tristan Yan wrote:
> Thanks Alan and Joe
> I added jaxp testset, now jaxp_tests is under jaxp testset only. Also
> I have added jaxp testset to pit.
>
> http://cr.openjdk.java.net/~tyan/JDK-8065673/webrev.01/
> <http://cr.openjdk.java.net/%7Etyan/JDK-8065673/webrev.01/>
> http://cr.openjdk.java.net/~tyan/JDK-8065673/jaxp/webrev.00/
> <http://cr.openjdk.java.net/%7Etyan/JDK-8065673/jaxp/webrev.00/>
>
> Thanks
>
I think this looks much better. A suggestion for the test group name is
"jaxp_all" rather than "jaxp_test". A minor comment on jprt.properties
is that the comments should probably use "JAXP" rather than "Jaxp".
A side question: Are there plans to move the JAXP tests from the
jdk/test directory to the jaxp repo so that they can be with their
friends? I think that would make this new testset more useful.
-Alan.
More information about the build-dev
mailing list