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

Re: [PATCH] fix for issue #2556: support working copies on the root of a (virtual) drive

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2006-08-19 02:15:39 CEST

"Lieven Govaerts" <lgo@mobsol.be> writes:

> Prepare fix for issue #2556: abstract root folder checks in
> svn_path_is_root.
> Add support for 'X:/' as a root folder on Windows.

Grepping for \[0\].*\'/\' shows that there might be a few places where
we assume that path[0]=='/' is special; I don't know whether your
patch invalidates those assumptions. If path is a Subversion FS path
then it's OK but if path is a WC path then it could be a problem. For

- libsvn_client/commit.c:1312 seems to be checking for "/", should it
  be checking for "x:/" as well?

- libsvn_wc/props.c:2110 seems to be checking for "/...", should it be
  checking for "x:/..." as well?

- libsvn_client/diff.c:401 and 408 seem to be checking for "/...",
  should they be checking for "x:/..." as well?

I don't know if these are bugs, and if they are I suspect they are
unrelated to your patch, but someone who groks this Windows stuff
probably ought to take a look :)

Philip Martin
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Aug 19 02:16:10 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.