Re: RFR(s): JDK-8169866: [TESTBUG] com/sun/security/ tests have undeclared modules dependencies

2016-11-28 Thread Wang Weijun 

Hi Sergei

Looks good to me.

Thanks
Max

On 11/28/2016 9:17 PM, Sergei Kovalev wrote:

R-sending request for review


17.11.16 15:43, Sergei Kovalev wrote:

Hello team,

Please review a small fix for security tests.

BugID: https://bugs.openjdk.java.net/browse/JDK-8169866
Web review: http://cr.openjdk.java.net/~skovalev/8169866/webrev.00/

Issue: Tests from package com/sun/security failing in case of usage
"--limit-modules java.base" command line options due to undeclared
dependencies.
Solution: add declaration of required modules.

Re: RFR(s): JDK-8169866: [TESTBUG] com/sun/security/ tests have undeclared modules dependencies

2016-11-28 Thread Sergei Kovalev 

R-sending request for review


17.11.16 15:43, Sergei Kovalev wrote:

Hello team,

Please review a small fix for security tests.

BugID: https://bugs.openjdk.java.net/browse/JDK-8169866
Web review: http://cr.openjdk.java.net/~skovalev/8169866/webrev.00/

Issue: Tests from package com/sun/security failing in case of usage 
"--limit-modules java.base" command line options due to undeclared 
dependencies.

Solution: add declaration of required modules.



--
With best regards,
Sergei

RFR(s): JDK-8169866: [TESTBUG] com/sun/security/ tests have undeclared modules dependencies

2016-11-17 Thread Sergei Kovalev 

Hello team,

Please review a small fix for security tests.

BugID: https://bugs.openjdk.java.net/browse/JDK-8169866
Web review: http://cr.openjdk.java.net/~skovalev/8169866/webrev.00/

Issue: Tests from package com/sun/security failing in case of usage 
"--limit-modules java.base" command line options due to undeclared 
dependencies.

Solution: add declaration of required modules.

--
With best regards,
Sergei