Re: [BUG] svn ls file:///

On Fri, Nov 05, 2004 at 03:18:02PM +0100, Uwe Zeisberger wrote:
> Hello,
>
> (I use svn 1.1.0rc2 while writing this mail, but observed similar
> behaviour with HEAD a few days ago.)
>
> if I issue the command
>
> svn log file:///
>
> (in a wc or not) I get:
>
> Assertion failed: is_canonical (path, len), file [..]/subversion/libsvn_subr/path.c, line 376
> Abort
>
> I think file:/// is a valid url standing for '/'. (Please correct me, if
> I'm wrong.)
>
> If file:/// is a valid (and canonical) path, is_canonical
> (defined in path.c, line 95+) returns a wrong result (namely false).
>
> If not, svn should not abort with a failed assertion.

Technically yes, file:/// is valid. That said why would anyone put a
repo directly in the root file system?

This might be as simple as fixing is_canonical or it might require
further changes to our path functions because we assume that a path
never ends in /. There are a lot of special casing for just '/' that
might need to be extended to 'file:///'.

-- 
Ben Reser <ben@reser.org>
http://ben.reser.org
"Conscience is the inner voice which warns us somebody may be looking."
- H.L. Mencken
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org