RFR: 8114830: (fs) Files.copy fails due to interference from something else changing the file system [v2]

Brian Burkhalter bpb at openjdk.org
Thu Aug 3 22:52:56 UTC 2023


On Thu, 3 Aug 2023 22:30:58 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Brian Burkhalter has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - 8114830: Convert test to JUnit 5 and expand coverage
>>  - 8114830: Extend EEXIST check to directories, links, and devices
>
> src/java.base/unix/classes/sun/nio/fs/UnixFileSystem.java line 525:
> 
>> 523:             if (x.errno() == EEXIST)
>> 524:                 throw new FileSystemException(target.toString(), null,
>> 525:                     "Directory exists");
> 
> I don't think "Directory exists", "Link exists", "Special file exists" will work for the exception message because the file that exists may be a regular file.

Removed these messages in 171fbd201b77f8321040a5e81092bd5508a64b7f.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15141#discussion_r1283790535


More information about the nio-dev mailing list