]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
get_maintainer: stop reporting subsystem status as maintainer role
authorVlastimil Babka <vbabka@suse.cz>
Mon, 3 Feb 2025 11:13:17 +0000 (12:13 +0100)
committerAndrew Morton <akpm@linux-foundation.org>
Mon, 17 Mar 2025 05:30:49 +0000 (22:30 -0700)
After introducing the --substatus option, we can stop adjusting the
reported maintainer role by the subsystem's status.

For compatibility with the --git-chief-penguins option, keep the "chief
penguin" role.

Link: https://lkml.kernel.org/r/20250203-b4-get_maintainer-v2-2-83ba008b491f@suse.cz
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Tested-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Cc: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Cc: Joe Perches <joe@perches.com>
Cc: Kees Cook <kees@kernel.org>
Cc: Ted Ts'o <tytso@mit.edu>
Cc: Thorsten Leemhuis <linux@leemhuis.info>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
scripts/get_maintainer.pl

index 3eb922b4d22c0114b204a93736ee342f57a718cb..4414194bedcfd747bd24199b5de9ccf04bf6d227 100755 (executable)
@@ -1084,7 +1084,7 @@ MAINTAINER field selection options:
     --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...)
+    --roles => show roles (role:subsystem, git-signer, list, etc...)
     --rolestats => show roles and statistics (commits/total_commits, %)
     --substatus => show subsystem status if not Maintained (default: match --roles when output is tty)"
     --file-emails => add email addresses found in -f file (default: 0 (off))
@@ -1298,8 +1298,9 @@ sub get_maintainer_role {
     my $start = find_starting_index($index);
     my $end = find_ending_index($index);
 
-    my $role = "unknown";
+    my $role = "maintainer";
     my $subsystem = get_subsystem_name($index);
+    my $status = "unknown";
 
     for ($i = $start + 1; $i < $end; $i++) {
        my $tv = $typevalue[$i];
@@ -1307,23 +1308,13 @@ sub get_maintainer_role {
            my $ptype = $1;
            my $pvalue = $2;
            if ($ptype eq "S") {
-               $role = $pvalue;
+               $status = $pvalue;
            }
        }
     }
 
-    $role = lc($role);
-    if      ($role eq "supported") {
-       $role = "supporter";
-    } elsif ($role eq "maintained") {
-       $role = "maintainer";
-    } elsif ($role eq "odd fixes") {
-       $role = "odd fixer";
-    } elsif ($role eq "orphan") {
-       $role = "orphan minder";
-    } elsif ($role eq "obsolete") {
-       $role = "obsolete minder";
-    } elsif ($role eq "buried alive in reporters") {
+    $status = lc($status);
+    if ($status eq "buried alive in reporters") {
        $role = "chief penguin";
     }