tar fixes for FreeBSD

jamessanders · isaacs · commit 3fc576736518 · 2010-09-02T11:14:38.000-07:00
Use the p flag to drop file owner/group information.  Specify "f -" rather than assuming stdin.
diff --git a/lib/cache.js b/lib/cache.js
@@ -213,7 +213,7 @@ function unpackTar (tarball, unpackTarget, cb) {
     // gzip {tarball} --decompress --stdout | tar xf - --strip-components=1 -C {unpackTarget}
     pipe( spawn(npm.config.get("gzipbin"), ["--decompress", "--stdout", tarball])
         , spawn( npm.config.get("tar")
-               , ["vx", "--strip-components=1", "-C", unpackTarget]
+               , ["vxpf", "-", "--strip-components=1", "-C", unpackTarget]
                )
         , log.er(cb, "Failed unpacking the tarball.\n"
                    + "This is very rare. Perhaps the 'gzip' or 'tar' configs\n"

Original file line number	Diff line number	Diff line change
`@@ -213,7 +213,7 @@ function unpackTar (tarball, unpackTarget, cb) {`
`213`	`213`	`// gzip {tarball} --decompress --stdout \| tar xf - --strip-components=1 -C {unpackTarget}`
`214`	`214`	`pipe( spawn(npm.config.get("gzipbin"), ["--decompress", "--stdout", tarball])`
`215`	`215`	`, spawn( npm.config.get("tar")`
`216`		`- , ["vx", "--strip-components=1", "-C", unpackTarget]`
	`216`	`+ , ["vxpf", "-", "--strip-components=1", "-C", unpackTarget]`
`217`	`217`	`)`
`218`	`218`	`, log.er(cb, "Failed unpacking the tarball.\n"`
`219`	`219`	`+ "This is very rare. Perhaps the 'gzip' or 'tar' configs\n"`