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

pip misses support for subversion 1.7 metadata handling #369

Closed
rsommer opened this Issue Oct 14, 2011 · 11 comments

Comments

Projects
None yet
7 participants
@rsommer

rsommer commented Oct 14, 2011

Hi,

metadata handling changed completely in subversion 1.7. The result is that calling get_url from pip/vcs/subversion.py returns None as URL. It just logs that the entry format (now 12) is unknown and continues, leading to

File "/usr/lib/python2.6/urllib.py", line 1184, in unquote
res = s.split('%')
AttributeError: 'NoneType' object has no attribute 'split'

I hacked a quick fix for me, maybe thats a start:

rsommer/pip@f50222a8de2e4d5e27224910255c3fb2ca3e168f

Thank you ...

@GeneKong

This comment has been minimized.

Show comment
Hide comment
@GeneKong

GeneKong Nov 1, 2011

Except hack the code manually and change svn version, is there any other suggestions?

GeneKong commented Nov 1, 2011

Except hack the code manually and change svn version, is there any other suggestions?

@carljm

This comment has been minimized.

Show comment
Hide comment
@carljm

carljm Nov 1, 2011

Contributor

Is it time to switch the svn backend to just use the CLI like every other VCS backend? Are there major blockers to doing that?

Contributor

carljm commented Nov 1, 2011

Is it time to switch the svn backend to just use the CLI like every other VCS backend? Are there major blockers to doing that?

@jezdez

This comment has been minimized.

Show comment
Hide comment
@jezdez

jezdez Nov 4, 2011

Contributor

+1 on using the CLI instead.

Contributor

jezdez commented Nov 4, 2011

+1 on using the CLI instead.

@GeneKong

This comment has been minimized.

Show comment
Hide comment
@GeneKong

GeneKong Nov 5, 2011

why not update some code to fix this?

GeneKong commented Nov 5, 2011

why not update some code to fix this?

@victorhooi

This comment has been minimized.

Show comment
Hide comment
@victorhooi

victorhooi Dec 15, 2011

Hi,

Hmm, I also seem to be hitting this issue when using to install django-trunk from Subversion.

Is there any update on this issue, or when it might be fixed?

Cheers,
Victor

victorhooi commented Dec 15, 2011

Hi,

Hmm, I also seem to be hitting this issue when using to install django-trunk from Subversion.

Is there any update on this issue, or when it might be fixed?

Cheers,
Victor

@carljm

This comment has been minimized.

Show comment
Hide comment
@carljm

carljm Dec 15, 2011

Contributor

This is open source; it'll get fixed when somebody who uses svn is motivated to fix it. Pull requests welcome :-)

Contributor

carljm commented Dec 15, 2011

This is open source; it'll get fixed when somebody who uses svn is motivated to fix it. Pull requests welcome :-)

@cra

This comment has been minimized.

Show comment
Hide comment
@cra

cra Dec 22, 2011

fix works for me too, just save it and apply with patch command

cra commented Dec 22, 2011

fix works for me too, just save it and apply with patch command

@victorhooi

This comment has been minimized.

Show comment
Hide comment
@victorhooi

victorhooi Jan 9, 2012

heya,

Is that patch likely to hit mainline?

Or are we just switching over to command-line svn?

Cheers,
Victor

victorhooi commented Jan 9, 2012

heya,

Is that patch likely to hit mainline?

Or are we just switching over to command-line svn?

Cheers,
Victor

@jezdez

This comment has been minimized.

Show comment
Hide comment
@jezdez

jezdez Jan 9, 2012

Contributor

The linked work around is not likely to land in pip, a command-line option would be preferred (like we do with the other VCS).

Contributor

jezdez commented Jan 9, 2012

The linked work around is not likely to land in pip, a command-line option would be preferred (like we do with the other VCS).

@hongqn

This comment has been minimized.

Show comment
Hide comment
@hongqn

hongqn Jan 18, 2012

Contributor

#434 has a command-line option. Please take a look.

Contributor

hongqn commented Jan 18, 2012

#434 has a command-line option. Please take a look.

@carljm

This comment has been minimized.

Show comment
Hide comment
@carljm

carljm Jan 18, 2012

Contributor

Fixed - thanks @hongqn!

Note that setuptools remains incompatible with Subversion 1.7; you must use Distribute instead in order to use pip with Subversion 1.7.

Contributor

carljm commented Jan 18, 2012

Fixed - thanks @hongqn!

Note that setuptools remains incompatible with Subversion 1.7; you must use Distribute instead in order to use pip with Subversion 1.7.

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