[MERGE] remove '--verbose' from 'bzr bundle'

John Arbash Meinel john at arbash-meinel.com
Thu Mar 29 16:24:41 BST 2007


Aaron Bentley wrote:
> John Arbash Meinel wrote:
>> PS> We really might want to think about supporting an "Author:" field in
>> revision properties. Since this happens every so often that we take raw
>> patches from 3rd parties who don't want to bother with bundles, etc.
> 
> I agree.  Not only Author, but Submitter, since that can be different, too.
> 
> My only question is, how do we decide what's a revision *field* like
> message, and what's a revision *property*, like branch-nick?
> 
> Aaron

A possible dividing line is based on whether things are hints, or
whether we expect the rest of the code to handle them.

Author could be a hint, but it seems reasonable that annotate and log
always show them. And for Author, I would consider it to take precedence
over "Committer" for things like "bzr log --short" and "bzr annotate".

However, since we can easily special case:

 if author is None:
   author = committer

Then maybe it is a hint, and could be a revision property instead.

For 'submitter' are you think to use it for the PQM? (So that things
submitted by you get a different tag than ones submitted by me?)

That also seems like more of a hint, since I wouldn't really expect a
command line 'bzr annotate' or 'bzr log --short' to use it. ('bzr log
--long' should)

At least that is how I would look at it.

John
=:->



More information about the bazaar mailing list