Re: RFR(s): 8166378: Missing dependencies in several java/security tests

2016-10-04 Thread Sergei Kovalev
Thank you for review 27.09.16 16:06, Xuelei Fan wrote: Looks fine to me. Xuelei On 9/20/2016 9:32 PM, Sergei Kovalev wrote: Hello team, Please review very small fix for several regression tests. BugID: https://bugs.openjdk.java.net/browse/JDK-8166378 WebReview: http://cr.openjdk.java.net/~

Re: RFR(s): 8166378: Missing dependencies in several java/security tests

2016-10-03 Thread Sergei Kovalev
Resending this for review 27.09.16 15:56, Sergei Kovalev wrote: Hello team, This is re-request for review. 20.09.16 16:32, Sergei Kovalev wrote: Hello team, Please review very small fix for several regression tests. BugID: https://bugs.openjdk.java.net/browse/JDK-8166378 WebReview: http:/

Re: RFR(s): 8166378: Missing dependencies in several java/security tests

2016-09-27 Thread Xuelei Fan
Looks fine to me. Xuelei On 9/20/2016 9:32 PM, Sergei Kovalev wrote: Hello team, Please review very small fix for several regression tests. BugID: https://bugs.openjdk.java.net/browse/JDK-8166378 WebReview: http://cr.openjdk.java.net/~skovalev/8166378/webrev.00 Issue: The tests has no declar

Re: RFR(s): 8166378: Missing dependencies in several java/security tests

2016-09-27 Thread Sergei Kovalev
Hello team, This is re-request for review. 20.09.16 16:32, Sergei Kovalev wrote: Hello team, Please review very small fix for several regression tests. BugID: https://bugs.openjdk.java.net/browse/JDK-8166378 WebReview: http://cr.openjdk.java.net/~skovalev/8166378/webrev.00 Issue: The tests

RFR(s): 8166378: Missing dependencies in several java/security tests

2016-09-20 Thread Sergei Kovalev
Hello team, Please review very small fix for several regression tests. BugID: https://bugs.openjdk.java.net/browse/JDK-8166378 WebReview: http://cr.openjdk.java.net/~skovalev/8166378/webrev.00 Issue: The tests has no declared dependencies on jdk.security modules. This leads the tests to fail i