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

Explain prominently how to install to /usr/local #230

Merged
merged 1 commit into from Apr 7, 2017

Conversation

alexntt
Copy link
Contributor

@alexntt alexntt commented Mar 14, 2016

No description provided.

@theacodes
Copy link
Member

theacodes commented Apr 6, 2017

@alexntt I'm happy to merge this, but can you re-confirm that this is still necessary and that it does the right thing?

@alexntt
Copy link
Contributor Author

alexntt commented Apr 7, 2017

Hi @jonparrott , I think the change is right, but the bug I opened for this was closed as "working as expected", so while I think it's correct, I suppose pypa thinks it's wrong, see: pypa/pip#3563

@pfmoore
Copy link
Member

pfmoore commented Apr 7, 2017

I think pypa/pip#3563 was closed because we don't expect to change the default behaviour of get-pip.py. But I think that adding a note on the use of --prefix to the docs is fine.

@alexntt
Copy link
Contributor Author

alexntt commented Apr 7, 2017

but can you re-confirm that this is still necessary and that it does the right thing?

$ docker run -it --rm centos:centos5 /bin/bash
# yum install epel-release --disablerepo=libselinux --disablerepo=updates
# yum install python-setuptools wget python26 --disablerepo=libselinux --disablerepo=updates
# md5sum /usr/bin/easy_install
6e55062cdaa4d19cd254821875eb675d  /usr/bin/easy_install
# wget --no-check-certificate https://bootstrap.pypa.io/get-pip.py
# python26 get-pip.py
# md5sum /usr/bin/easy_install
86d01bbcc03f7a93509e88ed1645c39e  /usr/bin/easy_install
$ docker run -it --rm centos:centos5 /bin/bash
# yum install epel-release --disablerepo=libselinux --disablerepo=updates
# yum install python-setuptools wget python26 --disablerepo=libselinux --disablerepo=updates
# md5sum /usr/bin/easy_install
6e55062cdaa4d19cd254821875eb675d  /usr/bin/easy_install
# wget --no-check-certificate https://bootstrap.pypa.io/get-pip.py
# python26 get-pip.py --prefix=/usr/local/
# md5sum /usr/bin/easy_install
6e55062cdaa4d19cd254821875eb675d  /usr/bin/easy_install

, so I think so.

@theacodes
Copy link
Member

theacodes commented Apr 7, 2017

Cool. LGTM. Thanks, @alexntt and @pfmoore. :)

@theacodes theacodes merged commit ecd32fa into pypa:master Apr 7, 2017
ncoghlan pushed a commit to ncoghlan/python-packaging-user-guide that referenced this pull request Jun 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants