Messages in this thread | | | From | Bryan O'Donoghue <> | Subject | [PATCH v2 0/2] Expand get_maintainer to be explicit about supporters | Date | Thu, 29 Sep 2022 01:24:58 +0100 |
| |
V2:
https://lore.kernel.org/lkml/20220928003006.230103-1-bryan.odonoghue@linaro.org/T/#u
- Documentation/process Added in text to also make clear subsystem mailing list should be included - Krzysztof
- Changed get_maintainer.pl to print maintainer[supporter] or maintainer[volunteer] depending on MAINTAINERS file. - Thorsten/Bryan
- Choose supporter and volunteer instead of supported and volunteer Supporter and volunteer describe the role of the person whereas supported and volunteer would describe an activity and a role which isn't consistent. - Thorsten/Bryan
- I didn't change Documentation/process/5.Posting.rst This file doesn't mention get_maintainer.pl and I was mostly aiming to fixup the process around get_maintainer.pl. - Thorsten
- Myself and Thorsten discussed changing get_maintainer.pl, how it seems like a desirable thing to do but also that "it might break scripts for people" and it might.
I don't know if get_maintainer.pl is or should be considered to be a stable interface and an explicit software contract but, making it clear a supporter is also a maintainer seems like the right thing to do from a transmission of information perspective.
There is still the option of just updating Documentation/process in isolation.
V1: - Sent a standalone change to Documentation/process stating get_maintainer.pl email addresses marked "supporter" should be included in a patch run.
Bryan O'Donoghue (2): get_maintainer: Describe supporters and maintainers as required email recipients Documentation/process: Add text to indicate supporters should be mailed
Documentation/process/submitting-patches.rst | 8 +++++--- scripts/get_maintainer.pl | 4 ++-- 2 files changed, 7 insertions(+), 5 deletions(-)
-- 2.37.3
| |