lkml.org 
[lkml]   [2021]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] get_maintainer: only show lkml as last resort
Date
The documentation implies that patches should be primarily sent to a
subsystem-specific mailing list [0]. Make get_maintainer only return the
generic linux-kernel@vger.kernel.org ("THE REST") list when no other
matching mailing list is found.

Most patches sent to lkml today are also sent to some other list. This
change should lower the message volume on lkml in the long run, making
the list more useful for those cases where it's the only option.

[0]: Documentation/process/submitting-patches.rst:
> You should also normally choose at least one mailing list to receive a
> copy of your patch set. linux-kernel@vger.kernel.org functions as a
> list of last resort, but the volume on that list has caused a number of
> developers to tune it out. Look in the MAINTAINERS file for a
> subsystem-specific list; your patch will probably get more attention
> there.

Signed-off-by: Hannu Hartikainen <hannu@hrtk.in>
---

I'm not sure if this is technically the best solution so I'm looking
forward to review comments. But process-wise I think this should be a
good change. Looking at tutorials and such, many people seem to just
blindly run scripts/get_maintainer.pl and send their patches to all the
addresses the script outputs. This must inflate the list a lot, making
it difficult to follow and making it much more difficult to get readers
for patches belonging to the list (including this one, ironically).

scripts/get_maintainer.pl | 19 +++++++++++++------
1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 2075db0c08b8..c0a8dd5dfdbf 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -627,6 +627,7 @@ my %email_hash_address;
my @email_to = ();
my %hash_list_to;
my @list_to = ();
+my @list_the_rest_to = ();
my @scm = ();
my @web = ();
my @subsystem = ();
@@ -951,6 +952,11 @@ sub get_maintainers {
}
}

+ # if no other list would be printed, fall back to THE REST
+ if (scalar(@list_to) == 0) {
+ @list_to = @list_the_rest_to
+ }
+
foreach my $email (@email_to, @list_to) {
$email->[0] = deduplicate_email($email->[0]);
}
@@ -1303,10 +1309,6 @@ sub get_list_role {

my $subsystem = get_subsystem_name($index);

- if ($subsystem eq "THE REST") {
- $subsystem = "";
- }
-
return $subsystem;
}

@@ -1355,8 +1357,13 @@ sub add_categories {
}
} else {
$hash_list_to{lc($list_address)} = 1;
- push(@list_to, [$list_address,
- "open list${list_role}"]);
+ if ($list_role eq ":THE REST") {
+ push(@list_the_rest_to, [$list_address,
+ "open list"]);
+ } else {
+ push(@list_to, [$list_address,
+ "open list${list_role}"]);
+ }
}
}
}
--
2.32.0
\
 
 \ /
  Last update: 2021-07-06 10:45    [W:0.045 / U:0.156 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site