JDK 9 RFR of JDK-8074870: Really add javax/xml/jaxp/testng/validation to othervm.dirs in TEST.ROOT

[Roger Riggs]

Hi Joe,

Looks fine.

Roger


On 3/10/2015 2:12 PM, joe darcy wrote:
> Hello,
>
> Recently I sent out a review for
>
>     JDK-8074714 : Add javax/xml/jaxp/testng/validation to othervm.dirs 
> in TEST.ROOT
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-March/032151.html 
>
>
> However, I actually added the directory in question to the 
> exclusiveAccess.dirs property rather than the othervm.dirs property. 
> (Doh!)
>
> Please review the patch below to remove 
> javax/xml/jaxp/testng/validation from exclusiveAccess.dirs and put it 
> on othervm.dirs, as originally intended.
>
> Thanks,
>
> -Joe
>
> --- old/test/TEST.ROOT    2015-03-10 11:07:18.027225609 -0700
> +++ new/test/TEST.ROOT    2015-03-10 11:07:17.843225610 -0700
> @@ -5,10 +5,10 @@
>  keys=2d dnd i18n
>
>  # Tests that must run in othervm mode
> -othervm.dirs=java/awt java/beans javax/accessibility javax/imageio 
> javax/sound javax/print javax/management com/sun/awt sun/awt 
> sun/java2d sun/pisce
> s
> +othervm.dirs=java/awt java/beans javax/accessibility javax/imageio 
> javax/sound javax/print javax/management com/sun/awt sun/awt 
> sun/java2d sun/pisce
> s javax/xml/jaxp/testng/validation
>
>  # Tests that cannot run concurrently
> -exclusiveAccess.dirs=java/rmi/Naming java/util/prefs 
> sun/management/jmxremote sun/tools/jstatd sun/security/mscapi 
> java/util/stream javax/xml/jaxp/t
> estng/validation
> +exclusiveAccess.dirs=java/rmi/Naming java/util/prefs 
> sun/management/jmxremote sun/tools/jstatd sun/security/mscapi 
> java/util/stream
>
>  # Group definitions
>  groups=TEST.groups [closed/TEST.groups]
>