plans for 0.0.6 release

[John A Meinel]

Michael Ellerman wrote:

>On Thu, 2 Jun 2005 00:18, John A Meinel wrote:
>
>
>>Right now I'm favoring -r date:blah..20, or -r 10..20
>>I saw your 10 to 20 patch, the problem is it doesn't really work right.
>>Having something take a variable number of arguments is kind of error
>>prone. For instance, what if there is a file in the directory named 'to'
>>and I type
>>bzr log -r 10 to
>>I'm trying to say give me the revision 10 log for the file to if it was
>>modified. Actually, more likely would be:
>>bzr log -r 10 to to
>>To request all logs from 10 onward that modify "to".
>>
>>
>
>The to syntax will only take 1 or 3 arguments, either "-r x" or "-r x to y".
>If you want to specify an open ended range you can use either "start" or
>"now" (maybe "end").
>
>And you're right, for all those millions of people out there who have files
>called "to" it might look a bit funny when they type "bzr log -r 10 to". But
>it's well defined what it means.
>
>Anyway we can argue about it more when I get it written the way I want. Which
>might be next week at this rate.
>
>cheers
>
>
But how do you know if there are 3 arguments to give -r or just 1? What
if I was looking at more than one file and typed:

bzr diff -r 10 other next
How does -r detect that it should consume these or not?
bzr diff -r 10 to next
Should also be valid either way. Now you can say, in that situation just
use:
bzr diff -r 10 next to
But the dynamic nature of argument parsing is actually what is bothering
me. Consistency is pretty useful. Adding a grammar so that you can parse
arguments is not the most obvious way to go.
It also requires quite a few changes to the regular argument parser to
support an option that can take a variable number of arguments (actually
anything other than 0/1). So it is a rather invasive change.

John
=:->

-------------- 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/20050601/2ca1d9f3/attachment.pgp