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

Re: [PATCH] back out changes with --change -N

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: 2006-02-16 22:51:27 CET

Bart Robinson wrote:
> [[[
> Extend -c/--change option in diff and merge to accept negative
> revisions as meaning the reverse of a change, making it possible to
> back out changes with -c rather than -r. E.g., -c -1234 is the same
> as -r1234:1233.

It seems likely that this will be accepted in principle, so I'm reviewing.

It all looks absolutely fine, log message and code, except that it would be
nice if the new test function were not an almost exact copy of an existing one
but somehow shared much of its implementation. That's probably not strictly
necessary, though.

(If you try to re-apply this patch to the latest trunk code, watch out: we've
just removed spaces after function names throughout the code, so you'll need to
adjust the patch first. )

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Feb 16 22:58:57 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.