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

Re: RFC: replace svn_path functions with svn_dirent and svn_uri.

From: Lieven Govaerts <svnlgo_at_mobsol.be>
Date: Sun, 21 Sep 2008 11:14:13 +0200

alan.wood_at_clear.net.nz wrote:
> Hi Lieven,
> Hopefully on the way through svn_path_is_ancestor() can be made to work
> with a little knowledge of the case sensitivity of the various parts of
> the url and path names. I came across this earlier with a segmentation
> fault during an svn copy operation. see:
> http://svn.haxx.se/dev/archive-2008-08/0667.shtml
> Just to be a bit nicer to the user when typing urls, if c: - C: and
> \\server\share - \\SERVER\SHARE could be treated as identical because they
> always are. The file names can not be in general as people can use Samba,
> nfs on Windows etc.

I don't think svn_path_is_ancestor is at fault here. The svn_path
functions are supposed to work with canonical paths. It's up to the
callers to make sure the directory entries, uris and urls are canonical.

Now canonical means something different on Windows than on POSIX
platforms, a difference we don't take into account right now.

> This also applies to server names, which are case insensitive, but are
> not in
> subversion ( svn copy svn://svn.Collab.net/repos/path/filename ./filename
> gives an error on a working copy checked out from svn.collab.net )

This is a known issue:

The best thing we can do is to lowercase drive letters and hostnames in
paths for internal use. In fact, I've written a patch (attached to issue
#2475) that does just that for the hostnames but it never got committed.
 I'll see if I can get it updated and committed.


To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-21 11:14:27 CEST

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.