Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <71599eac-04ef-66d1-50d5-ebc564a6bf1d@gmail.com>
Date:   Sun, 2 Oct 2022 09:27:59 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
From:   Akira Yokosawa <akiyks@gmail.com>
Subject: Re: [PATCH v3] Documentation/process: Add text to indicate supporters
 should be mailed
To:     Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Cc:     corbet@lwn.net, konstantin@linuxfoundation.org,
        krzysztof.kozlowski@linaro.org, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux@leemhuis.info,
        Theodore Ts'o <tytso@mit.edu>
References: <20220930064629.329514-2-bryan.odonoghue@linaro.org>
 <21f8d79a-0ad4-b28b-15d8-f4be0cfd9730@gmail.com>
 <b119fee5-807f-1940-3378-f5ad638794f5@linaro.org> <Yzen4X1Na0MKXHs9@mit.edu>
 <70759d05-2646-57e5-0b87-3a196488f622@linaro.org>
Content-Language: en-US
In-Reply-To: <70759d05-2646-57e5-0b87-3a196488f622@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Sat, 1 Oct 2022 11:37:03 +0100, Bryan O'Donoghue wrote:
> On 01/10/2022 03:37, Theodore Ts'o wrote:
>> FWIW, I actually think the output of get_maintainer.pl is pretty
>> broken in this regard.=C2=A0 (Then again, I've never thought all that
>> highly of get_maintainer.pl,*especially*=C2=A0 because of the bogus gi=
t
>> fallback, but that's another story.)
>>
>> Consider:
>>
>> % ./scripts/get_maintainer.pl --file=C2=A0 drivers/acpi/power.c
>> "Rafael J. Wysocki"<rafael@kernel.org>=C2=A0 (supporter:ACPI)
>> Len Brown<lenb@kernel.org>=C2=A0 (reviewer:ACPI)
>> linux-acpi@vger.kernel.org=C2=A0 (open list:ACPI)
>> linux-kernel@vger.kernel.org=C2=A0 (open list)
>>
>> I'm sorry, but that's just*wrong*.=C2=A0 Rafael is the*maintainer*=C2=A0=
 of the
>> ACPI subsystem, and the term "supporter" is rarely if ever used
>> anywhere in our docs.=C2=A0 As I said earlier, trying to treat S: fiel=
d to
>> say anything about the entitles listed under the M: field of the
>> Maintainers file is a category error.
>=20
> I agree, I made exactly this error.
>=20
> I wasn't sure how people would necessarily feel about having
> get_maintainer produce the string 'maintainer' for both Maintained and
> Supported but, IMO it is more consistent to have it do so, since we ref=
er
> to maintainers all throughout the doucmentation and as you say above Ra=
fael
> is the person you *need* to mail there because he's the maintainer.

You'd better CC Joe Perches, who is the maintainer of get_maintainer.pl.
You might want to start a new thread with a different subject.
The main point becomes the behavior of get_maintainer.pl.

>=20
> Lets consider
>=20
> - maintainer as a string for "S: Supported"
> - Documentation update to reflect Krzysztof's point on git-fallback

Sounds reasonable to me.
Good luck!

Thanks,
Akira

>=20
> ---
> bod