{"thread":{"id":"15623","subject":"[PATCH RFC 3/6] Docs: send-email usage text much sexier","startedAt":"2008-09-22T15:58:10Z","lastAt":"2008-09-27T01:13:44Z","messageCount":14,"participants":["Michael Witten","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":6},"messages":[{"id":"91319","messageId":"1222099095-50360-1-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":null,"subject":"[PATCH RFC 1/6] Docs: send-email's usage text and man page mention same options","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-22T15:58:10Z","receivedAt":"2008-09-22T15:58:10Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"Specifically, boolean options are now listed in the form\n\n --[no-]option\n\nand both forms of documentation now consistently use\n\n --[no-]signed-off-by-cc\n\nSigned-off-by: Michael Witten \n---\n Documentation/git-send-email.txt | 21 +++++++++++++--------\n git-send-email.perl | 8 ++++----\n 2 files changed, 17 insertions(+), 12 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex 3c3e1b0..9b31626 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -40,8 +40,7 @@ The --cc option must be repeated for each user you want on the cc list.\n \tOutput of this command must be single email address per line.\n \tDefault is the value of 'sendemail.cccmd' configuration value.\n \n---chain-reply-to::\n---no-chain-reply-to::\n+--[no-]chain-reply-to::\n \tIf this is set, each email will be sent as a reply to the previous\n \temail sent. If disabled with \"--no-chain-reply-to\", all emails after\n \tthe first will be sent as replies to the first email sent. When using\n@@ -66,8 +65,7 @@ The --cc option must be repeated for each user you want on the cc list.\n \tOnly necessary if --compose is also set. If --compose\n \tis not set, this will be prompted for.\n \n---signed-off-by-cc::\n---no-signed-off-by-cc::\n+--[no-]signed-off-by-cc::\n If this is set, add emails found in Signed-off-by: or Cc: lines to the\n cc list.\n Default is the value of 'sendemail.signedoffcc' configuration value;\n@@ -146,8 +144,7 @@ user is prompted for a password while the input is masked for privacy.\n \tOnly necessary if --compose is also set. If --compose\n \tis not set, this will be prompted for.\n \n---suppress-from::\n---no-suppress-from::\n+--[no-]suppress-from::\n If this is set, do not add the From: address to the cc: list.\n Default is the value of 'sendemail.suppressfrom' configuration value;\n if that is unspecified, default to --no-suppress-from.\n@@ -163,8 +160,7 @@ user is prompted for a password while the input is masked for privacy.\n \tif that is unspecified, default to 'self' if --suppress-from is\n \tspecified, as well as 'sob' if --no-signed-off-cc is specified.\n \n---thread::\n---no-thread::\n+--[no-]thread::\n \tIf this is set, the In-Reply-To header will be set on each email sent.\n \tIf disabled with \"--no-thread\", no emails will have the In-Reply-To\n \theader set.\n@@ -183,6 +179,15 @@ user is prompted for a password while the input is masked for privacy.\n \tvariable; if that is unspecified, choosing the envelope sender is left\n \tto your MTA.\n \n+--no-validate::\n+\tDon't perform any sanity checks on patches.\n+\tCurrently, validation means the following:\n++\n+--\n+\t\t*\tWarn of patches that contain lines longer than 998 characters; this\n+\t\t\tis due to SMTP limits as described by http://www.ietf.org/rfc/rfc2821.txt.\n+--\n+\n --to::\n \tSpecify the primary recipient of the emails generated.\n \tGenerally, this will be the upstream maintainer of the\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex d2fd899..9f56162 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -64,11 +64,11 @@ Options:\n Only used if --compose is also set. If --compose is not\n \t\t set, this will be prompted for.\n \n- --chain-reply-to If set, the replies will all be to the previous\n+ --[no-]chain-reply-to If set, the replies will all be to the previous\n email sent, rather than to the first email sent.\n Defaults to on.\n \n- --signed-off-cc Automatically add email addresses that appear in\n+ --[no-]signed-off-by-cc Automatically add email addresses that appear in\n Signed-off-by: or Cc: lines to the cc: list. Defaults to on.\n \n --identity The configuration identity, a subsection to prioritise over\n@@ -95,9 +95,9 @@ Options:\n \t\t 'cccmd' for the output of the cccmd, or 'all' to suppress\n \t\t all of these.\n \n- --suppress-from Suppress sending emails to yourself. Defaults to off.\n+ --[no-]suppress-from Suppress sending emails to yourself. Defaults to off.\n \n- --thread Specify that the \"In-Reply-To:\" header should be set on all\n+ --[no-]thread Specify that the \"In-Reply-To:\" header should be set on all\n emails. Defaults to on.\n \n --quiet\t Make git-send-email less verbose. One line per email\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91322","messageId":"1222099095-50360-2-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222099095-50360-1-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 2/6] Docs: send-email usage cleanup 1","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-22T15:58:11Z","receivedAt":"2008-09-22T15:58:11Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"The usage text mainly uses spaces to layout the text,\nso I replaced the few tabs that exit with spaces.\n\nSigned-off-by: Michael Witten \n---\n git-send-email.perl | 36 ++++++++++++++++++------------------\n 1 files changed, 18 insertions(+), 18 deletions(-)\n\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex 9f56162..d390364 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -51,25 +51,25 @@ Options:\n per file specific cc address entries\n \n --bcc Specify a list of email addresses that should be Bcc:\n-\t\t on all the emails.\n+ on all the emails.\n \n --compose Use \\$GIT_EDITOR, core.editor, \\$EDITOR, or \\$VISUAL to edit\n-\t\t an introductory message for the patch series.\n+ an introductory message for the patch series.\n \n --subject Specify the initial \"Subject:\" line.\n Only necessary if --compose is also set. If --compose\n-\t\t is not set, this will be prompted for.\n+ is not set, this will be prompted for.\n \n --in-reply-to Specify the first \"In-Reply-To:\" header line.\n Only used if --compose is also set. If --compose is not\n-\t\t set, this will be prompted for.\n+ set, this will be prompted for.\n \n --[no-]chain-reply-to If set, the replies will all be to the previous\n- email sent, rather than to the first email sent.\n- Defaults to on.\n+ email sent, rather than to the first email sent.\n+ Defaults to on.\n \n --[no-]signed-off-by-cc Automatically add email addresses that appear in\n- Signed-off-by: or Cc: lines to the cc: list. Defaults to on.\n+ Signed-off-by: or Cc: lines to the cc: list. Defaults to on.\n \n --identity The configuration identity, a subsection to prioritise over\n the default section.\n@@ -85,29 +85,29 @@ Options:\n --smtp-pass The password for SMTP-AUTH.\n \n --smtp-encryption Specify 'tls' for STARTTLS encryption, or 'ssl' for SSL.\n- Any other value disables the feature.\n+ Any other value disables the feature.\n \n --smtp-ssl Synonym for '--smtp-encryption=ssl'. Deprecated.\n \n --suppress-cc Suppress the specified category of auto-CC. The category\n-\t\t can be one of 'author' for the patch author, 'self' to\n-\t\t avoid copying yourself, 'sob' for Signed-off-by lines,\n-\t\t 'cccmd' for the output of the cccmd, or 'all' to suppress\n-\t\t all of these.\n+ can be one of 'author' for the patch author, 'self' to\n+ avoid copying yourself, 'sob' for Signed-off-by lines,\n+ 'cccmd' for the output of the cccmd, or 'all' to suppress\n+ all of these.\n \n --[no-]suppress-from Suppress sending emails to yourself. Defaults to off.\n \n --[no-]thread Specify that the \"In-Reply-To:\" header should be set on all\n- emails. Defaults to on.\n+ emails. Defaults to on.\n \n- --quiet\t Make git-send-email less verbose. One line per email\n- should be all that is output.\n+ --quiet Make git-send-email less verbose. One line per email\n+ should be all that is output.\n \n- --dry-run\t Do everything except actually send the emails.\n+ --dry-run Do everything except actually send the emails.\n \n- --envelope-sender\tSpecify the envelope sender used to send the emails.\n+ --envelope-sender Specify the envelope sender used to send the emails.\n \n- --no-validate\tDon't perform any sanity checks on patches.\n+ --no-validate Don't perform any sanity checks on patches.\n \n EOT\n \texit(1);\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91317","messageId":"1222099095-50360-3-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222099095-50360-2-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 3/6] Docs: send-email usage text much sexier","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-22T15:58:12Z","receivedAt":"2008-09-22T15:58:12Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"All of the descriptions are aligned, and no\nline is greater than 78 columns.\n\nSigned-off-by: Michael Witten \n---\n git-send-email.perl | 94 +++++++++++++++++++++++++++------------------------\n 1 files changed, 50 insertions(+), 44 deletions(-)\n\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex d390364..b86a3f3 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -40,74 +40,80 @@ sub usage {\n \tprint <...\n Options:\n- --from Specify the \"From:\" line of the email to be sent.\n+ --from Specify the \"From:\" line of the email to be sent.\n \n- --to Specify the primary \"To:\" line of the email.\n+ --to Specify the primary \"To:\" line of the email.\n \n- --cc Specify an initial \"Cc:\" list for the entire series\n- of emails.\n+ --cc Specify an initial \"Cc:\" list for the entire series\n+ of emails.\n \n- --cc-cmd Specify a command to execute per file which adds\n- per file specific cc address entries\n+ --cc-cmd Specify a command to execute per file which adds\n+ per file specific cc address entries\n \n- --bcc Specify a list of email addresses that should be Bcc:\n- on all the emails.\n+ --bcc Specify a list of email addresses that should be\n+ Bcc: on all the emails.\n \n- --compose Use \\$GIT_EDITOR, core.editor, \\$EDITOR, or \\$VISUAL to edit\n- an introductory message for the patch series.\n+ --compose Use \\$GIT_EDITOR, core.editor, \\$EDITOR, or\n+ \\$VISUAL to edit an introductory message for the\n+ patch series.\n \n- --subject Specify the initial \"Subject:\" line.\n- Only necessary if --compose is also set. If --compose\n- is not set, this will be prompted for.\n+ --subject Specify the initial \"Subject:\" line. Only necessary\n+ if --compose is also set. If --compose is not set,\n+ this will be prompted for.\n \n- --in-reply-to Specify the first \"In-Reply-To:\" header line.\n- Only used if --compose is also set. If --compose is not\n- set, this will be prompted for.\n+ --in-reply-to Specify the first \"In-Reply-To:\" header line. Only\n+ used if --compose is also set. If --compose is not\n+ set, this will be prompted for.\n \n- --[no-]chain-reply-to If set, the replies will all be to the previous\n- email sent, rather than to the first email sent.\n- Defaults to on.\n+ --[no-]chain-reply-to If set, the replies will all be to the previous\n+ email sent, rather than to the first email sent.\n+ Defaults to on.\n \n --[no-]signed-off-by-cc Automatically add email addresses that appear in\n- Signed-off-by: or Cc: lines to the cc: list. Defaults to on.\n+ Signed-off-by: or Cc: lines to the cc: list.\n+ Defaults to on.\n \n- --identity The configuration identity, a subsection to prioritise over\n- the default section.\n+ --identity The configuration identity to use.\n \n- --smtp-server If set, specifies the outgoing SMTP server to use.\n- Defaults to localhost. Port number can be specified here with\n- hostname:port format or by using --smtp-server-port option.\n+ --smtp-server If set, specifies the outgoing SMTP server to use.\n+ Defaults to localhost. Port number can be\n+ specified here with hostname:port format or by\n+ using --smtp-server-port option.\n \n- --smtp-server-port Specify a port on the outgoing SMTP server to connect to.\n+ --smtp-server-port Specify a port on the outgoing SMTP server to\n+ connect to.\n \n- --smtp-user The username for SMTP-AUTH.\n+ --smtp-user The username for SMTP-AUTH.\n \n- --smtp-pass The password for SMTP-AUTH.\n+ --smtp-pass The password for SMTP-AUTH.\n \n- --smtp-encryption Specify 'tls' for STARTTLS encryption, or 'ssl' for SSL.\n- Any other value disables the feature.\n+ --smtp-encryption Specify 'tls' for STARTTLS encryption, or 'ssl' for\n+ SSL. Any other value disables the feature.\n \n- --smtp-ssl Synonym for '--smtp-encryption=ssl'. Deprecated.\n+ --smtp-ssl Synonym for '--smtp-encryption=ssl'. Deprecated.\n \n- --suppress-cc Suppress the specified category of auto-CC. The category\n- can be one of 'author' for the patch author, 'self' to\n- avoid copying yourself, 'sob' for Signed-off-by lines,\n- 'cccmd' for the output of the cccmd, or 'all' to suppress\n- all of these.\n+ --suppress-cc Suppress the specified category of auto-CC. The\n+ category can be one of 'author' for the patch\n+ author, 'self' to avoid copying yourself, 'sob'\n+ for Signed-off-by lines, 'cccmd' for the output\n+ of the cccmd, or 'all' to suppress all of these.\n+ (Note: Linus uses 'self' and 'sob' interchangeably)\n \n- --[no-]suppress-from Suppress sending emails to yourself. Defaults to off.\n+ --[no-]suppress-from Suppress sending emails to yourself. Defaults to\n+ off.\n \n- --[no-]thread Specify that the \"In-Reply-To:\" header should be set on all\n- emails. Defaults to on.\n+ --[no-]thread Specify that the \"In-Reply-To:\" header should be\n+ set on all emails. Defaults to on.\n \n- --quiet Make git-send-email less verbose. One line per email\n- should be all that is output.\n+ --quiet Make git-send-email less verbose. One line per\n+ email should be all that is output.\n \n- --dry-run Do everything except actually send the emails.\n+ --dry-run Do everything except actually send the emails.\n \n- --envelope-sender Specify the envelope sender used to send the emails.\n+ --envelope-sender Specify the envelope sender used to send the\n+ emails.\n \n- --no-validate Don't perform any sanity checks on patches.\n+ --no-validate Don't perform any sanity checks on patches.\n \n EOT\n \texit(1);\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91321","messageId":"1222099095-50360-4-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222099095-50360-3-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 4/6] Docs: send-email: Option order the same in man page and usage text","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-22T15:58:13Z","receivedAt":"2008-09-22T15:58:13Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"The man page seems to have listed the options in alphabetical order,\nwhile the usage text used what I would consider an `intuitive' order\nthat groups related items and presents them in the order peopl think\nwhen writing an email.\n\nThe manual page has been updated to mirror this order.\n\nSigned-off-by: Michael Witten \n---\n Documentation/git-send-email.txt | 77 +++++++++++++++++++-------------------\n 1 files changed, 39 insertions(+), 38 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex 9b31626..6031eb7 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -24,10 +24,19 @@ OPTIONS\n -------\n The options available are:\n \n---bcc::\n-\tSpecify a \"Bcc:\" value for each email.\n+--from::\n+\tSpecify the sender of the emails. This will default to\n+\tthe value GIT_COMMITTER_IDENT, as returned by \"git var -l\".\n+\tThe user will still be prompted to confirm this entry.\n+\n+--to::\n+\tSpecify the primary recipient of the emails generated.\n+\tGenerally, this will be the upstream maintainer of the\n+\tproject involved.\n+\tDefault is the value of the 'sendemail.to' configuration value;\n+\tif that is unspecified, this will be prompted for.\n +\n-The --bcc option must be repeated for each user you want on the bcc list.\n+The --to option must be repeated for each user you want on the to list.\n \n --cc::\n \tSpecify a starting \"Cc:\" value for each email.\n@@ -40,23 +49,19 @@ The --cc option must be repeated for each user you want on the cc list.\n \tOutput of this command must be single email address per line.\n \tDefault is the value of 'sendemail.cccmd' configuration value.\n \n---[no-]chain-reply-to::\n-\tIf this is set, each email will be sent as a reply to the previous\n-\temail sent. If disabled with \"--no-chain-reply-to\", all emails after\n-\tthe first will be sent as replies to the first email sent. When using\n-\tthis, it is recommended that the first file given be an overview of the\n-\tentire patch series.\n-\tDefault is the value of the 'sendemail.chainreplyto' configuration\n-\tvalue; if that is unspecified, default to --chain-reply-to.\n+--bcc::\n+\tSpecify a \"Bcc:\" value for each email.\n++\n+The --bcc option must be repeated for each user you want on the bcc list.\n \n --compose::\n \tUse $GIT_EDITOR, core.editor, $VISUAL, or $EDITOR to edit an\n \tintroductory message for the patch series.\n \n---from::\n-\tSpecify the sender of the emails. This will default to\n-\tthe value GIT_COMMITTER_IDENT, as returned by \"git var -l\".\n-\tThe user will still be prompted to confirm this entry.\n+--subject::\n+\tSpecify the initial subject of the email thread.\n+\tOnly necessary if --compose is also set. If --compose\n+\tis not set, this will be prompted for.\n \n --in-reply-to::\n \tSpecify the contents of the first In-Reply-To header.\n@@ -65,16 +70,21 @@ The --cc option must be repeated for each user you want on the cc list.\n \tOnly necessary if --compose is also set. If --compose\n \tis not set, this will be prompted for.\n \n+--[no-]chain-reply-to::\n+\tIf this is set, each email will be sent as a reply to the previous\n+\temail sent. If disabled with \"--no-chain-reply-to\", all emails after\n+\tthe first will be sent as replies to the first email sent. When using\n+\tthis, it is recommended that the first file given be an overview of the\n+\tentire patch series.\n+\tDefault is the value of the 'sendemail.chainreplyto' configuration\n+\tvalue; if that is unspecified, default to --chain-reply-to.\n+\n --[no-]signed-off-by-cc::\n If this is set, add emails found in Signed-off-by: or Cc: lines to the\n cc list.\n Default is the value of 'sendemail.signedoffcc' configuration value;\n if that is unspecified, default to --signed-off-by-cc.\n \n---quiet::\n-\tMake git-send-email less verbose. One line per email should be\n-\tall that is output.\n-\n --identity::\n \tA configuration identity. When given, causes values in the\n \t'sendemail.' subsection to take precedence over\n@@ -139,16 +149,6 @@ user is prompted for a password while the input is masked for privacy.\n --smtp-ssl::\n \tLegacy alias for '--smtp-encryption=ssl'.\n \n---subject::\n-\tSpecify the initial subject of the email thread.\n-\tOnly necessary if --compose is also set. If --compose\n-\tis not set, this will be prompted for.\n-\n---[no-]suppress-from::\n- If this is set, do not add the From: address to the cc: list.\n- Default is the value of 'sendemail.suppressfrom' configuration value;\n- if that is unspecified, default to --no-suppress-from.\n-\n --suppress-cc::\n \tSpecify an additional category of recipients to suppress the\n \tauto-cc of. 'self' will avoid including the sender, 'author' will\n@@ -159,6 +159,12 @@ user is prompted for a password while the input is masked for privacy.\n \tDefault is the value of 'sendemail.suppresscc' configuration value;\n \tif that is unspecified, default to 'self' if --suppress-from is\n \tspecified, as well as 'sob' if --no-signed-off-cc is specified.\n+\t(Note: Linus uses 'self' and 'sob' interchangeably).\n+\n+--[no-]suppress-from::\n+ If this is set, do not add the From: address to the cc: list.\n+ Default is the value of 'sendemail.suppressfrom' configuration value;\n+ if that is unspecified, default to --no-suppress-from.\n \n --[no-]thread::\n \tIf this is set, the In-Reply-To header will be set on each email sent.\n@@ -167,6 +173,10 @@ user is prompted for a password while the input is masked for privacy.\n \tDefault is the value of the 'sendemail.thread' configuration value;\n \tif that is unspecified, default to --thread.\n \n+--quiet::\n+\tMake git-send-email less verbose. One line per email should be\n+\tall that is output.\n+\n --dry-run::\n \tDo everything except actually send the emails.\n \n@@ -188,15 +198,6 @@ user is prompted for a password while the input is masked for privacy.\n \t\t\tis due to SMTP limits as described by http://www.ietf.org/rfc/rfc2821.txt.\n --\n \n---to::\n-\tSpecify the primary recipient of the emails generated.\n-\tGenerally, this will be the upstream maintainer of the\n-\tproject involved.\n-\tDefault is the value of the 'sendemail.to' configuration value;\n-\tif that is unspecified, this will be prompted for.\n-+\n-The --to option must be repeated for each user you want on the to list.\n-\n \n CONFIGURATION\n -------------\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91318","messageId":"1222099095-50360-5-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222099095-50360-4-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 5/6] Docs: send-email: change --no-validate to boolean --[no-]validate","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-22T15:58:14Z","receivedAt":"2008-09-22T15:58:14Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"There is also now a configuration variable:\n\n sendemail[.].validate\n\nSigned-off-by: Michael Witten \n---\n Documentation/git-send-email.txt | 9 +++++++--\n git-send-email.perl | 10 ++++++----\n 2 files changed, 13 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex 6031eb7..fe53e9b 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -189,14 +189,16 @@ user is prompted for a password while the input is masked for privacy.\n \tvariable; if that is unspecified, choosing the envelope sender is left\n \tto your MTA.\n \n---no-validate::\n-\tDon't perform any sanity checks on patches.\n+--[no-]validate::\n+\tPerform sanity checks on patches.\n \tCurrently, validation means the following:\n +\n --\n \t\t*\tWarn of patches that contain lines longer than 998 characters; this\n \t\t\tis due to SMTP limits as described by http://www.ietf.org/rfc/rfc2821.txt.\n --\n++\n+This option can be set with sendemail[.].validate; default to --validate\n \n \n CONFIGURATION\n@@ -249,6 +251,9 @@ sendemail.smtpencryption::\n sendemail.smtpssl::\n \tLegacy boolean that sets 'smtpencryption=ssl' if enabled.\n \n+sendemail.validate::\n+ Boolean override for --[no-]validate.\n+\n Author\n ------\n Written by Ryan Anderson \ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex b86a3f3..8bc204f 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -113,7 +113,8 @@ Options:\n --envelope-sender Specify the envelope sender used to send the\n emails.\n \n- --no-validate Don't perform any sanity checks on patches.\n+ --[no-]validate Perform any sanity checks on patches.\n+ Default to on.\n \n EOT\n \texit(1);\n@@ -195,7 +196,7 @@ my ($quiet, $dry_run) = (0, 0);\n my ($thread, $chain_reply_to, $suppress_from, $signed_off_cc, $cc_cmd);\n my ($smtp_server, $smtp_server_port, $smtp_authuser, $smtp_encryption);\n my ($identity, $aliasfiletype, @alias_files, @smtp_host_parts);\n-my ($no_validate);\n+my ($validate);\n my (@suppress_cc);\n \n my %config_bool_settings = (\n@@ -203,6 +204,7 @@ my %config_bool_settings = (\n \"chainreplyto\" => [\\$chain_reply_to, 1],\n \"suppressfrom\" => [\\$suppress_from, undef],\n \"signedoffcc\" => [\\$signed_off_cc, undef],\n+ \"validate\" => [\\$validate, 1],\n );\n \n my %config_settings = (\n@@ -269,7 +271,7 @@ my $rc = GetOptions(\"sender|from=s\" => \\$sender,\n \t\t \"dry-run\" => \\$dry_run,\n \t\t \"envelope-sender=s\" => \\$envelope_sender,\n \t\t \"thread!\" => \\$thread,\n-\t\t \"no-validate\" => \\$no_validate,\n+\t\t \"validate!\" => \\$validate,\n \t );\n \n unless ($rc) {\n@@ -422,7 +424,7 @@ for my $f (@ARGV) {\n \t}\n }\n \n-if (!$no_validate) {\n+if ($validate) {\n \tforeach my $f (@files) {\n \t\tunless (-p $f) {\n \t\t\tmy $error = validate_patch($f);\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91320","messageId":"1222099095-50360-6-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222099095-50360-5-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 6/6] Docs: send-email: --chain_reply_to -> --[no-]chain-reply-to","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-22T15:58:15Z","receivedAt":"2008-09-22T15:58:15Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"Signed-off-by: Michael Witten \n---\n Documentation/git-send-email.txt | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex fe53e9b..76a525a 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -228,7 +228,7 @@ sendemail.bcc::\n \tEmail address (or alias) to always bcc.\n \n sendemail.chainreplyto::\n-\tBoolean value specifying the default to the '--chain_reply_to'\n+\tBoolean value specifying the default to the '--[no-]chain-reply-to'\n \tparameter.\n \n sendemail.smtpserver::\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91627","messageId":"1222375476-32911-1-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222099095-50360-1-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 1/6] Docs: send-email's usage text and man page mention same options","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-25T20:44:31Z","receivedAt":"2008-09-25T20:44:31Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"Specifically, boolean options are now listed in the form\n\n --[no-]option\n\nand both forms of documentation now consistently use\n\n --[no-]signed-off-by-cc\n\nSigned-off-by: Michael Witten \n---\n Documentation/git-send-email.txt | 21 +++++++++++++--------\n git-send-email.perl | 8 ++++----\n 2 files changed, 17 insertions(+), 12 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex 3c3e1b0..9b31626 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -40,8 +40,7 @@ The --cc option must be repeated for each user you want on the cc list.\n \tOutput of this command must be single email address per line.\n \tDefault is the value of 'sendemail.cccmd' configuration value.\n \n---chain-reply-to::\n---no-chain-reply-to::\n+--[no-]chain-reply-to::\n \tIf this is set, each email will be sent as a reply to the previous\n \temail sent. If disabled with \"--no-chain-reply-to\", all emails after\n \tthe first will be sent as replies to the first email sent. When using\n@@ -66,8 +65,7 @@ The --cc option must be repeated for each user you want on the cc list.\n \tOnly necessary if --compose is also set. If --compose\n \tis not set, this will be prompted for.\n \n---signed-off-by-cc::\n---no-signed-off-by-cc::\n+--[no-]signed-off-by-cc::\n If this is set, add emails found in Signed-off-by: or Cc: lines to the\n cc list.\n Default is the value of 'sendemail.signedoffcc' configuration value;\n@@ -146,8 +144,7 @@ user is prompted for a password while the input is masked for privacy.\n \tOnly necessary if --compose is also set. If --compose\n \tis not set, this will be prompted for.\n \n---suppress-from::\n---no-suppress-from::\n+--[no-]suppress-from::\n If this is set, do not add the From: address to the cc: list.\n Default is the value of 'sendemail.suppressfrom' configuration value;\n if that is unspecified, default to --no-suppress-from.\n@@ -163,8 +160,7 @@ user is prompted for a password while the input is masked for privacy.\n \tif that is unspecified, default to 'self' if --suppress-from is\n \tspecified, as well as 'sob' if --no-signed-off-cc is specified.\n \n---thread::\n---no-thread::\n+--[no-]thread::\n \tIf this is set, the In-Reply-To header will be set on each email sent.\n \tIf disabled with \"--no-thread\", no emails will have the In-Reply-To\n \theader set.\n@@ -183,6 +179,15 @@ user is prompted for a password while the input is masked for privacy.\n \tvariable; if that is unspecified, choosing the envelope sender is left\n \tto your MTA.\n \n+--no-validate::\n+\tDon't perform any sanity checks on patches.\n+\tCurrently, validation means the following:\n++\n+--\n+\t\t*\tWarn of patches that contain lines longer than 998 characters; this\n+\t\t\tis due to SMTP limits as described by http://www.ietf.org/rfc/rfc2821.txt.\n+--\n+\n --to::\n \tSpecify the primary recipient of the emails generated.\n \tGenerally, this will be the upstream maintainer of the\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex d2fd899..9f56162 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -64,11 +64,11 @@ Options:\n Only used if --compose is also set. If --compose is not\n \t\t set, this will be prompted for.\n \n- --chain-reply-to If set, the replies will all be to the previous\n+ --[no-]chain-reply-to If set, the replies will all be to the previous\n email sent, rather than to the first email sent.\n Defaults to on.\n \n- --signed-off-cc Automatically add email addresses that appear in\n+ --[no-]signed-off-by-cc Automatically add email addresses that appear in\n Signed-off-by: or Cc: lines to the cc: list. Defaults to on.\n \n --identity The configuration identity, a subsection to prioritise over\n@@ -95,9 +95,9 @@ Options:\n \t\t 'cccmd' for the output of the cccmd, or 'all' to suppress\n \t\t all of these.\n \n- --suppress-from Suppress sending emails to yourself. Defaults to off.\n+ --[no-]suppress-from Suppress sending emails to yourself. Defaults to off.\n \n- --thread Specify that the \"In-Reply-To:\" header should be set on all\n+ --[no-]thread Specify that the \"In-Reply-To:\" header should be set on all\n emails. Defaults to on.\n \n --quiet\t Make git-send-email less verbose. One line per email\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91631","messageId":"1222375476-32911-2-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222375476-32911-1-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 2/6] Docs: send-email usage cleanup 1","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-25T20:44:32Z","receivedAt":"2008-09-25T20:44:32Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"The usage text mainly uses spaces to layout the text,\nso I replaced the few tabs that exit with spaces.\n\nSigned-off-by: Michael Witten \n---\n git-send-email.perl | 36 ++++++++++++++++++------------------\n 1 files changed, 18 insertions(+), 18 deletions(-)\n\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex 9f56162..d390364 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -51,25 +51,25 @@ Options:\n per file specific cc address entries\n \n --bcc Specify a list of email addresses that should be Bcc:\n-\t\t on all the emails.\n+ on all the emails.\n \n --compose Use \\$GIT_EDITOR, core.editor, \\$EDITOR, or \\$VISUAL to edit\n-\t\t an introductory message for the patch series.\n+ an introductory message for the patch series.\n \n --subject Specify the initial \"Subject:\" line.\n Only necessary if --compose is also set. If --compose\n-\t\t is not set, this will be prompted for.\n+ is not set, this will be prompted for.\n \n --in-reply-to Specify the first \"In-Reply-To:\" header line.\n Only used if --compose is also set. If --compose is not\n-\t\t set, this will be prompted for.\n+ set, this will be prompted for.\n \n --[no-]chain-reply-to If set, the replies will all be to the previous\n- email sent, rather than to the first email sent.\n- Defaults to on.\n+ email sent, rather than to the first email sent.\n+ Defaults to on.\n \n --[no-]signed-off-by-cc Automatically add email addresses that appear in\n- Signed-off-by: or Cc: lines to the cc: list. Defaults to on.\n+ Signed-off-by: or Cc: lines to the cc: list. Defaults to on.\n \n --identity The configuration identity, a subsection to prioritise over\n the default section.\n@@ -85,29 +85,29 @@ Options:\n --smtp-pass The password for SMTP-AUTH.\n \n --smtp-encryption Specify 'tls' for STARTTLS encryption, or 'ssl' for SSL.\n- Any other value disables the feature.\n+ Any other value disables the feature.\n \n --smtp-ssl Synonym for '--smtp-encryption=ssl'. Deprecated.\n \n --suppress-cc Suppress the specified category of auto-CC. The category\n-\t\t can be one of 'author' for the patch author, 'self' to\n-\t\t avoid copying yourself, 'sob' for Signed-off-by lines,\n-\t\t 'cccmd' for the output of the cccmd, or 'all' to suppress\n-\t\t all of these.\n+ can be one of 'author' for the patch author, 'self' to\n+ avoid copying yourself, 'sob' for Signed-off-by lines,\n+ 'cccmd' for the output of the cccmd, or 'all' to suppress\n+ all of these.\n \n --[no-]suppress-from Suppress sending emails to yourself. Defaults to off.\n \n --[no-]thread Specify that the \"In-Reply-To:\" header should be set on all\n- emails. Defaults to on.\n+ emails. Defaults to on.\n \n- --quiet\t Make git-send-email less verbose. One line per email\n- should be all that is output.\n+ --quiet Make git-send-email less verbose. One line per email\n+ should be all that is output.\n \n- --dry-run\t Do everything except actually send the emails.\n+ --dry-run Do everything except actually send the emails.\n \n- --envelope-sender\tSpecify the envelope sender used to send the emails.\n+ --envelope-sender Specify the envelope sender used to send the emails.\n \n- --no-validate\tDon't perform any sanity checks on patches.\n+ --no-validate Don't perform any sanity checks on patches.\n \n EOT\n \texit(1);\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91632","messageId":"1222375476-32911-3-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222375476-32911-2-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 3/6] Docs: send-email usage text much sexier","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-25T20:44:33Z","receivedAt":"2008-09-25T20:44:33Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"All of the descriptions are aligned, and no\nline is greater than 78 columns.\n\nSigned-off-by: Michael Witten \n---\n git-send-email.perl | 94 +++++++++++++++++++++++++++------------------------\n 1 files changed, 50 insertions(+), 44 deletions(-)\n\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex d390364..b86a3f3 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -40,74 +40,80 @@ sub usage {\n \tprint <...\n Options:\n- --from Specify the \"From:\" line of the email to be sent.\n+ --from Specify the \"From:\" line of the email to be sent.\n \n- --to Specify the primary \"To:\" line of the email.\n+ --to Specify the primary \"To:\" line of the email.\n \n- --cc Specify an initial \"Cc:\" list for the entire series\n- of emails.\n+ --cc Specify an initial \"Cc:\" list for the entire series\n+ of emails.\n \n- --cc-cmd Specify a command to execute per file which adds\n- per file specific cc address entries\n+ --cc-cmd Specify a command to execute per file which adds\n+ per file specific cc address entries\n \n- --bcc Specify a list of email addresses that should be Bcc:\n- on all the emails.\n+ --bcc Specify a list of email addresses that should be\n+ Bcc: on all the emails.\n \n- --compose Use \\$GIT_EDITOR, core.editor, \\$EDITOR, or \\$VISUAL to edit\n- an introductory message for the patch series.\n+ --compose Use \\$GIT_EDITOR, core.editor, \\$EDITOR, or\n+ \\$VISUAL to edit an introductory message for the\n+ patch series.\n \n- --subject Specify the initial \"Subject:\" line.\n- Only necessary if --compose is also set. If --compose\n- is not set, this will be prompted for.\n+ --subject Specify the initial \"Subject:\" line. Only necessary\n+ if --compose is also set. If --compose is not set,\n+ this will be prompted for.\n \n- --in-reply-to Specify the first \"In-Reply-To:\" header line.\n- Only used if --compose is also set. If --compose is not\n- set, this will be prompted for.\n+ --in-reply-to Specify the first \"In-Reply-To:\" header line. Only\n+ used if --compose is also set. If --compose is not\n+ set, this will be prompted for.\n \n- --[no-]chain-reply-to If set, the replies will all be to the previous\n- email sent, rather than to the first email sent.\n- Defaults to on.\n+ --[no-]chain-reply-to If set, the replies will all be to the previous\n+ email sent, rather than to the first email sent.\n+ Defaults to on.\n \n --[no-]signed-off-by-cc Automatically add email addresses that appear in\n- Signed-off-by: or Cc: lines to the cc: list. Defaults to on.\n+ Signed-off-by: or Cc: lines to the cc: list.\n+ Defaults to on.\n \n- --identity The configuration identity, a subsection to prioritise over\n- the default section.\n+ --identity The configuration identity to use.\n \n- --smtp-server If set, specifies the outgoing SMTP server to use.\n- Defaults to localhost. Port number can be specified here with\n- hostname:port format or by using --smtp-server-port option.\n+ --smtp-server If set, specifies the outgoing SMTP server to use.\n+ Defaults to localhost. Port number can be\n+ specified here with hostname:port format or by\n+ using --smtp-server-port option.\n \n- --smtp-server-port Specify a port on the outgoing SMTP server to connect to.\n+ --smtp-server-port Specify a port on the outgoing SMTP server to\n+ connect to.\n \n- --smtp-user The username for SMTP-AUTH.\n+ --smtp-user The username for SMTP-AUTH.\n \n- --smtp-pass The password for SMTP-AUTH.\n+ --smtp-pass The password for SMTP-AUTH.\n \n- --smtp-encryption Specify 'tls' for STARTTLS encryption, or 'ssl' for SSL.\n- Any other value disables the feature.\n+ --smtp-encryption Specify 'tls' for STARTTLS encryption, or 'ssl' for\n+ SSL. Any other value disables the feature.\n \n- --smtp-ssl Synonym for '--smtp-encryption=ssl'. Deprecated.\n+ --smtp-ssl Synonym for '--smtp-encryption=ssl'. Deprecated.\n \n- --suppress-cc Suppress the specified category of auto-CC. The category\n- can be one of 'author' for the patch author, 'self' to\n- avoid copying yourself, 'sob' for Signed-off-by lines,\n- 'cccmd' for the output of the cccmd, or 'all' to suppress\n- all of these.\n+ --suppress-cc Suppress the specified category of auto-CC. The\n+ category can be one of 'author' for the patch\n+ author, 'self' to avoid copying yourself, 'sob'\n+ for Signed-off-by lines, 'cccmd' for the output\n+ of the cccmd, or 'all' to suppress all of these.\n+ (Note: Linus uses 'self' and 'sob' interchangeably)\n \n- --[no-]suppress-from Suppress sending emails to yourself. Defaults to off.\n+ --[no-]suppress-from Suppress sending emails to yourself. Defaults to\n+ off.\n \n- --[no-]thread Specify that the \"In-Reply-To:\" header should be set on all\n- emails. Defaults to on.\n+ --[no-]thread Specify that the \"In-Reply-To:\" header should be\n+ set on all emails. Defaults to on.\n \n- --quiet Make git-send-email less verbose. One line per email\n- should be all that is output.\n+ --quiet Make git-send-email less verbose. One line per\n+ email should be all that is output.\n \n- --dry-run Do everything except actually send the emails.\n+ --dry-run Do everything except actually send the emails.\n \n- --envelope-sender Specify the envelope sender used to send the emails.\n+ --envelope-sender Specify the envelope sender used to send the\n+ emails.\n \n- --no-validate Don't perform any sanity checks on patches.\n+ --no-validate Don't perform any sanity checks on patches.\n \n EOT\n \texit(1);\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91629","messageId":"1222375476-32911-4-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222375476-32911-3-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 4/6] Docs: send-email: Option order the same in man page and usage text","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-25T20:44:34Z","receivedAt":"2008-09-25T20:44:34Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"The man page seems to have listed the options in alphabetical order,\nwhile the usage text used what I would consider an `intuitive' order\nthat groups related items and presents them in the order peopl think\nwhen writing an email.\n\nThe manual page has been updated to mirror this order.\n\nSigned-off-by: Michael Witten \n---\n Documentation/git-send-email.txt | 77 +++++++++++++++++++-------------------\n 1 files changed, 39 insertions(+), 38 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex 9b31626..6031eb7 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -24,10 +24,19 @@ OPTIONS\n -------\n The options available are:\n \n---bcc::\n-\tSpecify a \"Bcc:\" value for each email.\n+--from::\n+\tSpecify the sender of the emails. This will default to\n+\tthe value GIT_COMMITTER_IDENT, as returned by \"git var -l\".\n+\tThe user will still be prompted to confirm this entry.\n+\n+--to::\n+\tSpecify the primary recipient of the emails generated.\n+\tGenerally, this will be the upstream maintainer of the\n+\tproject involved.\n+\tDefault is the value of the 'sendemail.to' configuration value;\n+\tif that is unspecified, this will be prompted for.\n +\n-The --bcc option must be repeated for each user you want on the bcc list.\n+The --to option must be repeated for each user you want on the to list.\n \n --cc::\n \tSpecify a starting \"Cc:\" value for each email.\n@@ -40,23 +49,19 @@ The --cc option must be repeated for each user you want on the cc list.\n \tOutput of this command must be single email address per line.\n \tDefault is the value of 'sendemail.cccmd' configuration value.\n \n---[no-]chain-reply-to::\n-\tIf this is set, each email will be sent as a reply to the previous\n-\temail sent. If disabled with \"--no-chain-reply-to\", all emails after\n-\tthe first will be sent as replies to the first email sent. When using\n-\tthis, it is recommended that the first file given be an overview of the\n-\tentire patch series.\n-\tDefault is the value of the 'sendemail.chainreplyto' configuration\n-\tvalue; if that is unspecified, default to --chain-reply-to.\n+--bcc::\n+\tSpecify a \"Bcc:\" value for each email.\n++\n+The --bcc option must be repeated for each user you want on the bcc list.\n \n --compose::\n \tUse $GIT_EDITOR, core.editor, $VISUAL, or $EDITOR to edit an\n \tintroductory message for the patch series.\n \n---from::\n-\tSpecify the sender of the emails. This will default to\n-\tthe value GIT_COMMITTER_IDENT, as returned by \"git var -l\".\n-\tThe user will still be prompted to confirm this entry.\n+--subject::\n+\tSpecify the initial subject of the email thread.\n+\tOnly necessary if --compose is also set. If --compose\n+\tis not set, this will be prompted for.\n \n --in-reply-to::\n \tSpecify the contents of the first In-Reply-To header.\n@@ -65,16 +70,21 @@ The --cc option must be repeated for each user you want on the cc list.\n \tOnly necessary if --compose is also set. If --compose\n \tis not set, this will be prompted for.\n \n+--[no-]chain-reply-to::\n+\tIf this is set, each email will be sent as a reply to the previous\n+\temail sent. If disabled with \"--no-chain-reply-to\", all emails after\n+\tthe first will be sent as replies to the first email sent. When using\n+\tthis, it is recommended that the first file given be an overview of the\n+\tentire patch series.\n+\tDefault is the value of the 'sendemail.chainreplyto' configuration\n+\tvalue; if that is unspecified, default to --chain-reply-to.\n+\n --[no-]signed-off-by-cc::\n If this is set, add emails found in Signed-off-by: or Cc: lines to the\n cc list.\n Default is the value of 'sendemail.signedoffcc' configuration value;\n if that is unspecified, default to --signed-off-by-cc.\n \n---quiet::\n-\tMake git-send-email less verbose. One line per email should be\n-\tall that is output.\n-\n --identity::\n \tA configuration identity. When given, causes values in the\n \t'sendemail.' subsection to take precedence over\n@@ -139,16 +149,6 @@ user is prompted for a password while the input is masked for privacy.\n --smtp-ssl::\n \tLegacy alias for '--smtp-encryption=ssl'.\n \n---subject::\n-\tSpecify the initial subject of the email thread.\n-\tOnly necessary if --compose is also set. If --compose\n-\tis not set, this will be prompted for.\n-\n---[no-]suppress-from::\n- If this is set, do not add the From: address to the cc: list.\n- Default is the value of 'sendemail.suppressfrom' configuration value;\n- if that is unspecified, default to --no-suppress-from.\n-\n --suppress-cc::\n \tSpecify an additional category of recipients to suppress the\n \tauto-cc of. 'self' will avoid including the sender, 'author' will\n@@ -159,6 +159,12 @@ user is prompted for a password while the input is masked for privacy.\n \tDefault is the value of 'sendemail.suppresscc' configuration value;\n \tif that is unspecified, default to 'self' if --suppress-from is\n \tspecified, as well as 'sob' if --no-signed-off-cc is specified.\n+\t(Note: Linus uses 'self' and 'sob' interchangeably).\n+\n+--[no-]suppress-from::\n+ If this is set, do not add the From: address to the cc: list.\n+ Default is the value of 'sendemail.suppressfrom' configuration value;\n+ if that is unspecified, default to --no-suppress-from.\n \n --[no-]thread::\n \tIf this is set, the In-Reply-To header will be set on each email sent.\n@@ -167,6 +173,10 @@ user is prompted for a password while the input is masked for privacy.\n \tDefault is the value of the 'sendemail.thread' configuration value;\n \tif that is unspecified, default to --thread.\n \n+--quiet::\n+\tMake git-send-email less verbose. One line per email should be\n+\tall that is output.\n+\n --dry-run::\n \tDo everything except actually send the emails.\n \n@@ -188,15 +198,6 @@ user is prompted for a password while the input is masked for privacy.\n \t\t\tis due to SMTP limits as described by http://www.ietf.org/rfc/rfc2821.txt.\n --\n \n---to::\n-\tSpecify the primary recipient of the emails generated.\n-\tGenerally, this will be the upstream maintainer of the\n-\tproject involved.\n-\tDefault is the value of the 'sendemail.to' configuration value;\n-\tif that is unspecified, this will be prompted for.\n-+\n-The --to option must be repeated for each user you want on the to list.\n-\n \n CONFIGURATION\n -------------\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91628","messageId":"1222375476-32911-5-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222375476-32911-4-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 5/6] Docs: send-email: change --no-validate to boolean --[no-]validate","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-25T20:44:35Z","receivedAt":"2008-09-25T20:44:35Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"There is also now a configuration variable:\n\n sendemail[.].validate\n\nSigned-off-by: Michael Witten \n---\n Documentation/git-send-email.txt | 9 +++++++--\n git-send-email.perl | 10 ++++++----\n 2 files changed, 13 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex 6031eb7..fe53e9b 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -189,14 +189,16 @@ user is prompted for a password while the input is masked for privacy.\n \tvariable; if that is unspecified, choosing the envelope sender is left\n \tto your MTA.\n \n---no-validate::\n-\tDon't perform any sanity checks on patches.\n+--[no-]validate::\n+\tPerform sanity checks on patches.\n \tCurrently, validation means the following:\n +\n --\n \t\t*\tWarn of patches that contain lines longer than 998 characters; this\n \t\t\tis due to SMTP limits as described by http://www.ietf.org/rfc/rfc2821.txt.\n --\n++\n+This option can be set with sendemail[.].validate; default to --validate\n \n \n CONFIGURATION\n@@ -249,6 +251,9 @@ sendemail.smtpencryption::\n sendemail.smtpssl::\n \tLegacy boolean that sets 'smtpencryption=ssl' if enabled.\n \n+sendemail.validate::\n+ Boolean override for --[no-]validate.\n+\n Author\n ------\n Written by Ryan Anderson \ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex b86a3f3..8bc204f 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -113,7 +113,8 @@ Options:\n --envelope-sender Specify the envelope sender used to send the\n emails.\n \n- --no-validate Don't perform any sanity checks on patches.\n+ --[no-]validate Perform any sanity checks on patches.\n+ Default to on.\n \n EOT\n \texit(1);\n@@ -195,7 +196,7 @@ my ($quiet, $dry_run) = (0, 0);\n my ($thread, $chain_reply_to, $suppress_from, $signed_off_cc, $cc_cmd);\n my ($smtp_server, $smtp_server_port, $smtp_authuser, $smtp_encryption);\n my ($identity, $aliasfiletype, @alias_files, @smtp_host_parts);\n-my ($no_validate);\n+my ($validate);\n my (@suppress_cc);\n \n my %config_bool_settings = (\n@@ -203,6 +204,7 @@ my %config_bool_settings = (\n \"chainreplyto\" => [\\$chain_reply_to, 1],\n \"suppressfrom\" => [\\$suppress_from, undef],\n \"signedoffcc\" => [\\$signed_off_cc, undef],\n+ \"validate\" => [\\$validate, 1],\n );\n \n my %config_settings = (\n@@ -269,7 +271,7 @@ my $rc = GetOptions(\"sender|from=s\" => \\$sender,\n \t\t \"dry-run\" => \\$dry_run,\n \t\t \"envelope-sender=s\" => \\$envelope_sender,\n \t\t \"thread!\" => \\$thread,\n-\t\t \"no-validate\" => \\$no_validate,\n+\t\t \"validate!\" => \\$validate,\n \t );\n \n unless ($rc) {\n@@ -422,7 +424,7 @@ for my $f (@ARGV) {\n \t}\n }\n \n-if (!$no_validate) {\n+if ($validate) {\n \tforeach my $f (@files) {\n \t\tunless (-p $f) {\n \t\t\tmy $error = validate_patch($f);\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91630","messageId":"1222375476-32911-6-git-send-email-mfwitten@mit.edu","threadId":"15623","inReplyTo":"1222375476-32911-5-git-send-email-mfwitten@mit.edu","subject":"[PATCH RFC 6/6] Docs: send-email: --chain_reply_to -> --[no-]chain-reply-to","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-25T20:44:36Z","receivedAt":"2008-09-25T20:44:36Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"Signed-off-by: Michael Witten \n---\n Documentation/git-send-email.txt | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex fe53e9b..76a525a 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -228,7 +228,7 @@ sendemail.bcc::\n \tEmail address (or alias) to always bcc.\n \n sendemail.chainreplyto::\n-\tBoolean value specifying the default to the '--chain_reply_to'\n+\tBoolean value specifying the default to the '--[no-]chain-reply-to'\n \tparameter.\n \n sendemail.smtpserver::\n-- \n1.6.0.2.302.ge6cbd1\n"},{"id":"91716","messageId":"20080926224032.GA24817@coredump.intra.peff.net","threadId":"15623","inReplyTo":"1222375476-32911-4-git-send-email-mfwitten@mit.edu","subject":"Re: [PATCH RFC 4/6] Docs: send-email: Option order the same in man page and usage text","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2008-09-26T22:40:32Z","receivedAt":"2008-09-26T22:40:32Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Sep 25, 2008 at 03:44:34PM -0500, Michael Witten wrote:\n\n> The man page seems to have listed the options in alphabetical order,\n> while the usage text used what I would consider an `intuitive' order\n> that groups related items and presents them in the order peopl think\n> when writing an email.\n> \n> The manual page has been updated to mirror this order.\n\nI think overall this serious is well-intentioned the results look sane\nto me (I diffed the manpage and usage text before and after).\n\nMy only concerns are:\n\n 4/6: I am not sure about making the order of options the same, as the\n two formats serve different purposes. I think \"git send-email\n --foo\" should present the options based on commonality of use.\n You clearly got the usage wrong, so I think it is helping you to\n figure out quickly what you probably meant.\n\n The manpage, on the other hand, is a comprehensive reference and\n so should probably be alphabetized for easy reading.\n\n That being said, I find the length of the current usage statement\n somewhat ridiculous. I wonder if it would be better still to just\n mention in the usage the overall syntax (i.e., \"send-email\n [options] \", the few most common options (with\n minimal description), and then refer to the manpage for complete.\n With the goal of taking no more than 10 or so lines (the current\n output is 71 lines!).\n\n 5/6: This isn't really a documentation patch, as it adds a new config\n option. I think the result is reasonable, but it was a surprise\n after reading the rest of the series.\n\n I was also going to complain that there was no entry in\n Documentation/config.txt for the new option, but I see that none\n of the sendemail options are there. Maybe while you are working\n on this, it is worth factoring them out to send-email-options.txt\n and including it in both git-send-email.txt and config.txt.\n\n-Peff\n"},{"id":"91724","messageId":"D779B9F2-9CDF-4301-89D9-2F63A5B7BBA0@mit.edu","threadId":"15623","inReplyTo":"20080926224032.GA24817@coredump.intra.peff.net","subject":"Re: [PATCH RFC 4/6] Docs: send-email: Option order the same in man page and usage text","fromName":"Michael Witten","fromEmail":"mfwitten@mit.edu","sentAt":"2008-09-27T01:13:44Z","receivedAt":"2008-09-27T01:13:44Z","isPatch":true,"sender":{"key":"mfwitten@gmail.com","avatar":"https://avatars.githubusercontent.com/u/597101?v=4"},"body":"\nOn 26 Sep 2008, at 5:40 PM, Jeff King wrote:\n\n> I think overall this serious is well-intentioned the results look sane\n> to me (I diffed the manpage and usage text before and after).\n>\n> My only concerns are:\n>\n> 4/6: ...\n> The manpage, on the other hand, is a comprehensive reference and\n> so should probably be alphabetized for easy reading.\n\nI can buy it. I'll change it back.\n\n> ...\n> That being said, I find the length of the current usage \n> statement\n> somewhat ridiculous.\n\nI'll take care of it.\n\n> 5/6: This isn't really a documentation patch, as it adds a new config\n> option. I think the result is reasonable, but it was a surprise\n> after reading the rest of the series.\n\nI didn't mean to mark those as Docs. I'll fix that too.\n\n> I was also going to complain that there was no entry in\n> Documentation/config.txt for the new option, but I see that none\n> of the sendemail options are there. Maybe while you are working\n> on this, it is worth factoring them out to send-email- \n> options.txt\n> and including it in both git-send-email.txt and config.txt.\n\nGrumble grumble grumble... erm.. hmm.. grrr.. ok.\n"}]}