RFR: 8023113: tools/jar/ChangeDir.java fails if /tmp/a exists

Xueming Shen xueming.shen at oracle.com
Wed Sep 18 19:45:21 UTC 2013

OK, how about this one? I'm even using lambda:-)



On 09/18/2013 09:42 AM, Xueming Shen wrote:
> On 09/18/2013 03:31 AM, Alan Bateman wrote:
>> On 17/09/2013 22:55, Xueming Shen wrote:
>>> Hi,
>>> Please help the small update to the test case tools/jar/ChangeDir.java,
>>> which might fail if the "dedicated" test directory /tmp/a exists and the
>>> user does not have privilege to delete/clean it up.
>>> The proposed change here is to use a random file name (from File.getTempFile())
>>> for the top test directory (instead of the fixed /tmp/a), and really clean it up
>>> after testing (the old test leaves /tmp/a and those temp files undeleted).
>>> http://cr.openjdk.java.net/~sherman/8023113/webrev
>> I've run into this a few times so thanks for fixing it.
>> What you have is okay although some of the setup and cleanup could be reduced, for example doTest could setup the test tree with:
>> Path topDir = Files.createTempDirectory("delete");
>> Path testDir = Files.createDirectories(topDir.resolve("a").resolve("b"));
>> It's not important for this fix of course, just an observation when looking at an old test.
> thought about using the nio,  but then decided to keep the test as the classic "old" io,
> otherwise I would end up with rewriting the test:-)
> -Sherman
>> -Alan.

More information about the core-libs-dev mailing list