[PATCH] -r 1-100 not -r1:100 please...

[John Arbash Meinel]

Michael Ellerman wrote:

>On Fri, 27 May 2005 15:04, John Arbash Meinel wrote:
>
>
>>Rusty Russell wrote:
>>
>>
>>>OK, you knew this was coming 8)
>>>
>>>
>>Check out my latest patch for revision handling. There is a very
>>specific reason to use ':' instead of '-'. Negative numbers.
>>
>>...
>>
>>It isn't the end-all be-all, but I do think negative revisions numbers
>>are a nice thing.
>>
>>
>
>Yeah I think negative revision numbers are kind of "neat", but I don't think
>you want to design your UI around them.
>
>By using - for ranges you free up : for seperating the name space prefix so as
>I see it, it's either:
>
>bzr diff -r 10:20
>and
>bzr diff -r {date:yesterday}
>
>OR
>
>bzr diff -r 10-20
>and
>bzr diff -r date:yesterday
>
>
>
The problem is when we support UUIDs, they have dashes in them, so you get:
bzr diff -r
uuid:aoue-aoeu-aoeu-aoeu-aouaoeuaoeu-uuid:aoeu-aeou-aoeu-aoeu-aouaouau

John
=:->

>I think the second option is better because a) 10-20 is a much more common
>syntax for specifying a range (eg. print something), and getting rid of the
>{} is nice.
>
>cheers
>
>
>
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 253 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050527/ecce9bae/attachment.pgp