[PATCH] patchbomb: don't prompt for cc when cc is set without value

David Soria Parra sn_ at gmx.net
Fri Jan 22 09:49:34 CST 2010


> +        # do not prompt for Cc when it is configured explicitly empty
> +        if opt == 'cc' and ui.config('email', opt) == '':
> +            prpt = None
Shouldn't we also check for ui.config('patchbomb', opt) == '', too?
Apart from that I think the patch looks good.

 if opt == 'cc' and (ui.config('email', opt) == '' or
ui.config('patchbomb', opt) == '')



More information about the Mercurial-devel mailing list