On 2/3/25 12:13, Vlastimil Babka wrote:
> The subsystem status is currently reported with --role(stats) by
> adjusting the maintainer role for any status different from Maintained.
> This has two downsides:
>
> - if a subsystem has only reviewers or mailing lists and no maintainers,
> the status is not reported (i.e. typically, Orphan subsystems have no
> maintainers)
>
> - the Supported status means that someone is paid for maintaining, but
> it is reported as "supporter" for all the maintainers, which can be
> incorrect. People have been also confused about what "supporter"
> means.
>
> This patch introduces a new --substatus option and functionality aimed
> to report the subsystem status separately, without adjusting the
> reported maintainer role. After the e-mails are output, the status of
> subsystems will follow, for example:
>
> ...
> linux-kernel@vger.kernel.org (open list:LIBRARY CODE)
> LIBRARY CODE status: Supported
>
> In order to allow replacing the role rewriting seamlessly, the new
> option works as follows:
>
> - it is automatically enabled when --email and --role are enabled
> (the defaults include --email and --rolestats which implies --role)
With the following fixup, the above changes to:
- it is automatically enabled when --email and --role are enabled and the
output is a terminal (the defaults include --email and --rolestats which
implies --role)
----8<----
>From f5523a85c742065fcb88a8aa26831f9dba9faf15 Mon Sep 17 00:00:00 2001
From: Vlastimil Babka <vba...@suse.cz>
Date: Tue, 11 Feb 2025 16:16:11 +0100
Subject: [PATCH] get_maintainer: add --substatus for reporting subsystem
status - fix
The automatically enabled --substatus can break existing scripts that do
not disable --rolestats. Require that script output goes to a terminal
to enable it automatically.
Signed-off-by: Vlastimil Babka <vba...@suse.cz>
---
scripts/get_maintainer.pl | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index cf902417a52c..4414194bedcf 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -317,7 +317,7 @@ $output_rolestats = 1 if ($interactive);
$output_roles = 1 if ($output_rolestats);
if (!defined $output_substatus) {
- $output_substatus = $email && $output_roles;
+ $output_substatus = $email && $output_roles && -t STDOUT;
}
if ($sections || $letters ne "") {
@@ -1086,7 +1086,7 @@ version: $V
--remove-duplicates => minimize duplicate email names/addresses
--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)"
+ --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))
--fixes => for patches, add signatures of commits with 'Fixes: <commit>'
(default: 1 (on))
--scm => print SCM tree(s) if any
--
2.48.1
