uid must be an unsigned int about pnpm

[rstacruz]

How'd you get that? Can I see your package.json?

On Wednesday, February 3, 2016, Francis Torda [email protected]
wrote:

pnpm is awesome but I'm having issue on installing please help

Issue:
bindingchown(pathModule_makeLong(path), uid, gid, req);
^

TypeError: uid must be an unsigned int
at TypeError (native)
at fschown (fsjs:1082:11)
at /usr/local/lib/node_modules/pnpmjs/node_modules/tar-fs/indexjs:215:7
at FSReqWraponcomplete (fsjs:83:15)

—
Reply to this email directly or view it on GitHub
#83.

[rstacruz]

And what's your os and npm/node versions?

$ npm version

On Wednesday, February 3, 2016, Rico Notifications <
[email protected]> wrote:

How'd you get that? Can I see your package.json?

On Wednesday, February 3, 2016, Francis Torda <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');> wrote:

pnpm is awesome but I'm having issue on installing please help

Issue:
bindingchown(pathModule_makeLong(path), uid, gid, req);
^

TypeError: uid must be an unsigned int
at TypeError (native)
at fschown (fsjs:1082:11)
at /usr/local/lib/node_modules/pnpmjs/node_modules/tar-fs/indexjs:215:7
at FSReqWraponcomplete (fsjs:83:15)

—
Reply to this email directly or view it on GitHub
#83.

[freeman29]

OS: Linux Mint
Node: v5.2.0
npm: 3.3.12

package.json: i'm using react-slingshot starter kit

{
"name": "react-slingshot",
"version": "1.1.1",
"description": "Starter kit for creating apps with React and Redux",
"scripts": {
"prestart": "npm run remove-dist",
"start": "parallelshell "npm run lint:tools" "npm run test:watch" "npm run open:src"",
"open:src": "node tools/srcServer.js",
"open:dist": "node tools/distServer.js",
"lint:tools": "eslint webpack.config.js tools",
"clean-dist": "npm run remove-dist && mkdir dist",
"remove-dist": "node_modules/.bin/rimraf ./dist",
"build:html": "node tools/buildHtml.js",
"prebuild": "npm run clean-dist && npm run build:html",
"build": "npm run test && node tools/build.js && npm run open:dist",
"test": "cross-env NODE_ENV=test mocha --reporter progress --compilers js:babel/register --recursive "./src/*/.spec.js"",
"test:watch": "npm run test -- --watch"
},
"author": "Cory House",
"license": "MIT",
"dependencies": {
"object-assign": "4.0.1",
"react": "0.14.6",
"react-dom": "0.14.6",
"react-redux": "4.0.6",
"redux": "3.0.5"
},
"devDependencies": {
"babel": "5.8.23",
"babel-eslint": "4.1.7",
"babel-loader": "5.1.4",
"babel-plugin-react-display-name": "2.0.0",
"babel-plugin-react-transform": "1.1.1",
"browser-sync": "2.11.1",
"chai": "3.4.1",
"cheerio": "0.19.0",
"colors": "1.1.2",
"cross-env": "1.0.7",
"css-loader": "0.23.1",
"eslint": "1.10.3",
"eslint-loader": "1.2.0",
"eslint-plugin-react": "3.15.0",
"extract-text-webpack-plugin": "1.0.1",
"file-loader": "0.8.5",
"mocha": "2.3.4",
"node-sass": "3.4.2",
"parallelshell": "2.0.0",
"react-transform-catch-errors": "1.0.1",
"react-transform-hmr": "1.0.1",
"redbox-react": "1.2.0",
"rimraf": "2.5.0",
"sass-loader": "3.1.2",
"style-loader": "0.13.0",
"watch": "0.17.1",
"webpack": "1.12.11",
"webpack-dev-middleware": "1.4.0",
"webpack-hot-middleware": "2.6.0",
"yargs": "3.32.0"
},
"keywords:": [
"react",
"reactjs",
"hot",
"reload",
"hmr",
"live",
"edit",
"webpack",
"redux",
"flux",
"boilerplate",
"starter"
]
}

[Ingramz]

Also possible to reproduce using following docker one-liner:
docker run --rm node:latest /bin/bash -c "npm install -g pnpm; pnpm install laravel-elixir@^4.0.0"

npm info using [email protected]
npm info using [email protected]

[rstacruz]

shudder... let's see.

[rstacruz]

also, super clever with that docker one-liner, kudos!

[brotchie]

Turns out this was an issue with the tar-stream package required by pnpm's tar-fs dependency. tar-stream didn't support parsing large uids and gids, instead returning NaN values.

I've submitted a pull request mafintosh/tar-stream#47 and this issue should be fixed once it's merged and published to npm.

This command now succeeds when you force npm to use my patched version of tar-stream:

docker run --rm node:latest /bin/bash -c "npm install -g brotchie/tar-stream pnpm; pnpm install laravel-elixir@^4.0.0"

[Ingramz]

The change in tar-stream has been merged and and a release has been made, which has automatically propagated to pnpm, so I believe this issue can be closed.

[rstacruz]

Nice work @brotchie!

[freeman29]

Good job James..

On 18 February 2016 at 01:29, Rico Sta. Cruz [email protected]
wrote:

Nice work @brotchie https://github.com/brotchie!

—
Reply to this email directly or view it on GitHub
#83 (comment).