Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Error installing from svn with revision number #52

Closed
vbabiy opened this Issue Mar 15, 2011 · 12 comments

Comments

Projects
None yet
5 participants
Contributor

vbabiy commented Mar 15, 2011

I use this command to install package from svn:

#!shell

pip install -e svn://mysvndomain.com/myapp/trunk@635#egg=myapp

And I got the following error:

#!shell

Checking out svn://mysvndomain.com/myapp/trunk (to revision 635#egg=myapp) to ./myapp
svn: Syntax error in revision argument '635#egg=myapp'

Contributor

vbabiy commented Mar 15, 2011

  • Changed content.

Original Comment By: plotnik
Contributor

vbabiy commented Mar 15, 2011

  • Changed content.

Original Comment By: plotnik
Contributor

vbabiy commented Mar 15, 2011

  • Changed content.

Original Comment By: plotnik
Contributor

vbabiy commented Mar 15, 2011

Hi, What pip version are you using? Because I didn't get problems with pip
0.7.2 (latest) and the dev version.

I tried to reproduce here with the development version and didn't get any
problem:

$ pip install -e

svn+http://svn.colorstudy.com/INITools/trunk@3472#egg=INITools

Obtaining INITools from

svn+http://svn.colorstudy.com/INITools/trunk@3472#egg=INITools

  Checking out http://svn.colorstudy.com/INITools/trunk (to revision 3472)

to /home/hugo/.virtualenvs/pip-hltbra-ready-to-merge/src/initools

  Running setup.py egg_info for package INITools

Installing collected packages: INITools

  Running setup.py develop for INITools

    Creating /home/hugo/.virtualenvs/pip-hltbra-ready-to-

merge/lib/python2.6/site-packages/INITools.egg-link (link to .)

    Adding INITools 0.2.1dev-r3472 to easy-install.pth file


    Installed /home/hugo/.virtualenvs/pip-hltbra-ready-to-

merge/src/initools

Successfully installed INITools

Cleaning up...

$ pip --version

pip 0.7.1 from /home/hugo/.virtualenvs/pip-hltbra-ready-to-

merge/lib/python2.6/site-packages/pip-0.7.1-py2.6.egg (python 2.6)

And tried with the latest release (pip 0.7.2) and didn't get any problem too:

$ pip install -e

svn+http://svn.colorstudy.com/INITools/trunk@3472#egg=INITools

Obtaining INITools from

svn+http://svn.colorstudy.com/INITools/trunk@3472#egg=INITools

  Checking out http://svn.colorstudy.com/INITools/trunk (to revision 3472)

to /home/hugo/.virtualenvs/testenv/src/initools

  Running setup.py egg_info for package INITools

Installing collected packages: INITools

  Running setup.py develop for INITools

    Creating /home/hugo/.virtualenvs/testenv/lib/python2.6/site-

packages/INITools.egg-link (link to .)

    Adding INITools 0.2.1dev-r3472 to easy-install.pth file


    Installed /home/hugo/.virtualenvs/testenv/src/initools

Successfully installed INITools

Cleaning up...

$ pip --version

pip 0.7.2 from /home/hugo/.virtualenvs/testenv/lib/python2.6/site-packages

(python 2.6)


Original Comment By: Hugo Lopes Tavares
Contributor

vbabiy commented Mar 15, 2011

plotnik clearly mentioned, that it occurs when you use
svn:mysvndomain.com/myapp/trunk not
svn+http://mysvndomain.com/myapp/trunk


Original Comment By: http://mysvndomain.com/myapp/trunk
Contributor

vbabiy commented Mar 15, 2011

hi,

same situation here. the problem is in urlsplit. the function doesn't split an
url like svn:_ as it firgures._

using pip 0.8.1 with python 2.6.6

thomas


Original Comment By: Anonymous
Contributor

vbabiy commented Mar 15, 2011

  • Changed status from invalid to open.

Original Comment By: Hugo Lopes Tavares
Member

hltbra commented Aug 2, 2011

Reading some old bitbucket notifications I've found the user @centralniak has sent a patch to us, and I think nobody took a look at that time: https://bitbucket.org/centralniak/pip/changeset/0271386efe0b

Saved as gist: https://gist.github.com/1120276

Feel free to contact me if it's broken. I'll do my best to come up with working version. :)

@ghost ghost assigned hltbra Aug 2, 2011

Member

hltbra commented Aug 2, 2011

I tried to reproduce right now creating a svn local repositoy in my computer, following this tutorial and then:

$ pip install -e svn://localhost/Users/hugo/SVN_REPOS/splinter@2#egg=splinter

and

$ pip install svn+svn://localhost/Users/hugo/SVN_REPOS/splinter@2#egg=splinter

Tried the same recipe using pip 1.0.1 and 1.0.2 (develop branch), and both worked with no problems.

PS.: I only had problems with:

$ pip install svn://localhost/Users/hugo/SVN_REPOS/splinter@2#egg=splinter
Downloading/unpacking splinter from svn://localhost/Users/hugo/SVN_REPOS/splinter@2
Exception:
Traceback (most recent call last):
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/basecommand.py", line 126, in main
    self.run(options, args)
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/commands/install.py", line 223, in run
    requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/req.py", line 961, in prepare_files
    self.unpack_url(url, location, self.is_download)
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/req.py", line 1073, in unpack_url
    return unpack_vcs_link(link, location, only_download)
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/download.py", line 293, in unpack_vcs_link
    vcs_backend.unpack(location)
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/vcs/__init__.py", line 225, in unpack
    self.obtain(location)
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/vcs/subversion.py", line 79, in obtain
    url, rev = self.get_url_rev()
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/vcs/subversion.py", line 156, in get_url_rev
    url, rev = super(Subversion, self).get_url_rev()
  File "/private/tmp/ENV_FOOBAR/lib/python2.6/site-packages/pip-1.0.1-py2.6.egg/pip/vcs/__init__.py", line 117, in get_url_rev
    url = self.url.split('+', 1)[1]
IndexError: list index out of range

Storing complete log in /Users/hugo/.pip/pip.log
Member

hltbra commented Aug 2, 2011

Hey, @centralniak, can you try to reproduce it?

Contributor

xavfernandez commented Mar 10, 2015

pip now seems to explicitly expect a '+' in the url (cf https://github.com/pypa/pip/blob/develop/pip/vcs/__init__.py#L147)
and raises an helpful AssertionError:

AssertionError: Sorry, 'svn://svn.colorstudy.com/INITools/trunk@3472#egg=INITools' is a malformed VCS url. The format is <vcs>+<protocol>://<url>, e.g. svn+http://myrepo/svn/MyApp#egg=MyApp

pip install svn+http://svn.colorstudy.com/INITools/trunk@3472#egg=INITools still works fine, so this can probably be closed.

@dstufft dstufft closed this Mar 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment