{"thread":{"id":"26003","subject":"[PATCH] git-send-email: Accept -n as a synonym for --dry-run","startedAt":"2010-12-09T04:44:38Z","lastAt":"2010-12-13T21:47:48Z","messageCount":9,"participants":["Alejandro R. Sedeño","Thomas Rast","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"157667","messageId":"1291869878-19645-1-git-send-email-asedeno@mit.edu","threadId":"26003","inReplyTo":null,"subject":"[PATCH] git-send-email: Accept -n as a synonym for --dry-run","fromName":"Alejandro R. Sedeño","fromEmail":"asedeno@mit.edu","sentAt":"2010-12-09T04:44:38Z","receivedAt":"2010-12-09T04:44:38Z","isPatch":true,"sender":{"key":"asedeno@mit.edu","avatar":"https://avatars.githubusercontent.com/u/28302?v=4"},"body":"git-send-email is not currently using -n for anything else, and it\nseems unlikely we will want to use it to mean anything else in the\nfuture, so add it as an alias for convenience.\n\nSigned-off-by: Alejandro R. Sedeño <asedeno@mit.edu>\n---\n git-send-email.perl |    4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex 76565de..7e3df9a 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -85,7 +85,7 @@ git send-email [options] <file | directory | rev-list options >\n     --confirm               <str>  * Confirm recipients before sending;\n                                      auto, cc, compose, always, or never.\n     --quiet                        * Output one line of info per email.\n-    --dry-run                      * Don't actually send the emails.\n+    -n, --dry-run                  * Don't actually send the emails.\n     --[no-]validate                * Perform patch sanity checks. Default on.\n     --[no-]format-patch            * understand any non optional arguments as\n                                      `git format-patch` ones.\n@@ -304,7 +304,7 @@ my $rc = GetOptions(\"sender|from=s\" => \\$sender,\n \t\t    \"suppress-cc=s\" => \\@suppress_cc,\n \t\t    \"signed-off-cc|signed-off-by-cc!\" => \\$signed_off_by_cc,\n \t\t    \"confirm=s\" => \\$confirm,\n-\t\t    \"dry-run\" => \\$dry_run,\n+\t\t    \"dry-run|n\" => \\$dry_run,\n \t\t    \"envelope-sender=s\" => \\$envelope_sender,\n \t\t    \"thread!\" => \\$thread,\n \t\t    \"validate!\" => \\$validate,\n-- \n1.7.3.3\n"},{"id":"157675","messageId":"4D007986.60809@mit.edu","threadId":"26003","inReplyTo":"1291869878-19645-1-git-send-email-asedeno@mit.edu","subject":"Re: [PATCH] git-send-email: Accept -n as a synonym for --dry-run","fromName":"Alejandro R. Sedeño","fromEmail":"asedeno@mit.edu","sentAt":"2010-12-09T06:39:02Z","receivedAt":"2010-12-09T06:39:02Z","isPatch":true,"sender":{"key":"asedeno@mit.edu","avatar":"https://avatars.githubusercontent.com/u/28302?v=4"},"body":"I noticed I forgot to make the corresponding documentation change. I'll\ninclude it in the next version of this patch, though I'm waiting to see\nif there's any other feedback first.\n\n-Alejandro\n"},{"id":"157687","messageId":"201012091435.02669.trast@student.ethz.ch","threadId":"26003","inReplyTo":"1291869878-19645-1-git-send-email-asedeno@mit.edu","subject":"Re: [PATCH] git-send-email: Accept -n as a synonym for --dry-run","fromName":"Thomas Rast","fromEmail":"trast@student.ethz.ch","sentAt":"2010-12-09T13:35:02Z","receivedAt":"2010-12-09T13:35:02Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Alejandro R. Sedeño wrote:\n> diff --git a/git-send-email.perl b/git-send-email.perl\n> index 76565de..7e3df9a 100755\n> --- a/git-send-email.perl\n> +++ b/git-send-email.perl\n> @@ -85,7 +85,7 @@ git send-email [options] <file | directory | rev-list options >\n>      --confirm               <str>  * Confirm recipients before sending;\n>                                       auto, cc, compose, always, or never.\n>      --quiet                        * Output one line of info per email.\n> -    --dry-run                      * Don't actually send the emails.\n> +    -n, --dry-run                  * Don't actually send the emails.\n>      --[no-]validate                * Perform patch sanity checks. Default on.\n>      --[no-]format-patch            * understand any non optional arguments as\n>                                       `git format-patch` ones.\n\nGood change by itself, but this is the first short option for\ngit-send-email.  Maybe --force should also get its analogous -f alias?\nAny others?\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"157699","messageId":"4D0101F6.9060602@mit.edu","threadId":"26003","inReplyTo":"201012091435.02669.trast@student.ethz.ch","subject":"Re: [PATCH] git-send-email: Accept -n as a synonym for --dry-run","fromName":"Alejandro R. Sedeño","fromEmail":"asedeno@mit.edu","sentAt":"2010-12-09T16:21:10Z","receivedAt":"2010-12-09T16:21:10Z","isPatch":true,"sender":{"key":"asedeno@mit.edu","avatar":"https://avatars.githubusercontent.com/u/28302?v=4"},"body":"On 12/09/2010 08:35 AM, Thomas Rast wrote:\n> Alejandro R. Sedeño wrote:\n>> diff --git a/git-send-email.perl b/git-send-email.perl\n>> index 76565de..7e3df9a 100755\n>> --- a/git-send-email.perl\n>> +++ b/git-send-email.perl\n>> @@ -85,7 +85,7 @@ git send-email [options] <file | directory | rev-list options >\n>>      --confirm               <str>  * Confirm recipients before sending;\n>>                                       auto, cc, compose, always, or never.\n>>      --quiet                        * Output one line of info per email.\n>> -    --dry-run                      * Don't actually send the emails.\n>> +    -n, --dry-run                  * Don't actually send the emails.\n>>      --[no-]validate                * Perform patch sanity checks. Default on.\n>>      --[no-]format-patch            * understand any non optional arguments as\n>>                                       `git format-patch` ones.\n> \n> Good change by itself, but this is the first short option for\n> git-send-email.  Maybe --force should also get its analogous -f alias?\n> Any others?\n\nSounds good to me, though I notice that --force isn't in git-send-email's\ndocumentation at all.\n\nIf we're expanding this patch[set] to include other short options, -q for\n--quiet makes sense to me as well.\n\nOther suggestions?\n\n-Alejandro\n"},{"id":"157833","messageId":"1292006656-1264-1-git-send-email-asedeno@mit.edu","threadId":"26003","inReplyTo":"1291869878-19645-1-git-send-email-asedeno@mit.edu","subject":"git-send-email: add some short options and update documentation","fromName":"Alejandro R. Sedeño","fromEmail":"asedeno@mit.edu","sentAt":"2010-12-10T18:44:14Z","receivedAt":"2010-12-10T18:44:14Z","isPatch":false,"sender":{"key":"asedeno@mit.edu","avatar":"https://avatars.githubusercontent.com/u/28302?v=4"},"body":"Add short options that were mentioned in the thread to git-send-email,\nspecifically:\n\n-n for --dry-run\n-f for --force\n-q for --quiet\n\nSince --force didn't have documentation in\nDocumentation/git-send-email.txt, a separate commit adds some\nfirst.\n\n[PATCH 1/2] Add --force to git-send-email documentation\n[PATCH 2/2] git-send-email: Add some short options\n\nDocumentation/git-send-email.txt |    6 ++++++\ngit-send-email.perl              |   12 ++++++------\n2 files changed, 12 insertions(+), 6 deletions(-)\n"},{"id":"157832","messageId":"1292006656-1264-2-git-send-email-asedeno@mit.edu","threadId":"26003","inReplyTo":"1291869878-19645-1-git-send-email-asedeno@mit.edu","subject":"[PATCH 1/2] Add --force to git-send-email documentation","fromName":"Alejandro R. Sedeño","fromEmail":"asedeno@mit.edu","sentAt":"2010-12-10T18:44:15Z","receivedAt":"2010-12-10T18:44:15Z","isPatch":true,"sender":{"key":"asedeno@mit.edu","avatar":"https://avatars.githubusercontent.com/u/28302?v=4"},"body":"Signed-off-by: Alejandro R. Sedeño <asedeno@mit.edu>\n---\n Documentation/git-send-email.txt |    3 +++\n 1 files changed, 3 insertions(+), 0 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex ebc024a..7ec9dab 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -322,6 +322,9 @@ have been specified, in which case default to 'compose'.\n Default is the value of 'sendemail.validate'; if this is not set,\n default to '--validate'.\n \n+--force::\n+\tSend emails even if safety checks would prevent it.\n+\n \n CONFIGURATION\n -------------\n-- \n1.7.3.3\n"},{"id":"157834","messageId":"1292006656-1264-3-git-send-email-asedeno@mit.edu","threadId":"26003","inReplyTo":"1291869878-19645-1-git-send-email-asedeno@mit.edu","subject":"[PATCH 2/2] git-send-email: Add some short options","fromName":"Alejandro R. Sedeño","fromEmail":"asedeno@mit.edu","sentAt":"2010-12-10T18:44:16Z","receivedAt":"2010-12-10T18:44:16Z","isPatch":true,"sender":{"key":"asedeno@mit.edu","avatar":"https://avatars.githubusercontent.com/u/28302?v=4"},"body":"* Accept -n as a synonym for --dry-run\n* Accept -f as a synonym for --force\n* Accept -q as a synonym for --quiet\n\nSigned-off-by: Alejandro R. Sedeño <asedeno@mit.edu>\n---\n Documentation/git-send-email.txt |    3 +++\n git-send-email.perl              |   12 ++++++------\n 2 files changed, 9 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt\nindex 7ec9dab..2d0faf2 100644\n--- a/Documentation/git-send-email.txt\n+++ b/Documentation/git-send-email.txt\n@@ -297,6 +297,7 @@ Default is the value of 'sendemail.confirm' configuration value; if that\n is unspecified, default to 'auto' unless any of the suppress options\n have been specified, in which case default to 'compose'.\n \n+-n::\n --dry-run::\n \tDo everything except actually send the emails.\n \n@@ -306,6 +307,7 @@ have been specified, in which case default to 'compose'.\n \tor as a file name ('--no-format-patch'). By default, when such a conflict\n \toccurs, git send-email will fail.\n \n+-q::\n --quiet::\n \tMake git-send-email less verbose.  One line per email should be\n \tall that is output.\n@@ -322,6 +324,7 @@ have been specified, in which case default to 'compose'.\n Default is the value of 'sendemail.validate'; if this is not set,\n default to '--validate'.\n \n+-f::\n --force::\n \tSend emails even if safety checks would prevent it.\n \ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex 76565de..ede7835 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -84,12 +84,12 @@ git send-email [options] <file | directory | rev-list options >\n   Administering:\n     --confirm               <str>  * Confirm recipients before sending;\n                                      auto, cc, compose, always, or never.\n-    --quiet                        * Output one line of info per email.\n-    --dry-run                      * Don't actually send the emails.\n+    -q, --quiet                    * Output one line of info per email.\n+    -n, --dry-run                  * Don't actually send the emails.\n     --[no-]validate                * Perform patch sanity checks. Default on.\n     --[no-]format-patch            * understand any non optional arguments as\n                                      `git format-patch` ones.\n-    --force                        * Send even if safety checks would prevent it.\n+    -f, --force                    * Send even if safety checks would prevent it.\n \n EOT\n \texit(1);\n@@ -298,19 +298,19 @@ my $rc = GetOptions(\"sender|from=s\" => \\$sender,\n \t\t    \"identity=s\" => \\$identity,\n \t\t    \"annotate\" => \\$annotate,\n \t\t    \"compose\" => \\$compose,\n-\t\t    \"quiet\" => \\$quiet,\n+\t\t    \"quiet|q\" => \\$quiet,\n \t\t    \"cc-cmd=s\" => \\$cc_cmd,\n \t\t    \"suppress-from!\" => \\$suppress_from,\n \t\t    \"suppress-cc=s\" => \\@suppress_cc,\n \t\t    \"signed-off-cc|signed-off-by-cc!\" => \\$signed_off_by_cc,\n \t\t    \"confirm=s\" => \\$confirm,\n-\t\t    \"dry-run\" => \\$dry_run,\n+\t\t    \"dry-run|n\" => \\$dry_run,\n \t\t    \"envelope-sender=s\" => \\$envelope_sender,\n \t\t    \"thread!\" => \\$thread,\n \t\t    \"validate!\" => \\$validate,\n \t\t    \"format-patch!\" => \\$format_patch,\n \t\t    \"8bit-encoding=s\" => \\$auto_8bit_encoding,\n-\t\t    \"force\" => \\$force,\n+\t\t    \"force|f\" => \\$force,\n \t );\n \n unless ($rc) {\n-- \n1.7.3.3\n"},{"id":"157996","messageId":"4D067E19.1060904@mit.edu","threadId":"26003","inReplyTo":"1292006656-1264-1-git-send-email-asedeno@mit.edu","subject":"Re: git-send-email: add some short options and update documentation","fromName":"Alejandro R. Sedeño","fromEmail":"asedeno@mit.edu","sentAt":"2010-12-13T20:12:09Z","receivedAt":"2010-12-13T20:12:09Z","isPatch":false,"sender":{"key":"asedeno@mit.edu","avatar":"https://avatars.githubusercontent.com/u/28302?v=4"},"body":"On 12/10/2010 01:44 PM, Alejandro R. Sedeño wrote:\n> Add short options that were mentioned in the thread to git-send-email,\n> specifically:\n> \n> -n for --dry-run\n> -f for --force\n> -q for --quiet\n> \n> Since --force didn't have documentation in\n> Documentation/git-send-email.txt, a separate commit adds some\n> first.\n> \n> [PATCH 1/2] Add --force to git-send-email documentation\n\nI see this has been pulled into master. Thanks, Junio.\n\n> [PATCH 2/2] git-send-email: Add some short options\n\nAny more comments on this?\n\n-Alejandro\n"},{"id":"158002","messageId":"7vtyihxs3f.fsf@alter.siamese.dyndns.org","threadId":"26003","inReplyTo":"4D067E19.1060904@mit.edu","subject":"Re: git-send-email: add some short options and update documentation","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2010-12-13T21:47:48Z","receivedAt":"2010-12-13T21:47:48Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Alejandro R. Sedeño\" <asedeno@mit.edu> writes:\n\n>> [PATCH 2/2] git-send-email: Add some short options\n>\n> Any more comments on this?\n\nNot from me, other than that I am not overly interested myself.\n"}]}