{"thread":{"id":"46334","subject":"[PATCH 0/4] Misc. documentation fixes","startedAt":"2017-07-08T20:39:55Z","lastAt":"2017-07-10T16:18:03Z","messageCount":10,"participants":["astian","Junio C Hamano","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":4},"messages":[{"id":"324002","messageId":"20170708203413.16442-1-astian@eclipso.at","threadId":"46334","inReplyTo":null,"subject":"[PATCH 0/4] Misc. documentation fixes","fromName":"astian","fromEmail":"astian@eclipso.at","sentAt":"2017-07-08T20:34:09Z","receivedAt":"2017-07-08T20:39:55Z","isPatch":true,"sender":{"key":"astian@eclipso.at","avatar":null},"body":"Hi there,\n\nI was looking into adding a new option to git-send-email (and I may\nlater send the patches implementing that) but as I looked over the code\nand associated documentation I found a few places where I thought\nimprovement could be made. So here are some documentation fixes and\nclarifications that you might find useful.\n\nThey apply cleanly on top of commit 8b2efe2a0 (Fifteenth batch for 2.14).\n\nNote 1: even though I'm sending them as a \"series\", all of these are in\nfact independent of each other.\n\nNote 2: see below for a couple of patch-specific comments.\n\nastian (4):\n Doc/config.txt: explain repeated sections\n Doc/config.txt: fix typos\n Doc/git-send-email.txt: clarify a couple of options\n\n Still to fix:\n --smtp-encryption:\n The current nomenclarure is braindamaged: what's currently\n called \"ssl\" should be called \"tls\", what's currently called\n \"tls\" should be called \"starttls\".\n\n --smtp-server-option:\n IMO the option is wrongly named: this option only works with a\n local sendmail-like binary. When talking to that program, SMTP\n is not used.\n\n Doc/config.txt: fix documentation for smtpEncryption\n\n I don't know what's the story behind this, but IMO having\n \"smtpEncryption\" configurable from \"sendemail.\"\n subsections is useful and should be preserved.\n\n For example, one can set \"ssl\" (meaning TLS) as global configuration\n (under \"sendemail\"), but set it to \"none\" (that is, plain text) for\n a particular identity that actually connects to an i2p or onion\n server, i.e. through an already encrypted transport.\n\n Documentation/config.txt | 5 +----\n Documentation/git-send-email.txt | 15 +++++++++------\n 2 files changed, 10 insertions(+), 10 deletions(-)\n\n-- \n2.13.2\n\n\n\n"},{"id":"324003","messageId":"20170708203413.16442-2-astian@eclipso.at","threadId":"46334","inReplyTo":"20170708203413.16442-1-astian@eclipso.at","subject":"[PATCH 1/4] Doc/config.txt: explain repeated sections","fromName":"astian","fromEmail":"astian@eclipso.at","sentAt":"2017-07-08T20:34:10Z","receivedAt":"2017-07-08T20:40:07Z","isPatch":true,"sender":{"key":"astian@eclipso.at","avatar":null},"body":"Add a paragraph explaining what happens when a section name is repeated\nin the configuration file(s).\n\nThe example configuration file shown in this document already implied\nGit's behaviour, this patch simply tries to make it explicit.\n\nSigned-off-by: astian \n---\n Documentation/config.txt | 4 ++++\n 1 file changed, 4 insertions(+)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex d5c9c4cab..afce25705 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -31,6 +31,10 @@ characters, `-` and `.` are allowed in section names. Each variable\n must belong to some section, which means that there must be a section\n header before the first setting of a variable.\n \n+Multiple occurrences of the same section are all logically merged. (There's\n+no special treatment for variables defined multiple times across physically\n+different sections, the variable is simply made multivalued.)\n+\n Sections can be further divided into subsections. To begin a subsection\n put its name in double quotes, separated by space from the section name,\n in the section header, like in the example below:\n-- \n2.13.2\n\n\n\n"},{"id":"324004","messageId":"20170708203413.16442-3-astian@eclipso.at","threadId":"46334","inReplyTo":"20170708203413.16442-1-astian@eclipso.at","subject":"[PATCH 2/4] Doc/config.txt: fix typos","fromName":"astian","fromEmail":"astian@eclipso.at","sentAt":"2017-07-08T20:34:11Z","receivedAt":"2017-07-08T20:40:23Z","isPatch":true,"sender":{"key":"astian@eclipso.at","avatar":null},"body":"Signed-off-by: astian \n---\n Documentation/config.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex d5c9c4cab..72e85b002 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -60,7 +60,7 @@ The variable names are case-insensitive, allow only alphanumeric characters\n and `-`, and must start with an alphabetic character.\n \n A line that defines a value can be continued to the next line by\n-ending it with a `\\`; the backquote and the end-of-line are\n+ending it with a `\\`; the backslash and the end-of-line are\n stripped. Leading whitespaces after 'name =', the remainder of the\n line after the first comment character '#' or ';', and trailing\n whitespaces of the line are discarded unless they are enclosed in\n@@ -2912,7 +2912,7 @@ sendemail.smtpsslcertpath::\n \n sendemail..*::\n \tIdentity-specific versions of the 'sendemail.*' parameters\n-\tfound below, taking precedence over those when the this\n+\tfound below, taking precedence over those when this\n \tidentity is selected, through command-line or\n \t`sendemail.identity`.\n \n-- \n2.13.2\n\n\n\n"},{"id":"324005","messageId":"20170708203413.16442-4-astian@eclipso.at","threadId":"46334","inReplyTo":"20170708203413.16442-1-astian@eclipso.at","subject":"[PATCH 3/4] Doc/git-send-email.txt: clarify a couple of options","fromName":"astian","fromEmail":"astian@eclipso.at","sentAt":"2017-07-08T20:34:12Z","receivedAt":"2017-07-08T20:40:24Z","isPatch":true,"sender":{"key":"astian@eclipso.at","avatar":null},"body":"Clarify the meaning of --smtp-encryption and --smtp-server-option.\n\nSigned-off-by: astian \n---\n Documentation/git-send-email.txt | 15 +++++++++------\n 1 file changed, 9 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex bac9014ac..158b51045 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -161,7 +161,8 @@ Sending\n \tunspecified, choosing the envelope sender is left to your MTA.\n \n --smtp-encryption=::\n-\tSpecify the encryption to use, either 'ssl' or 'tls'. Any other\n+\tSpecify the encryption to use, either 'ssl' (direct TLS) or\n+\t'tls' (plain text upgraded to TLS via STARTTLS). Any other\n \tvalue reverts to plain SMTP. Default is the value of\n \t`sendemail.smtpEncryption`.\n \n@@ -216,13 +217,15 @@ a password is obtained using 'git-credential'.\n \t`sendemail.smtpServerPort` configuration variable.\n \n --smtp-server-option=