Looks good to me Chris.

Approved for jdk8u-dev.

regards,
Sean.

On 25/04/14 18:22, 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

Reply via email to