Merge pull request #80 from npm/isaacs/unpack-no-mtime

fix: do not set mtime when unpacking
npm · Jul 2, 2021 · ac56329 · ac56329
2 parents 7660b0b + 2ed5b64
commit ac56329
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/lib/fetcher.js b/lib/fetcher.js
@@ -437,6 +437,7 @@ class FetcherBase {
     return {
       cwd,
       noChmod: true,
+      noMtime: true,
       filter: (name, entry) => {
         if (/Link$/.test(entry.type))
           return false

diff --git a/test/fetcher.js b/test/fetcher.js
@@ -14,6 +14,19 @@ if (fakeSudo) {
   fs.lchown = fakeChown('lchown')
 }
 
+fs.utimes = () => {
+  throw new Error('do not call utimes')
+}
+fs.futimes = () => {
+  throw new Error('do not call futimes')
+}
+fs.utimesSync = () => {
+  throw new Error('do not call utimesSync')
+}
+fs.futimesSync = () => {
+  throw new Error('do not call futimesSync')
+}
+
 const { relative, resolve, basename } = require('path')
 const t = require('tap')
 const me = t.testdir()