lkml.org 
[lkml]   [2019]   [Jun]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Subject[PATCH] get_maintainer: Add ability to skip moderated mailing lists
    From
    Date
    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.

    Signed-off-by: Joe Perches <joe@perches.com>
    Tested-by: Peter Zijlstra (Intel) <peterz@infradead.org>
    ---
    scripts/get_maintainer.pl | 14 ++++++++++----
    1 file changed, 10 insertions(+), 4 deletions(-)

    diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
    index c1c088ef1420..8c2fc22f3a11 100755
    --- a/scripts/get_maintainer.pl
    +++ b/scripts/get_maintainer.pl
    @@ -27,6 +27,7 @@ my $email_usename = 1;
    my $email_maintainer = 1;
    my $email_reviewer = 1;
    my $email_list = 1;
    +my $email_moderated_list = 1;
    my $email_subscriber_list = 0;
    my $email_git_penguin_chiefs = 0;
    my $email_git = 0;
    @@ -248,6 +249,7 @@ if (!GetOptions(
    'r!' => \$email_reviewer,
    'n!' => \$email_usename,
    'l!' => \$email_list,
    + 'moderated!' => \$email_moderated_list,
    's!' => \$email_subscriber_list,
    'multiline!' => \$output_multiline,
    'roles!' => \$output_roles,
    @@ -1023,7 +1025,8 @@ MAINTAINER field selection options:
    --r => include reviewer(s) if any
    --n => include name 'Full Name <addr\@domain.tld>'
    --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
    --roles => show roles (status:subsystem, git-signer, list, etc...)
    --rolestats => show roles and statistics (commits/total_commits, %)
    @@ -1313,11 +1316,14 @@ sub add_categories {
    } else {
    if ($email_list) {
    if (!$hash_list_to{lc($list_address)}) {
    - $hash_list_to{lc($list_address)} = 1;
    if ($list_additional =~ m/moderated/) {
    - push(@list_to, [$list_address,
    - "moderated list${list_role}"]);
    + if ($email_moderated_list) {
    + $hash_list_to{lc($list_address)} = 1;
    + push(@list_to, [$list_address,
    + "moderated list${list_role}"]);
    + }
    } else {
    + $hash_list_to{lc($list_address)} = 1;
    push(@list_to, [$list_address,
    "open list${list_role}"]);
    }

    \
     
     \ /
      Last update: 2019-06-28 11:25    [W:2.971 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site