{"thread":{"id":"60968","subject":"[PATCH v4] documentation: send-email: use camel case consistently","startedAt":"2024-02-20T22:22:59Z","lastAt":"2024-02-21T00:04:32Z","messageCount":3,"participants":["Dragan Simic","Junio C Hamano"],"isPatch":true,"patchVersion":4,"patchTotal":null},"messages":[{"id":"489032","messageId":"88f1fe08c3047e14090957093ee8d98b0f60cb6c.1708467601.git.dsimic@manjaro.org","threadId":"60968","inReplyTo":null,"subject":"[PATCH v4] documentation: send-email: use camel case consistently","fromName":"Dragan Simic","fromEmail":"dsimic@manjaro.org","sentAt":"2024-02-20T22:22:51Z","receivedAt":"2024-02-20T22:22:59Z","isPatch":true,"sender":{"key":"dsimic@manjaro.org","avatar":null},"body":"Correct a few random \"sendemail.*\" configuration parameter names in the\ndocumentation that, for some unknown reason and contrary to the expected,\ndidn't use camel case format.\n\nThe majority of the corrections are straightforward, by using camel case\nto denote boundaries of the individual words that, stringed together, make\nup configuration parameter names.  A couple of abbreviations found in some\nof the corrected configuration parameter names present some exceptions,\nwhich are described in detail below.\n\nFirst, there's \"SSL\" as the abbreviation for \"Secure Sockets Layer\". [1]\nAs such, it's written using all uppercase letters, which is pretty much the\ngeneral rule for making abbreviations, although with certain exceptions.\n\nSecond, there's \"Cc\" as the abbreviation for \"carbon copy\", which is another\nexception.  As the acronym for \"carbon copy\", \"cc\" (mind the all lowercase\nletters) stems from the rather old times when, literally, carbon copies were\nmade. [2]  Therefore, using \"CC\" (mind the all uppercase letters) or \"cc\"\n(mind the all lowercase letters) would be technically correct in the email\ndomain, as the abbreviation or as mentioned in RFC2076, [3] respectively, but\nthe age of email has established \"Cc\" (mind the mixed uppercase and lowercase\nletters) as some kind of de facto standard. [1][4][5]  Moreover, some of the\ngit utilities, primarily git-send-email(1), already refer to making email\ncarbon copies as specifying \"Cc:\" email headers.  As a result, \"Cc\" becomes\none of the exceptions to the general rule for making abbreviations.\n\n[1] https://en.wikipedia.org/wiki/Transport_Layer_Security\n[2] https://en.wikipedia.org/wiki/Carbon_copy\n[3] https://datatracker.ietf.org/doc/html/rfc2076\n[4] https://bugzilla.mozilla.org/show_bug.cgi?id=212059\n[5] https://bugzilla.mozilla.org/show_bug.cgi?id=50826\n\nSigned-off-by: Dragan Simic <dsimic@manjaro.org>\n---\n\nNotes:\n    The versions 3 and 4 include further feedback from Junio, by adding a\n    detailed rationale for using \"Cc\" instead of \"CC\", while also using \"SSL\",\n    to the commit description.  No code changes were introduced.\n    \n    The version 2 includes feedback from Junio, by changing \"Ssl\" to \"SSL\"\n    in a couple of places where \"sendemail.smtpSSLCertPath\" is mentioned.\n    There are already instances of \"SSL\" being used, for example in various\n    \"http.proxySSL*\" configuration parameter names.\n\n Documentation/config/sendemail.txt | 12 ++++++------\n Documentation/git-send-email.txt   | 18 +++++++++---------\n 2 files changed, 15 insertions(+), 15 deletions(-)\n\ndiff --git a/Documentation/config/sendemail.txt b/Documentation/config/sendemail.txt\nindex 7fc770ee9e69..6a869d67eb90 100644\n--- a/Documentation/config/sendemail.txt\n+++ b/Documentation/config/sendemail.txt\n@@ -8,7 +8,7 @@ sendemail.smtpEncryption::\n \tSee linkgit:git-send-email[1] for description.  Note that this\n \tsetting is not subject to the 'identity' mechanism.\n \n-sendemail.smtpsslcertpath::\n+sendemail.smtpSSLCertPath::\n \tPath to ca-certificates (either a directory or a single file).\n \tSet it to an empty string to disable certificate verification.\n \n@@ -62,27 +62,27 @@ sendemail.chainReplyTo::\n sendemail.envelopeSender::\n sendemail.from::\n sendemail.headerCmd::\n-sendemail.signedoffbycc::\n+sendemail.signedOffByCc::\n sendemail.smtpPass::\n-sendemail.suppresscc::\n+sendemail.suppressCc::\n sendemail.suppressFrom::\n sendemail.to::\n-sendemail.tocmd::\n+sendemail.toCmd::\n sendemail.smtpDomain::\n sendemail.smtpServer::\n sendemail.smtpServerPort::\n sendemail.smtpServerOption::\n sendemail.smtpUser::\n sendemail.thread::\n sendemail.transferEncoding::\n sendemail.validate::\n sendemail.xmailer::\n \tThese configuration variables all provide a default for\n \tlinkgit:git-send-email[1] command-line options. See its\n \tdocumentation for details.\n \n-sendemail.signedoffcc (deprecated)::\n-\tDeprecated alias for `sendemail.signedoffbycc`.\n+sendemail.signedOffCc (deprecated)::\n+\tDeprecated alias for `sendemail.signedOffByCc`.\n \n sendemail.smtpBatchSize::\n \tNumber of messages to be sent per connection, after that a relogin\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex d1ef6a204e68..8264f8738093 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -138,7 +138,7 @@ Note that no attempts whatsoever are made to validate the encoding.\n \n --compose-encoding=<encoding>::\n \tSpecify encoding of compose message. Default is the value of the\n-\t'sendemail.composeencoding'; if that is unspecified, UTF-8 is assumed.\n+\t'sendemail.composeEncoding'; if that is unspecified, UTF-8 is assumed.\n \n --transfer-encoding=(7bit|8bit|quoted-printable|base64|auto)::\n \tSpecify the transfer encoding to be used to send the message over SMTP.\n@@ -174,7 +174,7 @@ Sending\n \tSpecify a command to run to send the email. The command should\n \tbe sendmail-like; specifically, it must support the `-i` option.\n \tThe command will be executed in the shell if necessary.  Default\n-\tis the value of `sendemail.sendmailcmd`.  If unspecified, and if\n+\tis the value of `sendemail.sendmailCmd`.  If unspecified, and if\n \t--smtp-server is also unspecified, git-send-email will search\n \tfor `sendmail` in `/usr/sbin`, `/usr/lib` and $PATH.\n \n@@ -269,7 +269,7 @@ must be used for each option.\n \tcertificates concatenated together: see verify(1) -CAfile and\n \t-CApath for more information on these). Set it to an empty string\n \tto disable certificate verification. Defaults to the value of the\n-\t`sendemail.smtpsslcertpath` configuration variable, if set, or the\n+\t`sendemail.smtpSSLCertPath` configuration variable, if set, or the\n \tbacking SSL library's compiled-in default otherwise (which should\n \tbe the best choice on most platforms).\n \n@@ -313,7 +313,7 @@ Automating\n \tSpecify a command to execute once per patch file which\n \tshould generate patch file specific \"To:\" entries.\n \tOutput of this command must be single email address per line.\n-\tDefault is the value of 'sendemail.tocmd' configuration value.\n+\tDefault is the value of 'sendemail.toCmd' configuration value.\n \n --cc-cmd=<command>::\n \tSpecify a command to execute once per patch file which\n@@ -348,19 +348,19 @@ Automating\n \n --[no-]signed-off-by-cc::\n \tIf this is set, add emails found in the `Signed-off-by` trailer or Cc: lines to the\n-\tcc list. Default is the value of `sendemail.signedoffbycc` configuration\n+\tcc list. Default is the value of `sendemail.signedOffByCc` configuration\n \tvalue; if that is unspecified, default to --signed-off-by-cc.\n \n --[no-]cc-cover::\n \tIf this is set, emails found in Cc: headers in the first patch of\n \tthe series (typically the cover letter) are added to the cc list\n-\tfor each email set. Default is the value of 'sendemail.cccover'\n+\tfor each email set. Default is the value of 'sendemail.ccCover'\n \tconfiguration value; if that is unspecified, default to --no-cc-cover.\n \n --[no-]to-cover::\n \tIf this is set, emails found in To: headers in the first patch of\n \tthe series (typically the cover letter) are added to the to list\n-\tfor each email set. Default is the value of 'sendemail.tocover'\n+\tfor each email set. Default is the value of 'sendemail.toCover'\n \tconfiguration value; if that is unspecified, default to --no-to-cover.\n \n --suppress-cc=<category>::\n@@ -384,7 +384,7 @@ Automating\n - 'all' will suppress all auto cc values.\n --\n +\n-Default is the value of `sendemail.suppresscc` configuration value; if\n+Default is the value of `sendemail.suppressCc` configuration value; if\n that is unspecified, default to 'self' if --suppress-from is\n specified, as well as 'body' if --no-signed-off-cc is specified.\n \n@@ -471,7 +471,7 @@ Information\n \tInstead of the normal operation, dump the shorthand alias names from\n \tthe configured alias file(s), one per line in alphabetical order. Note\n \tthat this only includes the alias name and not its expanded email addresses.\n-\tSee 'sendemail.aliasesfile' for more information about aliases.\n+\tSee 'sendemail.aliasesFile' for more information about aliases.\n \n \n CONFIGURATION\n"},{"id":"489040","messageId":"xmqqle7elp62.fsf@gitster.g","threadId":"60968","inReplyTo":"88f1fe08c3047e14090957093ee8d98b0f60cb6c.1708467601.git.dsimic@manjaro.org","subject":"Re: [PATCH v4] documentation: send-email: use camel case consistently","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-02-20T22:57:25Z","receivedAt":"2024-02-20T22:57:31Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Dragan Simic <dsimic@manjaro.org> writes:\n\n> -sendemail.smtpsslcertpath::\n> +sendemail.smtpSSLCertPath::\n> -sendemail.signedoffbycc::\n> +sendemail.signedOffByCc::\n> -sendemail.suppresscc::\n> +sendemail.suppressCc::\n> -sendemail.tocmd::\n> +sendemail.toCmd::\n> -sendemail.signedoffcc (deprecated)::\n> -\tDeprecated alias for `sendemail.signedoffbycc`.\n> +sendemail.signedOffCc (deprecated)::\n> +\tDeprecated alias for `sendemail.signedOffByCc`.\n\nAll look good. \n\n> diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\n> index d1ef6a204e68..8264f8738093 100644\n> --- a/Documentation/git-send-email.txt\n> +++ b/Documentation/git-send-email.txt\n> @@ -138,7 +138,7 @@ Note that no attempts whatsoever are made to validate the encoding.\n>  \n>  --compose-encoding=<encoding>::\n>  \tSpecify encoding of compose message. Default is the value of the\n> -\t'sendemail.composeencoding'; if that is unspecified, UTF-8 is assumed.\n> +\t'sendemail.composeEncoding'; if that is unspecified, UTF-8 is assumed.\n\nGood.\n\n> @@ -174,7 +174,7 @@ Sending\n>  \tSpecify a command to run to send the email. The command should\n>  \tbe sendmail-like; specifically, it must support the `-i` option.\n>  \tThe command will be executed in the shell if necessary.  Default\n> -\tis the value of `sendemail.sendmailcmd`.  If unspecified, and if\n> +\tis the value of `sendemail.sendmailCmd`.  If unspecified, and if\n\nGood.\n\n> @@ -269,7 +269,7 @@ must be used for each option.\n>  \tcertificates concatenated together: see verify(1) -CAfile and\n>  \t-CApath for more information on these). Set it to an empty string\n>  \tto disable certificate verification. Defaults to the value of the\n> -\t`sendemail.smtpsslcertpath` configuration variable, if set, or the\n> +\t`sendemail.smtpSSLCertPath` configuration variable, if set, or the\n\nGood.\n\n> @@ -313,7 +313,7 @@ Automating\n>  \tSpecify a command to execute once per patch file which\n>  \tshould generate patch file specific \"To:\" entries.\n>  \tOutput of this command must be single email address per line.\n> -\tDefault is the value of 'sendemail.tocmd' configuration value.\n> +\tDefault is the value of 'sendemail.toCmd' configuration value.\n\nGood.\n\n> -\tcc list. Default is the value of `sendemail.signedoffbycc` configuration\n> +\tcc list. Default is the value of `sendemail.signedOffByCc` configuration\n\nGood.\n\n> -\tfor each email set. Default is the value of 'sendemail.cccover'\n> +\tfor each email set. Default is the value of 'sendemail.ccCover'\n\nGood.\n\n> -\tfor each email set. Default is the value of 'sendemail.tocover'\n> +\tfor each email set. Default is the value of 'sendemail.toCover'\n\nGood.\n\n> -Default is the value of `sendemail.suppresscc` configuration value; if\n> +Default is the value of `sendemail.suppressCc` configuration value; if\n\nGood.\n\n> -\tSee 'sendemail.aliasesfile' for more information about aliases.\n> +\tSee 'sendemail.aliasesFile' for more information about aliases.\n\nGood.\n\nThis version looks very good to me.  Thanks.  Will queue.\n"},{"id":"489042","messageId":"9900bb4dc72101b48267ee7bc4b978be@manjaro.org","threadId":"60968","inReplyTo":"xmqqle7elp62.fsf@gitster.g","subject":"Re: [PATCH v4] documentation: send-email: use camel case consistently","fromName":"Dragan Simic","fromEmail":"dsimic@manjaro.org","sentAt":"2024-02-21T00:04:30Z","receivedAt":"2024-02-21T00:04:32Z","isPatch":true,"sender":{"key":"dsimic@manjaro.org","avatar":null},"body":"On 2024-02-20 23:57, Junio C Hamano wrote:\n> \n> This version looks very good to me.  Thanks.  Will queue.\n\nGreat, thank you!  I'm glad we've reached this state of the entire \npatch.\n"}]}