Re: [PATCH] Fix #3390 Relative externals not updated during switch, for backport to 1.6.x

Stefan Sperling wrote:
> On Fri, Jan 22, 2010 at 01:04:13PM -0500, C. Michael Pilato wrote:
>> If this is all it takes to fix this issue, I'm going to cry while deleting
>> each and every file from the 'issue-3390-dev' branch one commit at a time.
>> Seriously? This is the whole fix?
>
> Well, try it and find out :)
> More review won't hurt.

I can't seem to find any problems with this. But man, I *know* I tried this
fix myself when I first started looking at issue #3390. Oh well, ignorance
(or forgetfulness ... or plain ol' stupidity ...) is bliss, I guess.

Oh, I know why I considered this approach sub-par now. Currently, 'svn
switch' doesn't update any external working copies unless the switch itself
causes the externals definitions that "own" those working copies to have a
syntactic change (the server sends a propdiff down the wire). I was trying
to preserve that behavior, though expanding the definition of "change" to
include changes in the semantic interpretation of a relative-url externals
definition. Daniel's patch (like my original pass at this issue) throws all
that to the wind and forces Subversion to attempt to update every externals
definition in the switched tree whether it or its interpretation changed at all.

FWIW, I find Daniel's approach the sane one (because I think of update as
merely a special-case of switch, and so expect them to behave similarly with
respect to handling externals).

I do think that there's a tiny typo in one of your test comments, Daniel:

> + # Okay. We now want to switch A to A_copy, which *should* cause
> + # A/D/ext to point to the URL for A_copy/D/ext.

Shouldn't that be "...to the URL for A_copy/D." ?

But aside from that little issue, +1 to commit. (And I'm happy to do so
myself once Daniel affirms my recommended tweak above.)

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