From: Michael Witten <mfwitten@mit.edu>
Date: Sun, 28 Sep 2008 19:04:30 GMT
Subject: Re: [PATCH 1/8] Docs: send-email's usage text and man page mention same options
Message-ID: <55747712-8E21-45F0-B2D2-4B4856568592@mit.edu>
In-Reply-To: <20080928050828.GZ23137@genesis.frugalware.org>


On 28 Sep 2008, at 12:08 AM, Miklos Vajna wrote:

> On Sat, Sep 27, 2008 at 08:09:49PM -0500, Michael Witten <mfwitten@MIT.EDU 
> > wrote:
>> Specifically, boolean options are now listed in the form
>>
>>    --[no-]option
>>
>> and both forms of documentation now consistently use
>>
>>    --[no-]signed-off-by-cc
>
> I don't think documenting --no-foo in the perl script itself is a good
> idea. See c3170a8 (git-merge: exclude unnecessary options from
> OPTIONS_SPEC, 2008-05-12) which removes --no-foo from git-merge as  
> well.
>
> Of course the man page part makes sense.

I like that it's in the help usage text, because it makes it clear how
it should be used.

Besides, I basically removed the same kind of cruft that was removed
with c3170a8, but I still managed to include the same information.