[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: Paul Burba <ptburba_at_gmail.com>
Date: Mon, 4 Oct 2010 16:08:07 -0400

On Mon, Oct 4, 2010 at 12:43 PM, Peter Samuelson <peter_at_p12n.org> wrote:

>> On Sat, Oct 2, 2010 at 9:34 PM, Peter Samuelson <peter_at_p12n.org> wrote:
>> > 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.
>>
>> No objections to supporting both.  Really all we are asking then is
>> that users remember that -c'X-Y' is inclusive of X and '-cX:Y' is
>> exclusive.  Of course we may rapidly be approaching the point where -r
>> and -c both support ':' and '-' and effectively become the same
>> option.
>
> Huh.  I didn't interpret ":" to always imply an exclusive range.  I
> figured -c vs. -r was the overriding factor here.  In Lieven's XFail
> test, -c2:5 is inclusive, but that doesn't tell us much, it's a 'log'
> test and 'log' is always inclusive.

Yup, Julian busted me on that too. Just ignore me, it's Monday :-\

> So if we support '-c2-5', we can say that's always inclusive, but if
> we (also) support '-c2:5' there's the question of whether it means the
> same as '-c2-5' or the same as '-r2:5'.
>
> I'm not wedded to -c2:5 anyway, as my use case is pasting svn:mergeinfo
> onto the command line, but if we're gonna support it we should know
> what it's supposed to mean.  (:

We can certainly proceed with what you have so far (i.e. -cX-Y
support) and if anybody wants to take up the -cX:Y banner, let them.

Paul
Received on 2010-10-04 22:08:45 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.