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

README: remove sudo from "sudo npm install"? #1370

Closed
benatkin opened this issue May 13, 2014 · 3 comments
Closed

README: remove sudo from "sudo npm install"? #1370

benatkin opened this issue May 13, 2014 · 3 comments
Assignees

Comments

@benatkin
Copy link

The creator of npm back in 2010 recommended against using sudo, and AFAIK that's still the policy. http://howtonode.org/introduction-to-npm

How about making the instructions simply say npm install -g ionic instead of sudo npm install -g ionic?

Besides being preferable, it's also more reliable instructions, because if the user's setup requires sudo and they don't use sudo, they'll get an error right away, where as if the user's setup doesn't require sudo and they use sudo, it will go ahead and install as root, which will make their permissions in their npm directory inconsistent.

@ashconnell
Copy link

It does depend on how the user has installed npm in the first place.

@benatkin
Copy link
Author

I asked in #npm on freenode and at least one person there was using sudo for global modules. I also noticed that cordova also suggests installing with sudo. Maybe it's just better to keep it how it is, for consistency's sake. I know on linux depending on what the box (or container) does either can be acceptable from a security standpoint (if the box has both an app and a database on it, it's probably better to keep files local to the app). I'm a homebrew user on OS X so it works to install without sudo. Unfortunately it's hard to see what a fresh install on OS X with just the .pkg from http://nodejs.org/ would be like. It might require sudo by default.

It is possible to configure an npm installation in a directory that requires sudo to install global modules in a different directory, but there doesn't seem to be many good resources showing how to do this on google. So I figure that a lot of people are using sudo to install global modules right now.

When this changes, maybe then the docs should be updated to remove sudo.

SobolevLexa pushed a commit to SobolevLexa/ionic that referenced this issue May 20, 2014
If a text input is located in the same area as a button which was just
tapped, which was probably because of a view transition, the text input
gets focus 300ms later. This is an issue on Android because it also
fires off a mousedown event. Resolved by remembering the touchend
target then checking if it’s different from the mousedown target.
Closes ionic-team#1370
@ionitron-bot
Copy link

ionitron-bot bot commented Sep 6, 2018

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants