{"thread":{"id":"29758","subject":"[PATCH] send-email: document the --smtp-debug option","startedAt":"2012-02-27T16:22:53Z","lastAt":"2012-02-27T20:19:23Z","messageCount":2,"participants":["Zbigniew Jędrzejewski-Szmek","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"185521","messageId":"1330359773-16759-1-git-send-email-zbyszek@in.waw.pl","threadId":"29758","inReplyTo":"87ehtiu5dg.fsf@smith.obbligato.org","subject":"[PATCH] send-email: document the --smtp-debug option","fromName":"Zbigniew Jędrzejewski-Szmek","fromEmail":"zbyszek@in.waw.pl","sentAt":"2012-02-27T16:22:53Z","receivedAt":"2012-02-27T16:22:53Z","isPatch":true,"sender":{"key":"zbyszek@in.waw.pl","avatar":"https://avatars.githubusercontent.com/u/349618?v=4"},"body":"The option was already shown in -h output, so it should be documented\nin the man page.\n\nSigned-off-by: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>\nSuggested-by: David A. Greene <greened@obbligato.org>\n---\nDavid Greene wrote:\n> I don't think --smtp-debug is documented in the man pages.  Was that a\n> deliberate decision or an oversight?\n\n Documentation/git-send-email.txt |    4 ++++\n 1 file changed, 4 insertions(+)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex 327233c..3241170 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -198,6 +198,10 @@ must be used for each option.\n \tif a username is not specified (with '--smtp-user' or 'sendemail.smtpuser'),\n \tthen authentication is not attempted.\n \n+--smtp-debug=0|1::\n+\tEnable (1) or disable (0) debug output. If enabled, SMTP\n+\tcommands and replies will be printed. Useful to debug TLS\n+\tconnection and authentication problems.\n \n Automating\n ~~~~~~~~~~\n-- \n1.7.9.2.378.g4d260\n"},{"id":"185555","messageId":"7v8vjo6nck.fsf@alter.siamese.dyndns.org","threadId":"29758","inReplyTo":"1330359773-16759-1-git-send-email-zbyszek@in.waw.pl","subject":"Re: [PATCH] send-email: document the --smtp-debug option","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-02-27T20:19:23Z","receivedAt":"2012-02-27T20:19:23Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> writes:\n\n> The option was already shown in -h output, so it should be documented\n> in the man page.\n>\n> Signed-off-by: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>\n> Suggested-by: David A. Greene <greened@obbligato.org>\n> ---\n> David Greene wrote:\n>> I don't think --smtp-debug is documented in the man pages.  Was that a\n>> deliberate decision or an oversight?\n\nProbably halfway in between \"Meh\"; anybody who is willing to look into the\nissue will notice when he opens the script and reads it for the first time\nanyway, so why bother.\n\nBut now \"A patch already exists, and it does not seem to have any funny\nletters that may screw up asciidoc, so why bother rejecting\" ;-)\n\nWill apply; thanks.\n\n>  Documentation/git-send-email.txt |    4 ++++\n>  1 file changed, 4 insertions(+)\n>\n> diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\n> index 327233c..3241170 100644\n> --- a/Documentation/git-send-email.txt\n> +++ b/Documentation/git-send-email.txt\n> @@ -198,6 +198,10 @@ must be used for each option.\n>  \tif a username is not specified (with '--smtp-user' or 'sendemail.smtpuser'),\n>  \tthen authentication is not attempted.\n>  \n> +--smtp-debug=0|1::\n> +\tEnable (1) or disable (0) debug output. If enabled, SMTP\n> +\tcommands and replies will be printed. Useful to debug TLS\n> +\tconnection and authentication problems.\n>  \n>  Automating\n>  ~~~~~~~~~~\n"}]}