Not so fast :) Paolo Bonzini <[email protected]> writes:
> From: Stephen Warren <[email protected]> > > Add an entry to MAINTAINERS that matches every patch, and requests the > user send patches to [email protected]. > > It's not 100% obvious to project newcomers that all patches should be sent > there; checkpatch doesn't say so, and since it mentions other lists to CC, s/checkpatch/get_maintainer.pl/ > the wording "the list" from the SubmitAPatch wiki page can be taken > to mean only those lists, not the main list too. > > The F: entries were taken from a similar entry in the Linux kernel. > > Modify get_maintainer.pl so that the all-matching list entry doesn't > prevent the git fallback from ever triggering. This suggests only the all-matching entry is exempt, but... > The fallback now relies > on finding a real person in MAINTAINERS, not just a mailing list for ... in fact all lists now are. Entries with L: and without M: Maintained: LINUX, POSIX, i386 target Odd fixes: GDB stub Orphan: Gumstix, Mipssim, Stable 0.10, Stable 0.14, Stable 0.15 For these, we now fall back to git. Intentional? If yes, mention it in the commit message. If no, the easiest fix is to give them an M, if we can find a willing victim. Aside: we also have entries with neither L: nor M: Orphan: LSI53C895A, M68K, an5206, dummy_m68k, mcf5208 Odd. > the relevant sub-community. This portion of the patch was suggested by > [email protected]. > > Cc: Paolo Bonzini <[email protected]> > Cc: Markus Armbruster <[email protected]> > Cc: John Snow <[email protected]> > Signed-off-by: Stephen Warren <[email protected]> > Message-Id: <[email protected]> > Signed-off-by: Paolo Bonzini <[email protected]> > --- > MAINTAINERS | 5 +++++ > scripts/get_maintainer.pl | 2 +- > 2 files changed, 6 insertions(+), 1 deletion(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index b6ed87a..2d78eea 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -52,6 +52,11 @@ General Project Administration > ------------------------------ > M: Peter Maydell <[email protected]> > > +All patches CC here "All patches must be sent to this mailing list" > +L: [email protected] > +F: * > +F: */ > + > Responsible Disclosure, Reporting Security Issues > ------------------------------ > W: http://wiki.qemu.org/SecurityProcess > diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl > index 7dacf32..8261bcb 100755 > --- a/scripts/get_maintainer.pl > +++ b/scripts/get_maintainer.pl > @@ -636,7 +636,7 @@ sub get_maintainers { > > if ($email) { > if (! $interactive) { > - $email_git_fallback = 0 if @email_to > 0 || @list_to > 0 || > $email_git || $email_git_blame; > + $email_git_fallback = 0 if @email_to > 0 || $email_git || > $email_git_blame; > if ($email_git_fallback) { > print STDERR "get_maintainer.pl: No maintainers found, printing > recent contributors.\n"; > print STDERR "get_maintainer.pl: Do not blindly cc: them on > patches! Use common sense.\n";
