On 12/12/2014 2:45 AM, Alan Bateman wrote:
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.
Yes, it's in our plan to move jaxp tests from jdk/test to jaxp/test.
-Joe
-Alan.
