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

Re: svn commit: r10133 - in trunk/subversion: include libsvn_subr tests/clients/cmdline tests/libsvn_subr

From: Greg Hudson <ghudson_at_MIT.EDU>
Date: 2004-07-04 17:34:12 CEST

On Sun, 2004-07-04 at 11:21, Josh Pieper wrote:
> Yes, I considered that case; this also stopped generating an error
> when you try "invalid_path/../real_path." Apache does this when
> resolving URLs to filesystem paths, so at least one other application
> handles paths the same way. Of course that doesn't mean it is a valid
> method, but collapsing the ".." makes the copy code more robust to
> differing path specifications in the common cases.

Apache can dictate that directory symlinks are disallowed in document
roots. I don't think it's fair for us to dictate that about all paths
on the command line, only about the area within working copies (and, of
course, the virtual area within repositories).

I followed the pre-commit discussion about this change, but the
discussion of "differing path specifications" never seemed to become
concrete. Where is it necessary to ensure that canon(p1) == canon(p2)
if p1 and p2 are two different paths expressing the same object?

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Jul 4 17:36:11 2004

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