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

Re: [PATCH] Fix a regression from 1.4 in trunk and branch

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Wed, 30 Jan 2008 15:57:48 -0500

Augie Fackler wrote:
>
> On Jan 30, 2008, at 2:41 PM, C. Michael Pilato wrote:
>
>> Of interest (to me, at least) is how we managed to get ahold of the
>> non-canonicalized path that we currently write to the entries file. I
>> mean, the first thing our command-line client does with the paths
>> provided to subcommands is canonicalize them, I thought. What gives?
>
> The path is the computed repository root - I haven't figured out in the
> code where that happens yet, but it looks to my eye like the repository
> root is being computed with the hostname and the path separate, so that
> when the path is just / it isn't stored in a canonical form, but if it
> was at /foo/ it would get stored as /foo
>
> I was seeing the bug when I was checking out
> http://svn.perian.org/trunk, which isn't the repo root, so svn must be
> computing the repo root of http://svn.perian.org/ on its own somewhere...

Oh, yes, right -- forgot that you were probably checking out something
deeper than the root. Yes, each RA implementation does this sort of root
calculation. Your solution will work, but still leaves us with
non-canonicalized paths being passed around the libraries in memory during
the checkout. I would suggest making the four RA layers do this
canonicalization at the moment they discern the repository root.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2008-01-30 21:58:01 CET

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.