On 29/09/2022 02:25, Bryan O'Donoghue wrote: > Recently when submitting a yaml change I found that I had omitted the > maintainer whose tree the change needed to go through. > > The reason for that is the path in MAINTAINERS is marked as Supported not > Maintained. Reading MAINTAINERS we see quote: > > Supported: Someone is actually paid to look after this. > Maintained: Someone actually looks after it. > > The current submitting-patches.rst only says to mail maintainers though not > supporters. When we run scripts/get_maintainer.pl anybody who is denoted a > paid maintainer will appear as a supporter. > > Let's add some text to the submitting-patches.rst to indicate that > supporters should similarly be mailed so that you can't do as I did and > mail every maintainer get_maintainer.pl tells you to, without actually > mailing the one supporter you need to. > > Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> > --- > Documentation/process/submitting-patches.rst | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst > index be49d8f2601b4..aabccaea93375 100644 > --- a/Documentation/process/submitting-patches.rst > +++ b/Documentation/process/submitting-patches.rst > @@ -227,9 +227,11 @@ You should always copy the appropriate subsystem maintainer(s) on any patch > to code that they maintain; look through the MAINTAINERS file and the > source code revision history to see who those maintainers are. The > script scripts/get_maintainer.pl can be very useful at this step (pass paths to > -your patches as arguments to scripts/get_maintainer.pl). If you cannot find a > -maintainer for the subsystem you are working on, Andrew Morton > -(akpm@xxxxxxxxxxxxxxxxxxxx) serves as a maintainer of last resort. > +your patches as arguments to scripts/get_maintainer.pl). You should mail > +everyone who appears as "maintainer[volunteer]" or "maintainer[supporter]" as As I said before, this still ignores reviewers. I don't think it is going to good direction. The submitter is expected to CC everyone/everything which is pointed by get_maintainers.pl except the Git-fallback entries. Best regards, Krzysztof
