The change looks fine to me. Xuelei
On 4/26/2014 1:22 AM, Chris Hegarty wrote: > Hi 8u maintainers, > > Stupidly I mixed up testing results, and consequently pushed a new test to > 8u-dev [1] that has a minor issue, that causes it to fail on all platforms, > all of the time. Root cause, the keystore used by the test has moved location > in JDK 9, and the test should be updated to refer to the keystore location in > JDK 8. Clearly this is not an issue in JDK 9. > > Since the change is trivial, it should be possible to do the approval and > review here? > > diff --git a/test/sun/net/www/http/HttpClient/B8025710.java > b/test/sun/net/www/http/HttpClient/B8025710.java > --- a/test/sun/net/www/http/HttpClient/B8025710.java > +++ b/test/sun/net/www/http/HttpClient/B8025710.java > @@ -43,7 +43,7 @@ > private final static AtomicBoolean connectInServer = new > AtomicBoolean(); > private static final String keystorefile = > System.getProperty("test.src", "./") > - + "/../../../../../javax/net/ssl/etc/keystore"; > + + "/../../../../../sun/security/ssl/etc/keystore"; > private static final String passphrase = "passphrase"; > > public static void main(String[] args) throws Exception { > > -Chris. > > [1] http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/47a5f3d6aeac >