Commit 49662503 authored by Joe Perches's avatar Joe Perches Committed by Linus Torvalds

get_maintainer: add ability to skip moderated mailing lists

Add a command line switch --no-moderated to skip L: mailing lists marked
with 'moderated'.

Some people prefer not emailing moderated mailing lists as the
moderation time can be indeterminate and some emails can be
intentionally dropped by a moderator.

This can cause fragmentation of email threads when some are subscribed
to a moderated list but others are not and emails are dropped.

Link: http://lkml.kernel.org/r/6f23c2918ad9fc744269feb8f909bdfb105c5afc.camel@perches.comSigned-off-by: default avatarJoe Perches <joe@perches.com>
Tested-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent c296d4dc
...@@ -27,6 +27,7 @@ my $email_usename = 1; ...@@ -27,6 +27,7 @@ my $email_usename = 1;
my $email_maintainer = 1; my $email_maintainer = 1;
my $email_reviewer = 1; my $email_reviewer = 1;
my $email_list = 1; my $email_list = 1;
my $email_moderated_list = 1;
my $email_subscriber_list = 0; my $email_subscriber_list = 0;
my $email_git_penguin_chiefs = 0; my $email_git_penguin_chiefs = 0;
my $email_git = 0; my $email_git = 0;
...@@ -248,6 +249,7 @@ if (!GetOptions( ...@@ -248,6 +249,7 @@ if (!GetOptions(
'r!' => \$email_reviewer, 'r!' => \$email_reviewer,
'n!' => \$email_usename, 'n!' => \$email_usename,
'l!' => \$email_list, 'l!' => \$email_list,
'moderated!' => \$email_moderated_list,
's!' => \$email_subscriber_list, 's!' => \$email_subscriber_list,
'multiline!' => \$output_multiline, 'multiline!' => \$output_multiline,
'roles!' => \$output_roles, 'roles!' => \$output_roles,
...@@ -1023,7 +1025,8 @@ MAINTAINER field selection options: ...@@ -1023,7 +1025,8 @@ MAINTAINER field selection options:
--r => include reviewer(s) if any --r => include reviewer(s) if any
--n => include name 'Full Name <addr\@domain.tld>' --n => include name 'Full Name <addr\@domain.tld>'
--l => include list(s) if any --l => include list(s) if any
--s => include subscriber only list(s) if any --moderated => include moderated lists(s) if any (default: true)
--s => include subscriber only list(s) if any (default: false)
--remove-duplicates => minimize duplicate email names/addresses --remove-duplicates => minimize duplicate email names/addresses
--roles => show roles (status:subsystem, git-signer, list, etc...) --roles => show roles (status:subsystem, git-signer, list, etc...)
--rolestats => show roles and statistics (commits/total_commits, %) --rolestats => show roles and statistics (commits/total_commits, %)
...@@ -1313,11 +1316,14 @@ sub add_categories { ...@@ -1313,11 +1316,14 @@ sub add_categories {
} else { } else {
if ($email_list) { if ($email_list) {
if (!$hash_list_to{lc($list_address)}) { if (!$hash_list_to{lc($list_address)}) {
$hash_list_to{lc($list_address)} = 1;
if ($list_additional =~ m/moderated/) { if ($list_additional =~ m/moderated/) {
push(@list_to, [$list_address, if ($email_moderated_list) {
"moderated list${list_role}"]); $hash_list_to{lc($list_address)} = 1;
push(@list_to, [$list_address,
"moderated list${list_role}"]);
}
} else { } else {
$hash_list_to{lc($list_address)} = 1;
push(@list_to, [$list_address, push(@list_to, [$list_address,
"open list${list_role}"]); "open list${list_role}"]);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment