Error getting externals with git-svn-clone-external

[ghost]

trafficstars

The script seems to be parsing malformed URLs when I try on our repository.

$ ./git-svn-clone-externals lib/vendor/http://svn.dwoo.org/trunk/lib -> dwoo git svn clone dwoo lib/vendor/http://svn.dwoo.org/trunk/lib Initialized empty Git repository in /home/drak/www/trunk/.git_externals/lib/vendor/http:/svn.dwoo.org/trunk/lib/.git/ Bad URL passed to RA layer: Illegal repository URL 'dwoo' at /usr/lib/git-core/git-svn line 1544

lib/vendor/http://svn.phpdoctrine.org/tags/1.2.1/lib -> Doctrine git svn clone Doctrine lib/vendor/http://svn.phpdoctrine.org/tags/1.2.1/lib Initialized empty Git repository in /home/drak/www/trunk/.git_externals/lib/vendor/http:/svn.phpdoctrine.org/tags/1.2.1/lib/.git/ Bad URL passed to RA layer: Illegal repository URL 'Doctrine' at /usr/lib/git-core/git-svn line 1544

[ghost]

After a little debugging is seems that the following code is producing results that result in invalid params:

git svn show-externals|grep -vE '#|^$' /lib/vendor/http://svn.dwoo.org/trunk/lib dwoo /lib/vendor/http://svn.phpdoctrine.org/tags/1.2.1/lib Doctrine /locale/

[alexspurling]

This is a bug with git-svn show-externals when parsing an svn:externals property which has the URL first, followed by the external link name. Eg:

https://blah.com/svn/myexternal/url/ ext

This could possibly be handled by the clone externals script but would be more appropriate to submit a patch to git. I'm planning on doing this when I figure out where git's git repository is kept and I can develop a fix :)

[tleish]

I've run into this same error. Is there a fix or workaround for this?

[bimargulies]

Add me to the list of interested parties.

[crazy2be]

Also running into this with paths like

/Tools/^/../repobin/NUnit/2.6.2@450 NUnit

And interested if anyone has a fix :).

[adamscybot]

The problem is where the svn:external property is using the new way (post SVN 1.5) of defining externals. It is documented here. You can swap the definition of remote_url and local_directory in the script and it works.