[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Ben Reser <ben_at_reser.org>
Date: 2004-11-05 16:21:18 CET

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
Received on Sat Nov 6 02:16:30 2004

This is an archived mail posted to the Subversion Dev mailing list.