[MSHARED-1107] Make copyFile succeed with source file having by kwin · Pull Request #105 · apache/maven-shared-utils (original) (raw)

Skip to content

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

asfgit

merged 2 commits into

Jul 25, 2022

Conversation

@kwin

lastModified() = 0

Also check for existence of destination file

@michael-o

Is NIO2 prone to this error? If not, why not use the FileTime there?

@kwin

NIO2 would throw a proper exception when checking last modified date on a non existing file, but that wouldn't help here either. So an existence check is necessary even when using NIO2 for modified date.

kwin and others added 2 commits

July 25, 2022 15:33

@kwin @michael-o

…fied() = 0

Also check for existence of destination file before overwriting.

This closes apache#105

@michael-o

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

Labels

bug

Something isn't working