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

Re: [PATCH] --reverse option to invert -c/--change

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: 2006-01-13 17:37:03 CET

Bart Robinson wrote:
> Hi,
> Here is a patch to help flesh out the utility of the -c/--change
> option.
>
> Some background in this thread:
> http://svn.haxx.se/dev/archive-2005-10/1324.shtml
> (should I put that in the commit message?)
>
> E.g.,
> svn merge -c 100 --reverse URL
> same as
> svn merge -r100:99 URL

It seems to me fairly certain that we want this ability to reverse the
direction of a change.

"--reverse" is perhaps too generic a name for the option: we might well want to
reverse other things than the direction of a change, and it seems unlikely that
it will make sense to share a common option name for reversing completely
different things. We should consider a more specific option name like
"--reverse-change".

If the switch will only apply to "-c" then we could make it an alternative to
"-c" rather than an addition to it: e.g. "--reverse-change REVNUM" (possibly
with a short name "-C REVNUM").

Do we want the reversal switch to apply to "-r" as well? That is not necessary
in the basic cases (such as "-r X:Y") but we might decide it is convenient and
consistent. Indeed it might enable some desirable diffs that are not possible
at present: like the reverse of a plain diff (WC against BASE -> BASE against
WC). (Am I right about that? If so, a better solution to that particular
problem might be to introduce a "WORKING" or "WC" revision-keyword.)

Thoughts?

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jan 13 18:08:55 2006

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.