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.

Reply via email to