Ok. Will do
03.10.16 17:22, Chris Hegarty wrote:
On 03/10/16 15:19, Sergei Kovalev wrote:
Changed
http://cr.openjdk.java.net/~skovalev/8166791/webrev.03/
This looks fine. Reviewed.
No need to generate the webrev, but can you please check that you list
the java.* modules before the jdk.*
On 03/10/16 14:43, Sergei Kovalev wrote:
Fixed
http://cr.openjdk.java.net/~skovalev/8166791/webrev.02/
java.compiler already exports javax.tools so no need for the
explicit export in the @modules tag. Otherwise this is fine.
-Chris.
03.10.16 16:36, Alan Bateman wrote:
On 03/10/2016
Fixed
http://cr.openjdk.java.net/~skovalev/8166791/webrev.02/
03.10.16 16:36, Alan Bateman wrote:
On 03/10/2016 14:33, Sergei Kovalev wrote:
SSL engin requires security provider. Its implementation located in
jdk.security.auth. In case no jdk module added we getting an exception:
test
On 03/10/2016 14:33, Sergei Kovalev wrote:
SSL engin requires security provider. Its implementation located in
jdk.security.auth. In case no jdk module added we getting an exception:
test ErrorTest.test(): failure
java.lang.IllegalArgumentException: Cannot support
Hi Chris,
Thank you for looking this.
03.10.16 16:05, Chris Hegarty wrote:
Sergei,
On 03/10/16 11:08, Sergei Kovalev wrote:
Resending this for review
27.09.16 18:44, Sergei Kovalev wrote:
Hi team,
Could you please review small fix for regression tests.
BugID:
Sergei,
On 03/10/16 11:08, Sergei Kovalev wrote:
Resending this for review
27.09.16 18:44, Sergei Kovalev wrote:
Hi team,
Could you please review small fix for regression tests.
BugID: https://bugs.openjdk.java.net/browse/JDK-8166791
WebRev:
