Re: BUG (?) in svn cp

Philip Martin <philip@codematters.co.uk> writes:

> Daniele Nicolodi <daniele@grinta.net> writes:
>
> > $ export R="http://localhost:8080/svn/code/"
> > $ svn cp ${R}/trunk ${R}/tags/0.6.1
> > svn: subversion/libsvn_subr/path.c:156: svn_path_join: Assertion
> > `is_canonical (base, blen)' failed.
> > Aborted
>
> Subversion's path library usually handles that sort of thing, but it
> looks like the path driver is bypassing the library and attempting to
> do it's own path processing. In svn_delta_path_driver I see things
> like strchr for '/' and ++ on char* pointers, I suspect these don't
> work as well as the path library.

The path driver *should* be able to rely on having input that has been
canonicalized -- meaning '//' has been converted to just '/'. If
you're going to raise an issue, perhaps it should be titled:

   svn_path_canonicalize() doesn't.

:-)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Mar 3 06:39:38 2004