On Fri, Mar 6, 2015 at 4:59 PM, Soren Brinkmann
soren.brinkm...@xilinx.com wrote:
Add email addresses from additional commonly used tags to the CC-list of
patches. Additional tags are:
- Acked-by
- Reviewed-by
- Tested-by
- Reported-by
- Reviewed-and-tested-by
--suppress-cc=ack suppresses these additional CCs.
This and similar suggestions have come up a number of times. Rather
than hard-coding an ever-growing list of tags, general consensus seems
to be that it would be better to provide some sort of mechanism for
people to customize the list for their needs. See, for instance, [1].
Such ability would also be a better fit for non-standard, potentially
contested tags, such as Reviewed-and-tested-by:.
[1]: http://thread.gmane.org/gmane.comp.version-control.git/233003/focus=233739
Signed-off-by: Soren Brinkmann soren.brinkm...@xilinx.com
---
Documentation/git-send-email.txt | 3 +++
git-send-email.perl | 5 +++--
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/Documentation/git-send-email.txt
b/Documentation/git-send-email.txt
index f248a8665e1f..1b521446ca11 100644
--- a/Documentation/git-send-email.txt
+++ b/Documentation/git-send-email.txt
@@ -293,6 +293,9 @@ Automating
patch body (commit message) except for self (use 'self' for that).
- 'sob' will avoid including anyone mentioned in Signed-off-by lines except
for self (use 'self' for that).
+- 'ack' will avoid including anyone who acked the patch (mentioned in
+ Acked-by, Reviewed-by, Tested-by, Reviewed-and-tested-by lines except for
+ self (use 'self' for that).
- 'cccmd' will avoid running the --cc-cmd.
- 'body' is equivalent to 'sob' + 'bodycc'
- 'all' will suppress all auto cc values.
diff --git a/git-send-email.perl b/git-send-email.perl
index 3092ab356c76..18eb8a5139a4 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -407,7 +407,7 @@ my(%suppress_cc);
if (@suppress_cc) {
foreach my $entry (@suppress_cc) {
die Unknown --suppress-cc field: '$entry'\n
- unless $entry =~
/^(?:all|cccmd|cc|author|self|sob|body|bodycc)$/;
+ unless $entry =~
/^(?:all|cccmd|cc|author|self|sob|body|bodycc|ack)$/;
$suppress_cc{$entry} = 1;
}
}
@@ -1452,7 +1452,7 @@ foreach my $t (@files) {
# Now parse the message body
while($fh) {
$message .= $_;
- if (/^(Signed-off-by|Cc): (.*)$/i) {
+ if
(/^(Signed-off-by|Cc|Acked-by|Reviewed-by|Tested-by|Reported-by|Reviewed-and-tested-by):
(.*)$/i) {
chomp;
my ($what, $c) = ($1, $2);
chomp $c;
@@ -1462,6 +1462,7 @@ foreach my $t (@files) {
} else {
next if $suppress_cc{'sob'} and $what =~
/Signed-off-by/i;
next if $suppress_cc{'bodycc'} and $what =~
/Cc/i;
+ next if $suppress_cc{'ack'} and $what =~
/(Acked-by|Reviewed-by|Tested-by|Reported-by|Reviewed-and-tested-by)/i;
}
push @cc, $c;
printf((body) Adding cc: %s from line '%s'\n,
--
2.3.1.2.g90df61e.dirty
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
