Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-18 Thread Alan Bateman
On 18/02/2016 02:44, Tim Du wrote: I grep the files which are using CertificateBuilder and SimpleOCSPServer from jake/jdk/test, the result as below: - javax/net/ssl/Stapling/HttpsUrlConnClient.java - javax/net/ssl/Stapling/SSLEngineWithStapling.java -

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-17 Thread Tim Du
On 2/17/2016 11:40 PM, Alan Bateman wrote: On 17/02/2016 09:57, Tim Du wrote: Sure. I am aslo updated java.base/sun/security/testlibrary to test/java/security/testlibrary/ , so that keep the testlibrary files could be reused by other tests. Re-generate the latest webrev is

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-17 Thread Alan Bateman
On 17/02/2016 09:57, Tim Du wrote: Sure. I am aslo updated java.base/sun/security/testlibrary to test/java/security/testlibrary/ , so that keep the testlibrary files could be reused by other tests. Re-generate the latest webrev is http://cr.openjdk.java.net/~tidu/8134487/webrev.03/ , Thanks.

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-17 Thread Alan Bateman
On 17/02/2016 09:57, Tim Du wrote: Sure. I am aslo updated java.base/sun/security/testlibrary to test/java/security/testlibrary/ , so that keep the testlibrary files could be reused by other tests. Re-generate the latest webrev is http://cr.openjdk.java.net/~tidu/8134487/webrev.03/ ,

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-17 Thread Tim Du
On 2/17/2016 4:23 PM, Alan Bateman wrote: On 17/02/2016 08:19, Tim Du wrote: Follow you suggestion, I updated the code as below: 1.Create java.base/sun/security/ssl under test/sun/security/ssl/StatusStapling/, move all test files and dependency class into java.base/sun/security/ssl ,removed

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-17 Thread Alan Bateman
On 17/02/2016 08:19, Tim Du wrote: Follow you suggestion, I updated the code as below: 1.Create java.base/sun/security/ssl under test/sun/security/ssl/StatusStapling/, move all test files and dependency class into java.base/sun/security/ssl ,removed the jtreg label from test java files.

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-17 Thread Tim Du
Hi Alan: On 2/16/2016 7:25 PM, Alan Bateman wrote: On 16/02/2016 11:05, Tim Du wrote: Hi Alan： I tried @compile/module to fix this issue before, but it does not work ,Use CertStatusReqExtensionTests.java as example, the process as below: 1. Create java.base/sun/security/ssl folder in

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-16 Thread Alan Bateman
On 16/02/2016 11:05, Tim Du wrote: Hi Alan： I tried @compile/module to fix this issue before, but it does not work ,Use CertStatusReqExtensionTests.java as example, the process as below: 1. Create java.base/sun/security/ssl folder in test/sun/security/ssl/StatusStapling. 2. Move the

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-16 Thread Tim Du
Hi Alan： I tried @compile/module to fix this issue before, but it does not work ,Use CertStatusReqExtensionTests.java as example, the process as below: 1. Create java.base/sun/security/ssl folder in test/sun/security/ssl/StatusStapling. 2. Move the dependency class like TestCase.java

Re: [9] RFR 8134487：updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-15 Thread Alan Bateman
On 16/02/2016 03:31, Tim Du wrote: Hi All: Please help to review the fix for Jigsaw test bug.Thanks. Bug: https://bugs.openjdk.java.net/browse/JDK-8134487 Webrev: http://cr.openjdk.java.net/~tidu/8134487/webrev.01/ The test to access non-public class.To workaround the issue , I create the