[PATCH] Re: FEATURE REQ: Add option --dry-run to 'add'

Robert Collins robertc at robertcollins.net
Mon Sep 19 07:49:36 BST 2005


On Mon, 2005-09-19 at 12:20 +1000, Michael Ellerman wrote:
..
> No add_reporter_null() is replaced with add_action_add(), which does the add 
> but doesn't print anything.

Ah.

> > That said, I think that a better approach here is actually to make dry
> > run something like ...
> 
> Yeah, that's the sexy way to do it. I guess I chose what I saw as the path of 
> least resistance.

So I'm not sure right now whether to merge your patch, then do the sexy
thing and revert the patch at that point as overkill.. or to convince
you to do the sexy thing :0.

Rob

-- 
GPG key available at: <http://www.robertcollins.net/keys.txt>.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050919/088fcf41/attachment.pgp 


More information about the bazaar mailing list