X Tutup
Skip to content

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

Merged
theacodes merged 1 commit intopypa:masterfrom
alexntt:patch-1
Apr 7, 2017
Merged

Explain prominently how to install to /usr/local#230
theacodes merged 1 commit intopypa:masterfrom
alexntt:patch-1

Conversation

@alexntt
Copy link
Contributor

@alexntt alexntt commented Mar 14, 2016

No description provided.

@alexntt alexntt mentioned this pull request Mar 14, 2016
Closed
@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/packaging.python.org that referenced this pull request Jun 24, 2017
@alexntt @ncoghlan
Explain prominently how to install pip in /usr/local (pypa#230)
3214f9c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

state: needs clarification

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexntt @theacodes @pfmoore
X Tutup