pip freeze generating format that's incorrectly parsed by pip install -r

[JordanReiter]

I have lines in my pip freeze output like:

-e git+git@example.com:widgets.git@3bdaf7d07c44a87c38beb1e8abd163a7edf99adc#egg=widgets
-e git+git@example.com:gadgets.git@3bda...etc...99adc#egg=gadgets

When it comes time to install using pip install -r requirements.txt it appears that pip incorrectly identifies example.com:gadgets.git@3bda...99adc as the full URL of the git repository when it fact it should just be example.com:gadgets.git and then it should check out the 3bda...99adc commit.

If I try to install using the pip freeze output, e.g.

pip install -e git+git@example.com:gadgets.git@3bdaf...99adc#egg=gadgets

I get the same error.

This is all using pip 1.5.4

[JordanReiter]

Note that with the following changes it works correctly:

pip install -e git+git@example.com:/path/to/project/project.git@3bdaf...99adc#egg=project

or

pip install -e git+git@example.com:project.git#egg=project

Although in the latter, it will obviously only pull up the current commit and not the one specified in 3bdaf…

Also, for reasons I don't fathom, it does work for requests to github.

That is,

pip install -e git+git@github.com:JordanReiter/github-project.git@3bdaf...99adc#egg=github_project

works.

[uranusjr]

This is similar to #6293. The parser (based on stdlib urlsplit has problems identifying the path part of the URL when the it contains no / at all, and identify the whole thing after + as netloc. The variant without rev works totally by accident because netloc and path are later joined back and passed to Git (which does the right thing automatically).

Discussions in #6293 seem to favour dropping SCP-style URL support entirely, so the better solution here would be fore pip freeze to spit out git+ssh:// URLs instead.

[sbidoul]

Closing as we propose to remove the git+git@ form in #7554.