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

Re: [PATCH] -cM-N command line syntax

From: Gavin Beau Baumanis <gavinb_at_thespidernet.com>
Date: Sun, 3 Oct 2010 18:58:29 +1100

Strictly from the view of the non-developer and user of SVN;
I have only ever used the ":" for specifying a range.
But I do see the sanity in having the input match the output too.
If the ability to support both is indeed trivial - and if both cases are "expected" by users - then that would certainly seem to be the most worthwhile choice.

Gavin "Beau" Baumanis

On 03/10/2010, at 12:34 PM, Peter Samuelson wrote:

>
> [Peter Samuelson]
>> Is this, then, a worthwhile feature addition? I don't want to add
>> syntax that nobody else wants. In particular, this patch highlights
>> the existing inconsistency of ":" vs. "-" as range operators.
>
> Hmmm, in tests/cmdline/log_tests.py XFail(log_chanage_range), I see
> Lieven thought -c might someday take ranges with ":" instead of "-".
> But since the output uses "-", I'd argue the cut-and-paste utility of
> sticking with that outweighs the consistency of ":". Unless perhaps we
> should support both, which would be trivial.
>
> Peter
Received on 2010-10-03 09:59:15 CEST

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.