Karl Fogel wrote:
> Vlad Georgescu <vgeorgescu_at_gmail.com> writes:
>> With this patch, you can relocate a WC with just a plain switch,
>> without --relocate. And you can relocate to a new repo root _and_
>> switch the FS path in one step.
>
> In other words, this makes the '--relocate' option obsolete?
>
> After we introduced the '--relocate' option, we sometimes wondered
> whether we'd made a mistake: maybe it should have been a separate
> command, because what 'switch --relocate' does is conceptually different
> from what a plain 'switch' does.
>
> However, I think it might be the case that having '--relocate' as an
> option is the worst of all worlds. Either it should be a separate
> command, or plain 'switch' should just DTRT when handed a new root. The
> latter is what your patch does, right? (And I see that it does protect
> against switching to a different repos UUID, as it should.)
>
> I'm close to +1 on this, but would like others' thoughts...
I had the following gut reactions to this post, in short order:
. o O ( Gack! Someone is disturbing the delicate balance of the
switch/relocated code!? )
. o O ( Yes, someone is. And what a *nice* disturbance it is... )
What is lost by this change, though, is the ability to relocate without
updating (which is a very valid thing to wish to do). Would 'svn switch
NEW-URL -rBASE' be a way to preserve that functionality?
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2008-08-27 22:10:09 CEST