Re: [PATCH RFC3 03/13] Docs: send-email: Remove superfluous information in CONFIGURATION

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 13, 2009, 20:45 UTC
Message-ID: <7vljq4e10q.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <1239647037-15381-4-git-send-email-mfwitten@gmail.com>

I Michael Witten <mfwitten@gmail.com> writes:

Show 20 quoted lines

> n particular, sendemail.confirm was removed, because it's already
> described along with its corresponding option.
>
> Signed-off-by: Michael Witten <mfwitten@gmail.com>
> ---
>  Documentation/git-send-email.txt |    6 ------
>  1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
> index 7b87d6e..b58b433 100644
> --- a/Documentation/git-send-email.txt
> +++ b/Documentation/git-send-email.txt
> @@ -271,12 +271,6 @@ sendemail.multiedit::
>  	summary when '--compose' is used). If false, files will be edited one
>  	after the other, spawning a new editor each time.
>  
> -sendemail.confirm::
> -	Sets the default for whether to confirm before sending. Must be
> -	one of 'always', 'never', 'cc', 'compose', or 'auto'. See '--confirm'
> -	in the previous section for the meaning of these values.

I am not sure if this is a good idea. It may look "superfluous" for people who read the manual cover-to-cover. However, a new person whose mentor allowed her to peek into his configuration to learn tricks from may find sendemail.confirm and would try to find an entry in the manual. The change in this patch will make it inconvenient to go from variable to description.