On Mon, 2022-10-03 at 17:24 +0100, Bryan O'Donoghue wrote:
> Throughout the documentation we make reference to maintainers as the people
> who ought to be mailed when sending patches.
>
> Right now get_maintainer.pl returns "supporter" for maintained files which
> are denoted "S: Supported".
As it should.
I think this patch is misguided and do not agree with the concept.
Either do away with the concept of S: Supported or use it.
As is, it merely hides the actual content of the MAINTAINERS file.
A reader of the MAINTAINERS file would still see the actual value.
Using other options of get_maintainer.pl like --sections would show
the actual content.
Other options would still show S:
>
> Update get_maintainer to output "maintainer" for "S: Supported" the
> differentiation between paid (v) non-paid maintainers doesn't impact who
> needs to be mailed for patch submissions.
[]
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
[]
> @@ -1283,7 +1283,7 @@ sub get_maintainer_role {
>
> $role = lc($role);
> if ($role eq "supported") {
> - $role = "supporter";
> + $role = "maintainer";
> } elsif ($role eq "maintained") {
> $role = "maintainer";
> } elsif ($role eq "odd fixes") {
