[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: Brian W. Fitzpatrick <fitz_at_red-bean.com>
Date: 2006-01-13 21:23:23 CET

On Jan 13, 2006, at 12:06 PM, Greg Hudson wrote:

> On Fri, 2006-01-13 at 16:37 +0000, Julian Foad wrote:
>> It seems to me fairly certain that we want this ability to reverse
>> the
>> direction of a change.
>
> For the record, I'm -0 on it. It feels like unnecessary syntactic
> fluff. The -c option itself makes sense because it's common to
> want to
> see the change made in a given revision, but I don't think it's common
> to want to see the inverse of that change, and you can get the inverse
> with svn diff -r N:N-1.
>
> Rephrased: it's good to have shorthand for talking about revisions as
> changes, but I don't see the point in having shorthand for talking
> about
> revisions as inverse-changes.
>
> (Yeah, I know, "svn merge -c N --reverse" to roll back a change. I
> don't think rollbacks are all that common.)

In my experience, they're not all that common just because it's
somewhat of a PITA for people (read: people who don't write version
control systems) to remember the syntax and get it right.

I think that if we made it easier ('-c REV' is the first step in that
direction, '-c NEGATIVE_REV' is another huge step in that direction),
then we'd see people rolling back very often.

BTW, one use case for rolling back is the desire to commit something
that I wrote but don't want to use in my code *at this moment*: I
commit the patch with an appropriate log message, then I roll it back
right away.

-Fitz

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jan 13 21:42:06 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.