Re: [PATCH 7/8] Docs: send-email: Added all config variables to man end
On 27 Sep 2008, at 10:18 PM, Jeff King wrote:
Show 7 quoted lines
> On Sat, Sep 27, 2008 at 08:09:55PM -0500, Michael Witten wrote:
>
>> Admittedly, I didn't take much pride in my work, but
>> I'm not trying to solve this problem.
>
> Remember that your commit message is supposed to be convincing us that
> your patch is worth applying. ;P
Well, I told the truth ;-)
> It seems like these are just mechanical "sendemail.foo is a default
> for
> --foo" additions.
> Maybe it is better to simply make a note of that (or
> mention it in the --foo section).
Then it would be better to do away with all but
sendemail.aliasesfile
sendemail.aliasfile
which sounds OK with me.