All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] send-email: document the --smtp-debug option
       [not found] <87ehtiu5dg.fsf@smith.obbligato.org>
@ 2012-02-27 16:22 ` Zbigniew Jędrzejewski-Szmek
  2012-02-27 20:19   ` Junio C Hamano
  0 siblings, 1 reply; 2+ messages in thread
From: Zbigniew Jędrzejewski-Szmek @ 2012-02-27 16:22 UTC (permalink / raw)
  To: git, gitster; +Cc: greened, Zbigniew Jędrzejewski-Szmek

The option was already shown in -h output, so it should be documented
in the man page.

Signed-off-by: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Suggested-by: David A. Greene <greened@obbligato.org>
---
David Greene wrote:
> I don't think --smtp-debug is documented in the man pages.  Was that a
> deliberate decision or an oversight?

 Documentation/git-send-email.txt |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
index 327233c..3241170 100644
--- a/Documentation/git-send-email.txt
+++ b/Documentation/git-send-email.txt
@@ -198,6 +198,10 @@ must be used for each option.
 	if a username is not specified (with '--smtp-user' or 'sendemail.smtpuser'),
 	then authentication is not attempted.
 
+--smtp-debug=0|1::
+	Enable (1) or disable (0) debug output. If enabled, SMTP
+	commands and replies will be printed. Useful to debug TLS
+	connection and authentication problems.
 
 Automating
 ~~~~~~~~~~
-- 
1.7.9.2.378.g4d260

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] send-email: document the --smtp-debug option
  2012-02-27 16:22 ` [PATCH] send-email: document the --smtp-debug option Zbigniew Jędrzejewski-Szmek
@ 2012-02-27 20:19   ` Junio C Hamano
  0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2012-02-27 20:19 UTC (permalink / raw)
  To: Zbigniew Jędrzejewski-Szmek; +Cc: git, greened

Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> writes:

> The option was already shown in -h output, so it should be documented
> in the man page.
>
> Signed-off-by: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
> Suggested-by: David A. Greene <greened@obbligato.org>
> ---
> David Greene wrote:
>> I don't think --smtp-debug is documented in the man pages.  Was that a
>> deliberate decision or an oversight?

Probably halfway in between "Meh"; anybody who is willing to look into the
issue will notice when he opens the script and reads it for the first time
anyway, so why bother.

But now "A patch already exists, and it does not seem to have any funny
letters that may screw up asciidoc, so why bother rejecting" ;-)

Will apply; thanks.

>  Documentation/git-send-email.txt |    4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
> index 327233c..3241170 100644
> --- a/Documentation/git-send-email.txt
> +++ b/Documentation/git-send-email.txt
> @@ -198,6 +198,10 @@ must be used for each option.
>  	if a username is not specified (with '--smtp-user' or 'sendemail.smtpuser'),
>  	then authentication is not attempted.
>  
> +--smtp-debug=0|1::
> +	Enable (1) or disable (0) debug output. If enabled, SMTP
> +	commands and replies will be printed. Useful to debug TLS
> +	connection and authentication problems.
>  
>  Automating
>  ~~~~~~~~~~

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-02-27 20:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <87ehtiu5dg.fsf@smith.obbligato.org>
2012-02-27 16:22 ` [PATCH] send-email: document the --smtp-debug option Zbigniew Jędrzejewski-Szmek
2012-02-27 20:19   ` Junio C Hamano

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.