On Thu, Aug 28, 2008 at 10:42 PM, Greg Hudson <ghudson_at_mit.edu> wrote:
> On Thu, 2008-08-28 at 22:21 +0300, Vlad Georgescu wrote:
>> > I don't understand how that implies to the new world (after your patch)
>> > where "switch" can also do a relocate. In the new world, it would seem
>> > like relocate could definitely be a flag for svn_client_switch. The
>> > flag just disables the update and portion of the switch (and, as a
>> > consequence, means the repository-relative part of the URL must be the
>> > same for the source and destination).
>
>> Are you suggesting we implement "svn sw --relocate" on top of
>> svn_client_switch() + my patch + the new flag? It's possible, but what's the
>> point, considering we already have svn_client_relocate() for that?
>
> The implementation should not drive the interface. It may be not worth
> the effort to reimplement relocate in terms of switch, but if relocate
> is conceptually just "switch with no update" in the new world, then
> there is no good argument for giving it a separate subcommand.
>
"switch with no update" is somewhat of a contradiction in terms, in my
mind. It would be a PITA trying to explain people that --no-update
only makes sense when changing the repository, and only when the paths
within the repository stay the same.
Besides, even with --no-update, you still don't cover 100% of the
functionality of --relocate. With relocate, you can do something like:
# svn sw --relocate http https
You wouldn't be able to do that with switch (as easily).
My intention is to remove a limitation of 'svn switch' which would
cover 95% of the use cases for relocate, with a simpler UI. "svn
switch --relocate" will still be there to cover the remaining 5%
(though I'd prefer to move it to a different subcommand to make it
less confusing).
--
Vlad
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-29 14:27:20 CEST