From: Michael Witten Date: Sun, 28 Sep 2008 19:03:42 GMT Subject: Re: [PATCH 7/8] Docs: send-email: Added all config variables to man end Message-ID: In-Reply-To: <20080928031822.GB15696@sigill.intra.peff.net> On 27 Sep 2008, at 10:18 PM, Jeff King wrote: > 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. They are indeed! > 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.