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
>