bower install necessary in postinstall?

[pogopaule]

I have to run npm install in a ember-cli project as root. A bower install with root rights causes some trouble. I'd rather call bower install myself with bower install --allow-root --config.interactive=false

Note that ember-cli once did the same and removed bower install in postinstall: ember-cli/ember-cli@01c16b4

Here is a short snippet of what npm screams at me when ember-cli-qunit runs bower install:

==> dev: bower ESUDO         Cannot be run with sudo
==> dev:
==> dev: Additional error details:
==> dev: Since bower is a user command, there is no need to execute it with superuser permissions.
==> dev: If you're having permission errors when using bower without sudo, please spend a few minutes learning more about how your system should work and make any necessary repairs.
==> dev:
==> dev: http://www.joyent.com/blog/installing-node-and-npm
==> dev: https://gist.github.com/isaacs/579814
==> dev:
==> dev: You can however run a command with sudo using --allow-root option
==> dev: npm
==> dev:
==> dev: ERR!
==> dev:  ember-cli-qunit@0.0.5 postinstall: `bower install`
==> dev: npm
==> dev:
==> dev: ERR!
==> dev:  Exit status 1
==> dev: npm
==> dev:
==> dev: ERR!
==> dev:
==> dev: npm
==> dev:
==> dev: ERR!
==> dev:  Failed at the ember-cli-qunit@0.0.5 postinstall script.
==> dev: npm
==> dev:
==> dev: ERR!
==> dev:  This is most likely a problem with the ember-cli-qunit package,
==> dev: npm
==> dev:
==> dev: ERR!
==> dev:  not with npm itself.
==> dev: npm

...

[jakecraige]

There is a known problem that installing ember-cli/npm with sudo, information found here The solution to your problem would be to set up node/npm so they don't need to run with sudo.

We can't just remove the bower install, this project would not install successfully so it is necessary to have it. If you have another solution I'd be happy to look at it.

Also, the reason it was removed from ember-cli was that the init command already does it, relevant code found here

[pogopaule]

ok thank you for your quick answer!

[stefanpenner]

@jakecraige this postInstall is causing all kinds of annoying output. Can you move it to a pre-publish hook instead? This would also improve reliability of installing this addon.

related: ember-cli/ember-cli#1746

[rwjblue]

@stefanpenner - how would prepublish help?

[rwjblue]

Also, the point is to not have the npm package contain the vendored code.