From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46034) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aT8dV-00032Y-3H for qemu-devel@nongnu.org; Tue, 09 Feb 2016 08:47:24 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aT8dP-0002jM-RO for qemu-devel@nongnu.org; Tue, 09 Feb 2016 08:47:21 -0500 Received: from mx1.redhat.com ([209.132.183.28]:47434) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aT8dP-0002ii-Jp for qemu-devel@nongnu.org; Tue, 09 Feb 2016 08:47:15 -0500 From: Markus Armbruster References: <1455020031-8268-1-git-send-email-pbonzini@redhat.com> <1455020031-8268-6-git-send-email-pbonzini@redhat.com> Date: Tue, 09 Feb 2016 14:47:12 +0100 In-Reply-To: <1455020031-8268-6-git-send-email-pbonzini@redhat.com> (Paolo Bonzini's message of "Tue, 9 Feb 2016 13:13:50 +0100") Message-ID: <87r3gmhu73.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PULL 31/32] MAINTAINERS: add all-match entry for qemu-devel@ List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: John Snow , qemu-devel@nongnu.org, Stephen Warren Not so fast :) Paolo Bonzini writes: > From: Stephen Warren > > Add an entry to MAINTAINERS that matches every patch, and requests the > user send patches to qemu-devel@nongnu.org. > > 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 > pbonzini@redhat.com. > > Cc: Paolo Bonzini > Cc: Markus Armbruster > Cc: John Snow > Signed-off-by: Stephen Warren > Message-Id: <1454987065-12961-1-git-send-email-swarren@wwwdotorg.org> > Signed-off-by: Paolo Bonzini > --- > 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 > > +All patches CC here "All patches must be sent to this mailing list" > +L: qemu-devel@nongnu.org > +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";