Re: [PATCH 1/2] List send-email config options in config.txt.
- From
Yann Dirson <ydirson@altern.org>
- Date
- Jul 16, 2009, 21:20 UTC
- Message-ID
- <20090716212044.GC5762@nan92-1-81-57-214-146.fbx.proxad.net>
- In-Reply-To
- <7v3a9ae1s6.fsf@alter.siamese.dyndns.org>
On Sun, Jul 05, 2009 at 11:58:01PM -0700, Junio C Hamano wrote:
Show 17 quoted lines
> > +sendemail.aliasesfile:: > > + To avoid typing long email addresses, point this to one or more > > + email aliases files. You must also supply 'sendemail.aliasfiletype'. > > + > > +sendemail.aliasfiletype:: > > + Format of the file(s) specified in sendemail.aliasesfile. Must be > > + one of 'mutt', 'mailrc', 'pine', 'elm', or 'gnus'. > > + > > +sendemail.bcc:: > > +sendemail.cc:: > > +sendemail.cccmd:: > > +sendemail.chainreplyto:: > > + > > At least, we would want to have some description for these. Something > along the lines of "Specifies values used for the corresponding command > line options when they are not given." should suffice.
Sure - that's part of why this series reads WIP :)
I'm also not entirely satisfied with duplicating the existing descriptions from the send-email manpage, like I have done for those with a desctiption. We could just mention all the flags with a "See linkgit:git-send-email[1] for details.", although it looks a bit inconsistent with the common usage, before going into a massive migration of these parameters into config.txt.
Would that be seen as reasonable ?