search

treyhunner / names

Random name generator
MIT License
462 stars 107 forks source link

Removed sudo from installation command #14

Closed svisser closed 9 years ago

svisser commented 9 years ago

Recommending installation via sudo should not be done as use of virtualenv is preferred. Given the popularity of virtualenv for isolating dependencies (which is now also part of Python 3), it makes sense to only recommend installation with regular permissions, not sudo. This library should be installed as one of the other isolated dependencies of a project and as such it shouldn't be installed as a global dependency (only in rare cases).

treyhunner commented 9 years ago

As a command-line script I think it makes sense to install this system-wide. I do agree that sudo does not need to me in the README though. Merging now. Thanks!

svisser commented 9 years ago

Thanks for merging!