fsevents 1.2.3 → 1.2.8

package/node_modules/set-blocking/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "set-blocking@2.0.0",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "set-blocking@2.0.0",
+  "_from": "set-blocking@~2.0.0",
   "_id": "set-blocking@2.0.0",
   "_inBundle": false,
   "_integrity": "sha1-BF+XgtARrppoA93TgrJDkrPYkPc=",
   "_location": "/set-blocking",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "set-blocking@2.0.0",
+    "raw": "set-blocking@~2.0.0",
     "name": "set-blocking",
     "escapedName": "set-blocking",
-    "rawSpec": "2.0.0",
+    "rawSpec": "~2.0.0",
     "saveSpec": null,
-    "fetchSpec": "2.0.0"
+    "fetchSpec": "~2.0.0"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/npmlog"
+  ],
   "_resolved": "https://registry.npmjs.org/set-blocking/-/set-blocking-2.0.0.tgz",
-  "_spec": "2.0.0",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "045f9782d011ae9a6803ddd382b24392b3d890f7",
+  "_spec": "set-blocking@~2.0.0",
+  "_where": "/Users/pipobscure/fsevents/node_modules/npmlog",
   "author": {
     "name": "Ben Coe",
     "email": "ben@npmjs.com"
@@ -32,6 +29,8 @@
   "bugs": {
     "url": "https://github.com/yargs/set-blocking/issues"
   },
+  "bundleDependencies": false,
+  "deprecated": false,
   "description": "set blocking stdio and stderr ensuring that terminal output does not truncate",
   "devDependencies": {
     "chai": "^3.5.0",

package/node_modules/signal-exit/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "signal-exit@3.0.2",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "signal-exit@3.0.2",
+  "_from": "signal-exit@^3.0.0",
   "_id": "signal-exit@3.0.2",
   "_inBundle": false,
   "_integrity": "sha1-tf3AjxKH6hF4Yo5BXiUTK3NkbG0=",
   "_location": "/signal-exit",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "signal-exit@3.0.2",
+    "raw": "signal-exit@^3.0.0",
     "name": "signal-exit",
     "escapedName": "signal-exit",
-    "rawSpec": "3.0.2",
+    "rawSpec": "^3.0.0",
     "saveSpec": null,
-    "fetchSpec": "3.0.2"
+    "fetchSpec": "^3.0.0"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/gauge"
+  ],
   "_resolved": "https://registry.npmjs.org/signal-exit/-/signal-exit-3.0.2.tgz",
-  "_spec": "3.0.2",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "b5fdc08f1287ea1178628e415e25132b73646c6d",
+  "_spec": "signal-exit@^3.0.0",
+  "_where": "/Users/pipobscure/fsevents/node_modules/gauge",
   "author": {
     "name": "Ben Coe",
     "email": "ben@npmjs.com"
@@ -32,6 +29,8 @@
   "bugs": {
     "url": "https://github.com/tapjs/signal-exit/issues"
   },
+  "bundleDependencies": false,
+  "deprecated": false,
   "description": "when you want to fire an event no matter how a process exits.",
   "devDependencies": {
     "chai": "^3.5.0",

package/node_modules/string-width/package.json

@@ -1,30 +1,28 @@
 {
-  "_args": [
-    [
-      "string-width@1.0.2",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "string-width@1.0.2",
+  "_from": "string-width@^1.0.1",
   "_id": "string-width@1.0.2",
   "_inBundle": false,
   "_integrity": "sha1-EYvfW4zcUaKn5w0hHgfisLmxB9M=",
   "_location": "/string-width",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "string-width@1.0.2",
+    "raw": "string-width@^1.0.1",
     "name": "string-width",
     "escapedName": "string-width",
-    "rawSpec": "1.0.2",
+    "rawSpec": "^1.0.1",
     "saveSpec": null,
-    "fetchSpec": "1.0.2"
+    "fetchSpec": "^1.0.1"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/gauge",
+    "/wide-align"
+  ],
   "_resolved": "https://registry.npmjs.org/string-width/-/string-width-1.0.2.tgz",
-  "_spec": "1.0.2",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "118bdf5b8cdc51a2a7e70d211e07e2b0b9b107d3",
+  "_spec": "string-width@^1.0.1",
+  "_where": "/Users/pipobscure/fsevents/node_modules/gauge",
   "author": {
     "name": "Sindre Sorhus",
     "email": "sindresorhus@gmail.com",
@@ -33,11 +31,13 @@
   "bugs": {
     "url": "https://github.com/sindresorhus/string-width/issues"
   },
+  "bundleDependencies": false,
   "dependencies": {
     "code-point-at": "^1.0.0",
     "is-fullwidth-code-point": "^1.0.0",
     "strip-ansi": "^3.0.0"
   },
+  "deprecated": false,
   "description": "Get the visual width of a string - the number of columns required to display it",
   "devDependencies": {
     "ava": "*",

package/node_modules/string_decoder/package.json

@@ -1,36 +1,35 @@
 {
-  "_args": [
-    [
-      "string_decoder@1.1.1",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "string_decoder@1.1.1",
+  "_from": "string_decoder@~1.1.1",
   "_id": "string_decoder@1.1.1",
   "_inBundle": false,
   "_integrity": "sha512-n/ShnvDi6FHbbVfviro+WojiFzv+s8MPMHBczVePfUpDJLwoLT0ht1l4YwBCbi8pJAveEEdnkHyPyTP/mzRfwg==",
   "_location": "/string_decoder",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "string_decoder@1.1.1",
+    "raw": "string_decoder@~1.1.1",
     "name": "string_decoder",
     "escapedName": "string_decoder",
-    "rawSpec": "1.1.1",
+    "rawSpec": "~1.1.1",
     "saveSpec": null,
-    "fetchSpec": "1.1.1"
+    "fetchSpec": "~1.1.1"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/readable-stream"
+  ],
   "_resolved": "https://registry.npmjs.org/string_decoder/-/string_decoder-1.1.1.tgz",
-  "_spec": "1.1.1",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "9cf1611ba62685d7030ae9e4ba34149c3af03fc8",
+  "_spec": "string_decoder@~1.1.1",
+  "_where": "/Users/pipobscure/fsevents/node_modules/readable-stream",
   "bugs": {
     "url": "https://github.com/nodejs/string_decoder/issues"
   },
+  "bundleDependencies": false,
   "dependencies": {
     "safe-buffer": "~5.1.0"
   },
+  "deprecated": false,
   "description": "The string_decoder module from Node core",
   "devDependencies": {
     "babel-polyfill": "^6.23.0",

package/node_modules/strip-ansi/package.json

@@ -1,30 +1,28 @@
 {
-  "_args": [
-    [
-      "strip-ansi@3.0.1",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "strip-ansi@3.0.1",
+  "_from": "strip-ansi@^3.0.1",
   "_id": "strip-ansi@3.0.1",
   "_inBundle": false,
   "_integrity": "sha1-ajhfuIU9lS1f8F0Oiq+UJ43GPc8=",
   "_location": "/strip-ansi",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "strip-ansi@3.0.1",
+    "raw": "strip-ansi@^3.0.1",
     "name": "strip-ansi",
     "escapedName": "strip-ansi",
-    "rawSpec": "3.0.1",
+    "rawSpec": "^3.0.1",
     "saveSpec": null,
-    "fetchSpec": "3.0.1"
+    "fetchSpec": "^3.0.1"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/gauge",
+    "/string-width"
+  ],
   "_resolved": "https://registry.npmjs.org/strip-ansi/-/strip-ansi-3.0.1.tgz",
-  "_spec": "3.0.1",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "6a385fb8853d952d5ff05d0e8aaf94278dc63dcf",
+  "_spec": "strip-ansi@^3.0.1",
+  "_where": "/Users/pipobscure/fsevents/node_modules/gauge",
   "author": {
     "name": "Sindre Sorhus",
     "email": "sindresorhus@gmail.com",
@@ -33,9 +31,11 @@
   "bugs": {
     "url": "https://github.com/chalk/strip-ansi/issues"
   },
+  "bundleDependencies": false,
   "dependencies": {
     "ansi-regex": "^2.0.0"
   },
+  "deprecated": false,
   "description": "Strip ANSI escape codes",
   "devDependencies": {
     "ava": "*",

package/node_modules/strip-json-comments/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "strip-json-comments@2.0.1",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "strip-json-comments@2.0.1",
+  "_from": "strip-json-comments@~2.0.1",
   "_id": "strip-json-comments@2.0.1",
   "_inBundle": false,
   "_integrity": "sha1-PFMZQukIwml8DsNEhYwobHygpgo=",
   "_location": "/strip-json-comments",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "strip-json-comments@2.0.1",
+    "raw": "strip-json-comments@~2.0.1",
     "name": "strip-json-comments",
     "escapedName": "strip-json-comments",
-    "rawSpec": "2.0.1",
+    "rawSpec": "~2.0.1",
     "saveSpec": null,
-    "fetchSpec": "2.0.1"
+    "fetchSpec": "~2.0.1"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/rc"
+  ],
   "_resolved": "https://registry.npmjs.org/strip-json-comments/-/strip-json-comments-2.0.1.tgz",
-  "_spec": "2.0.1",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "3c531942e908c2697c0ec344858c286c7ca0a60a",
+  "_spec": "strip-json-comments@~2.0.1",
+  "_where": "/Users/pipobscure/fsevents/node_modules/rc",
   "author": {
     "name": "Sindre Sorhus",
     "email": "sindresorhus@gmail.com",
@@ -33,6 +30,8 @@
   "bugs": {
     "url": "https://github.com/sindresorhus/strip-json-comments/issues"
   },
+  "bundleDependencies": false,
+  "deprecated": false,
   "description": "Strip comments from JSON. Lets you use comments in your JSON files!",
   "devDependencies": {
     "ava": "*",

package/node_modules/tar/README.md

@@ -108,7 +108,7 @@ tar.c( // or tar.create
     gzip: <true|gzip options>
   },
   ['some', 'files', 'and', 'folders']
-).pipe(fs.createWriteStream('my-tarball.tgz')
+).pipe(fs.createWriteStream('my-tarball.tgz'))
 ```
 
 To replicate `tar xf my-tarball.tgz` you'd do:
@@ -230,6 +230,9 @@ The following options are supported:
   Note that this prevents using other mtime-based features like
   `tar.update` or the `keepNewer` option with the resulting tar archive.
   [Alias: `m`, `no-mtime`]
+- `mtime` Set to a `Date` object to force a specific `mtime` for
+  everything added to the archive.  Overridden by `noMtime`.
+
 
 The following options are mostly internal, but can be modified in some
 advanced use cases, such as re-using caches between runs.
@@ -534,7 +537,6 @@ The following options are supported:
 - `mtime` Set to a `Date` object to force a specific `mtime` for
   everything added to the archive.  Overridden by `noMtime`.
 
-
 #### add(path)
 
 Adds an entry to the archive.  Returns the Pack stream.
@@ -784,6 +786,9 @@ The following options are supported:
 - `noMtime` Set to true to omit writing `mtime` values for entries.
   Note that this prevents using other mtime-based features like
   `tar.update` or the `keepNewer` option with the resulting tar archive.
+- `umask` Set to restrict the modes on the entries in the archive,
+  somewhat like how umask works on file creation.  Defaults to
+  `process.umask()` on unix systems, or `0o22` on Windows.
 
 #### warn(message, data)
 

package/node_modules/tar/lib/header.js

@@ -6,13 +6,14 @@
 
 const Buffer = require('./buffer.js')
 const types = require('./types.js')
-const pathModule = require('path')
+const pathModule = require('path').posix
 const large = require('./large-numbers.js')
 
+const SLURP = Symbol('slurp')
 const TYPE = Symbol('type')
 
 class Header {
-  constructor (data, off) {
+  constructor (data, off, ex, gex) {
     this.cksumValid = false
     this.needPax = false
     this.nullBlock = false
@@ -35,12 +36,12 @@ class Header {
     this.ctime = null
 
     if (Buffer.isBuffer(data))
-      this.decode(data, off || 0)
+      this.decode(data, off || 0, ex, gex)
     else if (data)
       this.set(data)
   }
 
-  decode (buf, off) {
+  decode (buf, off, ex, gex) {
     if (!off)
       off = 0
 
@@ -55,6 +56,11 @@ class Header {
     this.mtime = decDate(buf, off + 136, 12)
     this.cksum = decNumber(buf, off + 148, 12)
 
+    // if we have extended or global extended headers, apply them now
+    // See https://github.com/npm/node-tar/pull/187
+    this[SLURP](ex)
+    this[SLURP](gex, true)
+
     // old tar versions marked dirs as a file with a trailing /
     this[TYPE] = decString(buf, off + 156, 1)
     if (this[TYPE] === '')
@@ -101,6 +107,16 @@ class Header {
       this.nullBlock = true
   }
 
+  [SLURP] (ex, global) {
+    for (let k in ex) {
+      // we slurp in everything except for the path attribute in
+      // a global extended header, because that's weird.
+      if (ex[k] !== null && ex[k] !== undefined &&
+          !(global && k === 'path'))
+        this[k] = ex[k]
+    }
+  }
+
   encode (buf, off) {
     if (!buf) {
       buf = this.block = Buffer.alloc(512)

package/node_modules/tar/lib/mode-fix.js

@@ -0,0 +1,14 @@
+'use strict'
+module.exports = (mode, isDir) => {
+  mode &= 0o7777
+  // if dirs are readable, then they should be listable
+  if (isDir) {
+    if (mode & 0o400)
+      mode |= 0o100
+    if (mode & 0o40)
+      mode |= 0o10
+    if (mode & 0o4)
+      mode |= 0o1
+  }
+  return mode
+}

package/node_modules/tar/lib/pack.js

@@ -71,6 +71,7 @@ const Pack = warner(class Pack extends MiniPass {
     this.linkCache = opt.linkCache || new Map()
     this.statCache = opt.statCache || new Map()
     this.readdirCache = opt.readdirCache || new Map()
+
     this[WRITEENTRYCLASS] = WriteEntry
     if (typeof opt.onwarn === 'function')
       this.on('warn', opt.onwarn)

package/node_modules/tar/lib/parse.js

@@ -29,6 +29,7 @@ const maxMetaEntrySize = 1024 * 1024
 const Entry = require('./read-entry.js')
 const Pax = require('./pax.js')
 const zlib = require('minizlib')
+const Buffer = require('./buffer.js')
 
 const gzipHeader = Buffer.from([0x1f, 0x8b])
 const STATE = Symbol('state')
@@ -101,7 +102,7 @@ module.exports = warner(class Parser extends EE {
   }
 
   [CONSUMEHEADER] (chunk, position) {
-    const header = new Header(chunk, position)
+    const header = new Header(chunk, position, this[EX], this[GEX])
 
     if (header.nullBlock)
       this[EMIT]('nullBlock')

package/node_modules/tar/lib/unpack.js

@@ -12,6 +12,7 @@ const wc = require('./winchars.js')
 
 const ONENTRY = Symbol('onEntry')
 const CHECKFS = Symbol('checkFs')
+const ISREUSABLE = Symbol('isReusable')
 const MAKEFS = Symbol('makeFs')
 const FILE = Symbol('file')
 const DIRECTORY = Symbol('directory')
@@ -32,6 +33,45 @@ const SKIP = Symbol('skip')
 const DOCHOWN = Symbol('doChown')
 const UID = Symbol('uid')
 const GID = Symbol('gid')
+const crypto = require('crypto')
+
+// Unlinks on Windows are not atomic.
+//
+// This means that if you have a file entry, followed by another
+// file entry with an identical name, and you cannot re-use the file
+// (because it's a hardlink, or because unlink:true is set, or it's
+// Windows, which does not have useful nlink values), then the unlink
+// will be committed to the disk AFTER the new file has been written
+// over the old one, deleting the new file.
+//
+// To work around this, on Windows systems, we rename the file and then
+// delete the renamed file.  It's a sloppy kludge, but frankly, I do not
+// know of a better way to do this, given windows' non-atomic unlink
+// semantics.
+//
+// See: https://github.com/npm/node-tar/issues/183
+/* istanbul ignore next */
+const unlinkFile = (path, cb) => {
+  if (process.platform !== 'win32')
+    return fs.unlink(path, cb)
+
+  const name = path + '.DELETE.' + crypto.randomBytes(16).toString('hex')
+  fs.rename(path, name, er => {
+    if (er)
+      return cb(er)
+    fs.unlink(name, cb)
+  })
+}
+
+/* istanbul ignore next */
+const unlinkFileSync = path => {
+  if (process.platform !== 'win32')
+    return fs.unlinkSync(path)
+
+  const name = path + '.DELETE.' + crypto.randomBytes(16).toString('hex')
+  fs.renameSync(path, name)
+  fs.unlinkSync(name)
+}
 
 // this.gid, entry.gid, this.processUid
 const uint32 = (a, b, c) =>
@@ -138,6 +178,12 @@ class Unpack extends Parser {
       if (parts.length < this.strip)
         return false
       entry.path = parts.slice(this.strip).join('/')
+
+      if (entry.type === 'Link') {
+        const linkparts = entry.linkpath.split(/\/|\\/)
+        if (linkparts.length >= this.strip)
+          entry.linkpath = linkparts.slice(this.strip).join('/')
+      }
     }
 
     if (!this.preservePaths) {
@@ -291,8 +337,10 @@ class Unpack extends Parser {
     })
 
     const tx = this.transform ? this.transform(entry) || entry : entry
-    if (tx !== entry)
+    if (tx !== entry) {
+      tx.on('error', er => this[ONERROR](er, entry))
       entry.pipe(tx)
+    }
     tx.pipe(stream)
   }
 
@@ -351,6 +399,17 @@ class Unpack extends Parser {
     entry.resume()
   }
 
+  // Check if we can reuse an existing filesystem entry safely and
+  // overwrite it, rather than unlinking and recreating
+  // Windows doesn't report a useful nlink, so we just never reuse entries
+  [ISREUSABLE] (entry, st) {
+    return entry.type === 'File' &&
+      !this.unlink &&
+      st.isFile() &&
+      st.nlink <= 1 &&
+      process.platform !== 'win32'
+  }
+
   // check if a thing is there, and if so, try to clobber it
   [CHECKFS] (entry) {
     this[PEND]()
@@ -360,7 +419,7 @@ class Unpack extends Parser {
       fs.lstat(entry.absolute, (er, st) => {
         if (st && (this.keep || this.newer && st.mtime > entry.mtime))
           this[SKIP](entry)
-        else if (er || (entry.type === 'File' && !this.unlink && st.isFile()))
+        else if (er || this[ISREUSABLE](entry, st))
           this[MAKEFS](null, entry)
         else if (st.isDirectory()) {
           if (entry.type === 'Directory') {
@@ -371,7 +430,7 @@ class Unpack extends Parser {
           } else
             fs.rmdir(entry.absolute, er => this[MAKEFS](er, entry))
         } else
-          fs.unlink(entry.absolute, er => this[MAKEFS](er, entry))
+          unlinkFile(entry.absolute, er => this[MAKEFS](er, entry))
       })
     })
   }
@@ -422,7 +481,7 @@ class UnpackSync extends Unpack {
       const st = fs.lstatSync(entry.absolute)
       if (this.keep || this.newer && st.mtime > entry.mtime)
         return this[SKIP](entry)
-      else if (entry.type === 'File' && !this.unlink && st.isFile())
+      else if (this[ISREUSABLE](entry, st))
         return this[MAKEFS](null, entry)
       else {
         try {
@@ -433,7 +492,7 @@ class UnpackSync extends Unpack {
             } else
               fs.rmdirSync(entry.absolute)
           } else
-            fs.unlinkSync(entry.absolute)
+            unlinkFileSync(entry.absolute)
           return this[MAKEFS](null, entry)
         } catch (er) {
           return this[ONERROR](er, entry)
@@ -461,8 +520,10 @@ class UnpackSync extends Unpack {
       return oner(er)
     }
     const tx = this.transform ? this.transform(entry) || entry : entry
-    if (tx !== entry)
+    if (tx !== entry) {
+      tx.on('error', er => this[ONERROR](er, entry))
       entry.pipe(tx)
+    }
 
     tx.on('data', chunk => {
       try {

package/node_modules/tar/lib/write-entry.js

@@ -23,9 +23,12 @@ const ONREADLINK = Symbol('onreadlink')
 const OPENFILE = Symbol('openfile')
 const ONOPENFILE = Symbol('onopenfile')
 const CLOSE = Symbol('close')
+const MODE = Symbol('mode')
 const warner = require('./warn-mixin.js')
 const winchars = require('./winchars.js')
 
+const modeFix = require('./mode-fix.js')
+
 const WriteEntry = warner(class WriteEntry extends MiniPass {
   constructor (p, opt) {
     opt = opt || {}
@@ -104,6 +107,10 @@ const WriteEntry = warner(class WriteEntry extends MiniPass {
     }
   }
 
+  [MODE] (mode) {
+    return modeFix(mode, this.type === 'Directory')
+  }
+
   [HEADER] () {
     if (this.type === 'Directory' && this.portable)
       this.noMtime = true
@@ -113,7 +120,7 @@ const WriteEntry = warner(class WriteEntry extends MiniPass {
       linkpath: this.linkpath,
       // only the permissions and setuid/setgid/sticky bitflags
       // not the higher-order bits that specify file type
-      mode: this.stat.mode & 0o7777,
+      mode: this[MODE](this.stat.mode),
       uid: this.portable ? null : this.stat.uid,
       gid: this.portable ? null : this.stat.gid,
       size: this.stat.size,
@@ -220,11 +227,21 @@ const WriteEntry = warner(class WriteEntry extends MiniPass {
 
   [ONREAD] (fd, buf, offset, length, pos, remain, blockRemain, bytesRead) {
     if (bytesRead <= 0 && remain > 0) {
-      const er = new Error('unexpected EOF')
+      const er = new Error('encountered unexpected EOF')
+      er.path = this.absolute
+      er.syscall = 'read'
+      er.code = 'EOF'
+      this[CLOSE](fd)
+      return this.emit('error', er)
+    }
+
+    if (bytesRead > remain) {
+      const er = new Error('did not encounter expected EOF')
       er.path = this.absolute
       er.syscall = 'read'
       er.code = 'EOF'
-      this.emit('error', er)
+      this[CLOSE](fd)
+      return this.emit('error', er)
     }
 
     // null out the rest of the buffer, if we could fit the block padding
@@ -312,9 +329,7 @@ const WriteEntryTar = warner(class WriteEntryTar extends MiniPass {
       this.noMtime = true
 
     this.path = readEntry.path
-    this.mode = readEntry.mode
-    if (this.mode)
-      this.mode = this.mode & 0o7777
+    this.mode = this[MODE](readEntry.mode)
     this.uid = this.portable ? null : readEntry.uid
     this.gid = this.portable ? null : readEntry.gid
     this.uname = this.portable ? null : readEntry.uname
@@ -376,6 +391,10 @@ const WriteEntryTar = warner(class WriteEntryTar extends MiniPass {
     readEntry.pipe(this)
   }
 
+  [MODE] (mode) {
+    return modeFix(mode, this.type === 'Directory')
+  }
+
   write (data) {
     const writeLen = data.length
     if (writeLen > this.blockRemain)