+1, looks good.

Roger

On 5/31/2017 11:19 AM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8181307/webrev.00/index.html
2 lines changed: 1 ins; 0 del; 1 mod;
Hi all,

misfortunately, 8180887[1] was created and tested before 8166139[2] has been 
integrated. then the fixes met in jdk10/jdk10, 
java/net/URLClassLoader/closetest/CloseTest.java started to fail due to 'Error. 
can't find jdk.testlibrary.FileUtils in test directory or libraries'. the test 
should be updated to use /test/lib library and not to build removed 
jdk.testlibrary.FileUtils.

jbs: https://bugs.openjdk.java.net/browse/JDK-8181307
webrev: http://cr.openjdk.java.net/~iignatyev//8181307/webrev.00/index.html
testing: java/net tests


[1] https://bugs.openjdk.java.net/browse/JDK-8180887
[2] https://bugs.openjdk.java.net/browse/JDK-8166139

Thanks.
-- Igor

Reply via email to