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

Recommend virtualenv instead of pip install --user #36

Closed
DanielSank opened this issue Oct 14, 2017 · 3 comments
Closed

Recommend virtualenv instead of pip install --user #36

DanielSank opened this issue Oct 14, 2017 · 3 comments

Comments

@DanielSank
Copy link
Collaborator

The README recommends the --user flag for pip install. This is generally a very good idea, but it seems like an attempt to avoid the generally accepted "right" way to isolate installed python packages from the system python, namely with virtualenv.

@babbush
Copy link
Contributor

babbush commented Oct 14, 2017

We actually provide a docker image which is a preconfigured virtual environment essentially. I'd like to hear some other opinions on this one because I am of the opinion that we want to avoid unnecessarily complicated install instructions, especially for those who are avoiding a develop install and are likely to be less sophisticated users. In other words, I think that most people who know how to use virtual environments aren't going to need our install instructions in the first place.

@DanielSank
Copy link
Collaborator Author

DanielSank commented Oct 14, 2017

we want to avoid unnecessarily complicated install instructions

Agreed. It's unfortunate that virtualenv is still unnecessarily complicated, but I guess that's the reality of life.

especially for those who are avoiding a develop install and are likely to
be less sophisticated users.

So, in that case, why recommend the --user flag in the first place? Is that to avoid having to sudo?

In other words, I think that most people who know how to use virtual environments
aren't going to need our install instructions in the first place.

Well, see issue #38...

@babbush
Copy link
Contributor

babbush commented Oct 14, 2017

Yes, its so they don't need to use sudo (we certainly don't want that). Regarding #38, I think the solution is to (as you say) remind people to first upgrade pip. I will do this on my next pull request sometime this weekend.

@babbush babbush closed this as completed Oct 14, 2017
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

2 participants