Skip to content

Conversation

@shama
Copy link
Member

@shama shama commented Apr 5, 2016

Fixes GH-1499

This fixes the issue where grunt-cli might not yet exist when npm is trying to create a binary.

@vladikoff Mind giving this a full vetting? I tested on npm@2 and npm@3 but want to make sure I'm not missing something big with this. Thanks!

@vladikoff
Copy link
Member

Same problem it seems:

➜  lnn npm i grunt-cli gruntjs/grunt#fixbin
npm WARN package.json [email protected] No description
npm WARN package.json [email protected] No repository field.
npm WARN package.json [email protected] No README data
npm ERR! Darwin 15.4.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "i" "grunt-cli" "gruntjs/grunt#fixbin"
npm ERR! node v4.2.3
npm ERR! npm  v2.15.3
npm ERR! path /Users/vladikoff/tmp/lnn/node_modules/grunt/bin/grunt
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall chmod

npm ERR! enoent ENOENT: no such file or directory, chmod '/Users/vladikoff/tmp/lnn/node_modules/grunt/bin/grunt'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent 

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/vladikoff/tmp/lnn/npm-debug.log
➜  lnn 

@vladikoff
Copy link
Member

Even after removing node modules: rm -rf node_modules

@MylesBorins
Copy link

Looks like you are going to have to include bin/ in the files section of the package.json or it will be ignored.

Line 69

@shama
Copy link
Member Author

shama commented Apr 5, 2016

Nice catch @thealphanerd! Thank you!

"files": [
"lib"
"lib",
"bin"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bin, lib and squash :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BTW second time we got bitten by this today :/

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah dang, sorry about that!

@vladikoff vladikoff merged commit d097191 into master Apr 5, 2016
@vladikoff
Copy link
Member

@shama ok 1.0.1 right? I'll draft a changelog

@shama shama deleted the fixbin branch April 5, 2016 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

install fails with ENOENT and off path

5 participants