Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

npm install on ember-cli 0.0.39 fails #8

Closed
tute opened this issue Jul 7, 2014 · 7 comments
Closed

npm install on ember-cli 0.0.39 fails #8

tute opened this issue Jul 7, 2014 · 7 comments

Comments

@tute
Copy link
Contributor

tute commented Jul 7, 2014

client master % npm install --save ember-sync
npm WARN package.json ember-cli-ic-ajax@0.1.0 No repository field.
npm http GET https://registry.npmjs.org/ember-sync
npm http 304 https://registry.npmjs.org/ember-sync

> ember-sync@0.1.1 postinstall /Users/tute/Sites/agromentor/client/node_modules/ember-sync
> ./node_modules/bower/bin/bower install

sh: ./node_modules/bower/bin/bower: No such file or directory
npm ERR! ember-sync@0.1.1 postinstall: `./node_modules/bower/bin/bower install`
npm ERR! Exit status 127
npm ERR! 
npm ERR! Failed at the ember-sync@0.1.1 postinstall script.
npm ERR! This is most likely a problem with the ember-sync package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ./node_modules/bower/bin/bower install
npm ERR! You can get their info via:
npm ERR!     npm owner ls ember-sync
npm ERR! There is likely additional logging output above.

npm ERR! System Darwin 13.2.0
npm ERR! command "/usr/local/Cellar/node/0.10.26/bin/node" "/usr/local/bin/npm" "install" "--save" "ember-sync"
npm ERR! cwd /Users/tute/Sites/agromentor/client
npm ERR! node -v v0.10.26
npm ERR! npm -v 1.4.3
npm ERR! code ELIFECYCLE
npm ERR! 
npm ERR! Additional logging details can be found in:
npm ERR!     /Users/tute/Sites/agromentor/client/npm-debug.log
npm ERR! not ok code 0

Thank you for your work!

@kurko
Copy link
Owner

kurko commented Jul 7, 2014

The package.json has specifies that bower install should be run after npm install. That doesn't make sense.

In your case, you can either install bower locally or just send a PR to modify the package.json file and remove this bower command. I'll try doing that myself when I get home. If you have some time, a patch is welcome :)

@tute
Copy link
Contributor Author

tute commented Jul 7, 2014

Which package.json? Happy to help, but I'll need a little more guidance. Thanks for your quick response!

@kurko
Copy link
Owner

kurko commented Jul 7, 2014

This line: https://github.com/kurko/ember-sync/blob/master/package.json#L18 Whenever you run npm install ember-sync, it'll run bower install. If you send a patch removing that line, it should be fixed.

I'll only have time to look into it myself tomorrow evening.

tute added a commit to tute/ember-sync that referenced this issue Jul 7, 2014
tute added a commit to tute/ember-sync that referenced this issue Jul 7, 2014
tute added a commit to tute/ember-sync that referenced this issue Jul 7, 2014
tute added a commit to tute/ember-sync that referenced this issue Jul 7, 2014
tute added a commit to tute/ember-sync that referenced this issue Jul 7, 2014
kurko added a commit that referenced this issue Jul 7, 2014
Remove postinstall step. [progress #8]
@jakecraige
Copy link
Collaborator

closed from that PR and when npm gets updated

@john-kurkowski
Copy link

The package.json has specifies that bower install should be run after npm install. That doesn't make sense.

@kurko, this is unrelated to the PR, but I arrived here from a Google search for the exact error message in another NPM package. For posterity, why doesn't it make sense?

@jakecraige
Copy link
Collaborator

@john-kurkowski The error message is simply saying the bower is not installed globally on your machine. You need to install it with npm install -g bower and the error should go away.

@john-kurkowski
Copy link

My case is the same as the original issue's:

./node_modules/bower/bin/bower: No such file or directory

Some other package is actually looking for bundled Bower, not global.

Regardless I'm not curious about how to fix the error. I'm curious why @kurko thought the npm install + bower install sequence doesn't make sense as a postinstall hook. It's the sequence ember-cli Getting Started requires, for example. Albeit they don't mention postinstall.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants