kfogel@collab.net wrote:
>Kevin Pilch-Bisson <kevin@pilch-bisson.net> writes:
>
>>There is another slight hurdle though. The option parsing code in main.c
>>currently won't support the same option twice, so -r 4 -r 9 won't work with=
>>out
>>at least a little re-write. I know our code won't handle it, I'm not sure
>>the APR functions used for options.
>>
>
>We can either rewrite it, or allow colon-separated values, i.e.,
>
> $ svn diff -r 7 /* diff from working to 7 */
> $ svn diff -r :7 /* diff from working to 7 */
> $ svn diff -r 7: /* diff from 7 to working */
> $ svn diff -r BASE:7 /* diff from base to 7 */
> $ svn diff -r 3:7 /* diff from 3 to 7 */
> $ svn diff -r 7:3 /* diff from 7 to 3 */
>
>Formally, when the colon is omitted, it is assumed to be on the left
>of the revision, and when a revision is omitted, it is assumed to be
>"WORKING". As a special case, when no -r option is given, diff from
>base to working.
>
+1, Except for the switching the default when the colon is ommitted
(i.e., it should be assumed to be on the right of the revision).
I find this cleaner than having two -r options:
svn -r 317 diff foo -r 713. :-( horrible
svn -r 317:713 diff foo :-| not nice, but acceptable
svn diff -r 317:713 foo :-) nicest
svn diff foo -r 317:713 :-) almost nicest
--
Brane �ibej <brane_at_xbc.nu> http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:44 2006