use close instead of finish event for file copy completion (yarnpkg#3234

) Windows Defender updates timestamps on files immediately after a write stream closes. Moving utimes update to after the file has closed corrects this issue.
orgycat · Apr 26, 2017 · 0f2de58 · 0f2de58
1 parent 086f9b8
commit 0f2de58
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/util/fs.js b/src/util/fs.js
@@ -523,7 +523,7 @@ export async function copyBulk(
         readStream.pipe(writeStream);
       });
 
-      writeStream.once('finish', function() {
+      writeStream.once('close', function() {
         fs.utimes(data.dest, data.atime, data.mtime, function(err) {
           if (err) {
             reject(err);