1206687 - Python setup script points to the pip install/upgrade guide that doesn't exist anymore

Status: RESOLVED FIXED

(Testing Graveyard :: JSMarionette, defect)

Description

[Oleg Zasypkin [:azasypkin]]

It's just trivial as it's, seems link to the pip upgrade/install has changed recently that may be confusing for newcomers who try to setup build/test environment for the first time [1].

So likely instead of:

http://pip.readthedocs.org/en/latest/installing.html#install-or-upgrade-pip

we should use:

http://pip.readthedocs.org/en/latest/installing/

[1] https://github.com/mozilla-b2g/gaia/blob/29991414eb94b6baa1ec2e63fdb4f6dfae05fb01/tests/jsmarionette/runner/marionette-js-runner/scripts/setup_python.sh#L58

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] azasypkin:bug-1206687-change-pip-install-link > mozilla-b2g:master

Comment 2

[Oleg Zasypkin [:azasypkin]]

Comment on attachment 8663626 [details] [review]
[gaia] azasypkin:bug-1206687-change-pip-install-link > mozilla-b2g:master

Hey Gareth,

Here is a tiny patch to correct this. 

How does it look?

Thanks!

Comment 3

[Gareth Aye [:gaye] (back from PTO)]

Comment on attachment 8663626 [details] [review]
[gaia] azasypkin:bug-1206687-change-pip-install-link > mozilla-b2g:master

LGTM thanks!

Comment 4

[Gareth Aye [:gaye] (back from PTO)]

Landed on master https://github.com/mozilla-b2g/gaia/commit/9936391ba3ede92752f23d66f7d976ed870bd839