threads / patch / 9897

patchgit-send-email: Add a --suppress-all option

Subject: [PATCH 1/1] git-send-email: Add a --suppress-all option

## tl;dr

6 messages between Sep 17, 2007 and Sep 18, 2007. Diffs are folded; open one to read it.

replies: 5people: 4as markdown or json

Felipe Balbi· Sep 17, 2007, 18:33 UTC · lore
From: Felipe Balbi <felipe.lima@indt.org.br>

This patch adds a --suppress-all option to avoid sending emails to everybody but the ones listed by --to option.

Signed-off-by: Felipe Balbi <felipe.lima@indt.org.br>
---
 git-send-email.perl |   16 ++++++++++++----
 1 files changed, 12 insertions(+), 4 deletions(-)
Show changes to git-send-email.perl +12 −4
diff --git a/git-send-email.perl b/git-send-email.perl
index d8319d4..e845c5f 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -88,6 +88,9 @@ Options:
    --suppress-from Suppress sending emails to yourself if your address
                   appears in a From: line. Defaults to off.
 
+   --suppress-all Suppress sending emails to everybody but addresses
+		  listed by --to option.
+
    --thread       Specify that the "In-Reply-To:" header should be set on all
                   emails. Defaults to on.
 
@@ -171,7 +174,7 @@ if ($@) {
 my ($quiet, $dry_run) = (0, 0);
 
 # Variables with corresponding config settings
-my ($thread, $chain_reply_to, $suppress_from, $signed_off_cc, $cc_cmd);
+my ($thread, $chain_reply_to, $suppress_from, $suppress_all, $signed_off_cc, $cc_cmd);
 my ($smtp_server, $smtp_authuser, $smtp_authpass, $smtp_ssl);
 my ($identity, $aliasfiletype, @alias_files);
 
@@ -179,6 +182,7 @@ my %config_bool_settings = (
     "thread" => [\$thread, 1],
     "chainreplyto" => [\$chain_reply_to, 1],
     "suppressfrom" => [\$suppress_from, 0],
+    "suppressall" => [\$suppress_all, 0],
     "signedoffcc" => [\$signed_off_cc, 1],
     "smtpssl" => [\$smtp_ssl, 0],
 );
@@ -212,6 +216,7 @@ my $rc = GetOptions("sender|from=s" => \$sender,
 		    "quiet" => \$quiet,
 		    "cc-cmd=s" => \$cc_cmd,
 		    "suppress-from!" => \$suppress_from,
+		    "suppress-all!" => \$suppress_all,
 		    "signed-off-cc|signed-off-by-cc!" => \$signed_off_cc,
 		    "dry-run" => \$dry_run,
 		    "envelope-sender=s" => \$envelope_sender,
@@ -662,8 +667,11 @@ foreach my $t (@files) {
 					$subject = $1;
 
 				} elsif (/^(Cc|From):\s+(.*)$/) {
-					if (unquote_rfc2047($2) eq $sender) {
-						next if ($suppress_from);
+					if (unquote_rfc2047($2)) {
+						next if ($suppress_all);
+					}
+					elsif (unquote_rfc2047($2) eq $sender) {
+						next if ($suppress_from|$suppress_all);
 					}
 					elsif ($1 eq 'From') {
 						$author = unquote_rfc2047($2);
@@ -700,7 +708,7 @@ foreach my $t (@files) {
 			}
 		} else {
 			$message .=  $_;
-			if (/^(Signed-off-by|Cc): (.*)$/i && $signed_off_cc) {
+			if (/^(Signed-off-by|Cc): (.*)$/i && $signed_off_cc && !$suppress_all) {
 				my $c = $2;
 				chomp $c;
 				push @cc, $c;
-- 
1.5.3.1.91.gd3392
Andreas Ericsson· Sep 18, 2007, 07:40 UTC · re: Felipe Balbi · lore

Re: [PATCH 1/1] git-send-email: Add a --suppress-all option

Felipe Balbi wrote:
Show 5 quoted lines
> From: Felipe Balbi <felipe.lima@indt.org.br>
> 
> This patch adds a --suppress-all option to avoid sending emails
> to everybody but the ones listed by --to option.
> 

To my minds eye, --suppress-all is equivalent to --dry-run. Could you rename it to "--cc-nobody" or some such?

On a side-note, I've never really understood why git-send-email *by default* sends to a bazillion people. Does anybody ever use it without suppressing most of the CC targets?

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231
J. Bruce Fields· Sep 18, 2007, 13:22 UTC · re: Andreas Ericsson · lore

Re: [PATCH 1/1] git-send-email: Add a --suppress-all option

On Tue, Sep 18, 2007 at 09:40:55AM +0200, Andreas Ericsson wrote:
Show 12 quoted lines
> Felipe Balbi wrote:
>> From: Felipe Balbi <felipe.lima@indt.org.br>
>> This patch adds a --suppress-all option to avoid sending emails
>> to everybody but the ones listed by --to option.
>
> To my minds eye, --suppress-all is equivalent to --dry-run. Could you
> rename it to "--cc-nobody" or some such?
>
> On a side-note, I've never really understood why git-send-email *by 
> default*
> sends to a bazillion people. Does anybody ever use it without suppressing
> most of the CC targets?

Yes. I never suppress the cc's. The cc-everyone thing is standard on the kernel mailing lists.

The one exception is if I'm just sending the series to myself as a test.
--b.
Felipe Balbi· Sep 18, 2007, 13:29 UTC · re: J. Bruce Fields · lore

Re: [PATCH 1/1] git-send-email: Add a --suppress-all option

Hi,
On 9/18/07, J. Bruce Fields <bfields@fieldses.org> wrote:
Show 18 quoted lines
> On Tue, Sep 18, 2007 at 09:40:55AM +0200, Andreas Ericsson wrote:
> > Felipe Balbi wrote:
> >> From: Felipe Balbi <felipe.lima@indt.org.br>
> >> This patch adds a --suppress-all option to avoid sending emails
> >> to everybody but the ones listed by --to option.
> >
> > To my minds eye, --suppress-all is equivalent to --dry-run. Could you
> > rename it to "--cc-nobody" or some such?
> >
> > On a side-note, I've never really understood why git-send-email *by
> > default*
> > sends to a bazillion people. Does anybody ever use it without suppressing
> > most of the CC targets?
>
> Yes.  I never suppress the cc's.  The cc-everyone thing is standard on
> the kernel mailing lists.
>
> The one exception is if I'm just sending the series to myself as a test.
And that the motivation for this patch :-p

Internally (in my company, where I work), we send patches to each other for code review before sending it publicly.

>
> --b.
>
-- 
Best Regards,

Felipe Balbi
felipebalbi@users.sourceforge.net
David Kastrup· Sep 18, 2007, 14:15 UTC · re: J. Bruce Fields · lore

Re: [PATCH 1/1] git-send-email: Add a --suppress-all option

"J. Bruce Fields" <bfields@fieldses.org> writes:
Show 18 quoted lines
> On Tue, Sep 18, 2007 at 09:40:55AM +0200, Andreas Ericsson wrote:
>> Felipe Balbi wrote:
>>> From: Felipe Balbi <felipe.lima@indt.org.br>
>>> This patch adds a --suppress-all option to avoid sending emails
>>> to everybody but the ones listed by --to option.
>>
>> To my minds eye, --suppress-all is equivalent to --dry-run. Could you
>> rename it to "--cc-nobody" or some such?
>>
>> On a side-note, I've never really understood why git-send-email *by 
>> default*
>> sends to a bazillion people. Does anybody ever use it without suppressing
>> most of the CC targets?
>
> Yes.  I never suppress the cc's.  The cc-everyone thing is standard on
> the kernel mailing lists.
>
> The one exception is if I'm just sending the series to myself as a test.
Wouldn't --no-cc be a nicer option name?
-- 
David Kastrup
Felipe Balbi· Sep 18, 2007, 14:39 UTC · re: David Kastrup · lore

Re: [PATCH 1/1] git-send-email: Add a --suppress-all option

Hi,
On 9/18/07, David Kastrup <dak@gnu.org> wrote:
Show 22 quoted lines
> "J. Bruce Fields" <bfields@fieldses.org> writes:
>
> > On Tue, Sep 18, 2007 at 09:40:55AM +0200, Andreas Ericsson wrote:
> >> Felipe Balbi wrote:
> >>> From: Felipe Balbi <felipe.lima@indt.org.br>
> >>> This patch adds a --suppress-all option to avoid sending emails
> >>> to everybody but the ones listed by --to option.
> >>
> >> To my minds eye, --suppress-all is equivalent to --dry-run. Could you
> >> rename it to "--cc-nobody" or some such?
> >>
> >> On a side-note, I've never really understood why git-send-email *by
> >> default*
> >> sends to a bazillion people. Does anybody ever use it without suppressing
> >> most of the CC targets?
> >
> > Yes.  I never suppress the cc's.  The cc-everyone thing is standard on
> > the kernel mailing lists.
> >
> > The one exception is if I'm just sending the series to myself as a test.
>
> Wouldn't --no-cc be a nicer option name?

It'll probably conflict if I don't wanna CC patch's author nor Signed-off-by but DO want to CC somebody special like the maintainer.

the option is changed to --cc-nobody and it's already resent. :-)
Show 9 quoted lines
>
> --
> David Kastrup
>
> -
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
-- 
Best Regards,

Felipe Balbi
felipebalbi@users.sourceforge.net

← back to recent threads