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/~tyan/JDK-8065673/webrev.01/> http://cr.openjdk.java.net/~tyan/JDK-8065673/jaxp/webrev.00/ <http://cr.openjdk.java.net/~tyan/JDK-8065673/jaxp/webrev.00/> Thanks > On Dec 11, 2014, at 9:46 AM, huizhe wang <huizhe.w...@oracle.com> wrote: > > > On 12/11/2014 7:07 AM, Alan Bateman wrote: >> On 11/12/2014 14:38, Tristan Yan wrote: >>> Hi everyone >>> >>> Could you please to review this small makefile change for JAXP tests >>> >>> http://cr.openjdk.java.net/~tyan/JDK-8065673/webrev.00/ >>> <http://cr.openjdk.java.net/~tyan/JDK-8065673/webrev.00/> >>> http://cr.openjdk.java.net/~tyan/JDK-8065673/jaxp/webrev.00/ >>> <http://cr.openjdk.java.net/~tyan/JDK-8065673/jaxp/webrev.00/> >>> >>> corresponding bug is https://bugs.openjdk.java.net/browse/JDK-8065673 >>> <https://bugs.openjdk.java.net/browse/JDK-8065673> >>> >>> Here is the background for this change. >>> As you may know we’ve pushed some new jaxp tests into openjdk. Since these >>> tests have been put into different repo with current idk tests. We need a >>> makefile to support these tests running in jprt system and nightly. In the >>> future, we may put these tests back to idk repo, we have to maintain a >>> consistent makefile with current idk tests. So I steal the makefile from >>> idk test repo and make minimal change as possible. >>> For supporting running tests in jprt system one new target(jaxp_test) has >>> been added in top level makefile and jaxp_test will be considered as part >>> of core testset. >>> >> The alternative to adding this to the "core" testset is to create a new >> testset ("jaxp" or "xml" for example). You could then include this in the >> "pit" testset so that it runs langtools, core, svc, and jaxp. > > Ok, we'll add a new testset "jaxp". > > -Joe > >> >> -Alan >
