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