Comment 0 for bug 56357

Revision history for this message
David Allouche (ddaa) wrote : Trailing slash in subversion url causes import to fail

Bad formatting of svnrepository sometimes causes cscvs to fail with a mysterious assertion.

(19:18:50) ddaa: python2.4: subversion/libsvn_subr/path.c:343: svn_path_remove_component: Assertion `is_canonical (path->data, path->len)' failed. process killed by signal 6
(19:19:40) ddaa: e.g. with that repo http://reductivelabs.com/svn/puppet/trunk/
(19:23:35) jelmer: ah, that's caused by a trailing slash
(19:25:10) jelmer: the SWIG stuff should take care of that, but nobody's gotten around to fixing it yet
(19:32:05) ddaa: there's a bunch of imports that have a trailing slash like that and still work...
(19:32:28) ddaa: Like that one
(19:32:36) ddaa: svn://svn.debian.org/adduser/trunk/
(19:38:08) ddaa: but still, removing the trailing slash does fix at least one import that failed
(19:39:46) jelmer: pretty much all the times I've seen that particular assertion being triggered, it was due to invalid paths (double slashes, trailing slashes, etc)