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

Projects
None yet
3 participants
@alexntt
Contributor

alexntt commented Mar 14, 2016

No description provided.

@theacodes

This comment has been minimized.

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

This comment has been minimized.

Contributor

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

This comment has been minimized.

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

This comment has been minimized.

Contributor

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

This comment has been minimized.

Member

theacodes commented Apr 7, 2017

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

@theacodes theacodes merged commit ecd32fa into pypa:master Apr 7, 2017

ncoghlan added 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