Review Request: 8157391: jdeps left JarFile open

Mandy Chung mandy.chung at oracle.com
Fri May 20 15:12:22 UTC 2016


> On May 20, 2016, at 7:24 AM, Mandy Chung <mandy.chung at oracle.com> wrote:
> 
>> 
>> On May 19, 2016, at 11:39 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>> 
>> 
>> 
>> On 20/05/2016 04:55, Mandy Chung wrote:
>>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8157391/webrev.00/index.html
>>> 
>>> tools/jdeps/modules/GenModuleInfo.java
>>> tools/jdeps/modules/TransitiveDeps.java
>>> tools/jdeps/modules/InverseDeps.java
>>> 
>>> These tests still failed on windows after JDK-8152502.
>>> 
>>> Now I have the jtreg logs showing that JAR files can’t be deleted although I still can’t reproduce them locally and on JPRT (will find out what’s the difference tomorrow).
>>> 
>> This looks okay to me. One thing you could do in the test is have JdepsUtil.Command be Closeable so that the tests can use try-with-resources and avoid failing with a file open.
> 
> Good point.  I’ll update that.

http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8157391/webrev.01/index.html

Mandy


More information about the core-libs-dev mailing list