[PATCH 0 of 2] patchbomb: only 1 diffstat confirmation prompt, add --confirm option (2nd try)

Christian Ebert blacktrash at gmx.net
Fri Sep 10 09:12:36 CDT 2010


* Martin Geisler on Friday, September 10, 2010 at 16:05:24 +0200
> Christian Ebert <blacktrash at gmx.net> writes:
>> Another attack on patchbomb's --diffstat prompts per patch.
>> 
>> The first patch changes the behaviour of "hg email --diffstat":
>> Instead of the confirmation prompts for each message/patch which also
>> show the patch description, there is now only one prompt showing the
>> series' message details and diffstats.
>> 
>> The second patch adds a --confirm option which presents the same
>> prompt as --diffstat, but does not write the diffstats in the message
>> bodies.
>> 
>> In contrast to the first submission, empty header fields are left out
>> in the confirmation prompt.
> 
> I like this, so I decide we go with this version :)

Ah, you split out the variable renaming, good idea.

>> thx to Mads & Martin for patience & feedback.
> 
> Well, thank you for not walking away from these patches.

I'm a stubborn beast.

> They are pushed as aebb39d45500 and 5bfa45651cf6.

thx

c
-- 
All of old. Nothing else ever. Ever tried. Ever failed.
No matter. Try again. Fail again. Fail better.
--Samuel Beckett, Worstward Ho


More information about the Mercurial-devel mailing list