| Message ID | 1454865729-5791-1-git-send-email-swarren@wwwdotorg.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Stephen Warren <swarren@wwwdotorg.org> writes: > 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, > 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. > > Signed-off-by: Stephen Warren <swarren@wwwdotorg.org> > --- > MAINTAINERS | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index b6ed87a26349..2d78eea5d62d 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -52,6 +52,11 @@ General Project Administration > ------------------------------ > M: Peter Maydell <peter.maydell@linaro.org> > > +All patches CC here > +L: qemu-devel@nongnu.org > +F: * > +F: */ > + > Responsible Disclosure, Reporting Security Issues > ------------------------------ > W: http://wiki.qemu.org/SecurityProcess I acknowledge the problem, but your patch breaks the "No maintainers found, printing recent contributors" functionality. Before: $ scripts/get_maintainer.pl -f MAINTAINERS get_maintainer.pl: No maintainers found, printing recent contributors. get_maintainer.pl: Do not blindly cc: them on patches! Use common sense. Paolo Bonzini <pbonzini@redhat.com> (commit_signer:20/74=27%) Kevin Wolf <kwolf@redhat.com> (commit_signer:13/74=18%) Stefan Hajnoczi <stefanha@redhat.com> (commit_signer:13/74=18%) "Michael S. Tsirkin" <mst@redhat.com> (commit_signer:10/74=14%) Peter Maydell <peter.maydell@linaro.org> (commit_signer:8/74=11%) After: $ scripts/get_maintainer.pl -f MAINTAINERS qemu-devel@nongnu.org (open list:All patches CC here)
On 02/08/2016 04:05 AM, Markus Armbruster wrote: > Stephen Warren <swarren@wwwdotorg.org> writes: > >> 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, >> 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. >> >> Signed-off-by: Stephen Warren <swarren@wwwdotorg.org> >> --- >> MAINTAINERS | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index b6ed87a26349..2d78eea5d62d 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -52,6 +52,11 @@ General Project Administration >> ------------------------------ >> M: Peter Maydell <peter.maydell@linaro.org> >> >> +All patches CC here >> +L: qemu-devel@nongnu.org >> +F: * >> +F: */ >> + >> Responsible Disclosure, Reporting Security Issues >> ------------------------------ >> W: http://wiki.qemu.org/SecurityProcess > > I acknowledge the problem, but your patch breaks the "No maintainers > found, printing recent contributors" functionality. > > Before: > > $ scripts/get_maintainer.pl -f MAINTAINERS > get_maintainer.pl: No maintainers found, printing recent contributors. > get_maintainer.pl: Do not blindly cc: them on patches! Use common sense. > > Paolo Bonzini <pbonzini@redhat.com> (commit_signer:20/74=27%) > Kevin Wolf <kwolf@redhat.com> (commit_signer:13/74=18%) > Stefan Hajnoczi <stefanha@redhat.com> (commit_signer:13/74=18%) > "Michael S. Tsirkin" <mst@redhat.com> (commit_signer:10/74=14%) > Peter Maydell <peter.maydell@linaro.org> (commit_signer:8/74=11%) > > After: > > $ scripts/get_maintainer.pl -f MAINTAINERS > qemu-devel@nongnu.org (open list:All patches CC here) > Agree ... Might this also break any "unmaintained" scripts we use? How about: 1) Update the wiki to be clearer on the submission process, and 2) Update the "no contributors found" output to also include the list to send to?
On 02/08/2016 11:52 AM, John Snow wrote: > > > On 02/08/2016 04:05 AM, Markus Armbruster wrote: >> Stephen Warren <swarren@wwwdotorg.org> writes: >> >>> 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, >>> 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. >>> >>> Signed-off-by: Stephen Warren <swarren@wwwdotorg.org> >>> --- >>> MAINTAINERS | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/MAINTAINERS b/MAINTAINERS >>> index b6ed87a26349..2d78eea5d62d 100644 >>> --- a/MAINTAINERS >>> +++ b/MAINTAINERS >>> @@ -52,6 +52,11 @@ General Project Administration >>> ------------------------------ >>> M: Peter Maydell <peter.maydell@linaro.org> >>> >>> +All patches CC here >>> +L: qemu-devel@nongnu.org >>> +F: * >>> +F: */ >>> + >>> Responsible Disclosure, Reporting Security Issues >>> ------------------------------ >>> W: http://wiki.qemu.org/SecurityProcess >> >> I acknowledge the problem, but your patch breaks the "No maintainers >> found, printing recent contributors" functionality. >> >> Before: >> >> $ scripts/get_maintainer.pl -f MAINTAINERS >> get_maintainer.pl: No maintainers found, printing recent contributors. >> get_maintainer.pl: Do not blindly cc: them on patches! Use common sense. >> >> Paolo Bonzini <pbonzini@redhat.com> (commit_signer:20/74=27%) >> Kevin Wolf <kwolf@redhat.com> (commit_signer:13/74=18%) >> Stefan Hajnoczi <stefanha@redhat.com> (commit_signer:13/74=18%) >> "Michael S. Tsirkin" <mst@redhat.com> (commit_signer:10/74=14%) >> Peter Maydell <peter.maydell@linaro.org> (commit_signer:8/74=11%) >> >> After: >> >> $ scripts/get_maintainer.pl -f MAINTAINERS >> qemu-devel@nongnu.org (open list:All patches CC here) >> > > Agree ... Might this also break any "unmaintained" scripts we use? > > How about: > 1) Update the wiki to be clearer on the submission process, and > 2) Update the "no contributors found" output to also include the list > to send to? My case was that maintainers were found, but the list only included the maintainer (Peter) and the ARM list, but not the main qemu list. So (2) above wouldn't have solved my original issue. qemu's get_maintainers.pl must be doing something different to the Linux kernel's version then, since IIRC the Linux kernel version always prints out recent contributors irrespective of whether maintainers were found, rather than as an alternative. I think if qemu's get_maintainers.pl acted in that way it would solve the problem my patch caused?
On 8 February 2016 at 19:04, Stephen Warren <swarren@wwwdotorg.org> wrote: > qemu's get_maintainers.pl must be doing something different to the Linux > kernel's version then, since IIRC the Linux kernel version always prints out > recent contributors irrespective of whether maintainers were found, rather > than as an alternative. I think if qemu's get_maintainers.pl acted in that > way it would solve the problem my patch caused? That difference from the kernel script's behaviour is deliberate, because the attempt to list recent contributors tends to list a whole lot of irrelevant people. So if there's a documented maintainer we prefer to just list them, and not all the random other people looking at the git logs drags up. Commit c6561586f0f8586 is where we changed the behaviour of get_maintainer.pl and the commit message has some rationale. thanks -- PMM
diff --git a/MAINTAINERS b/MAINTAINERS index b6ed87a26349..2d78eea5d62d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -52,6 +52,11 @@ General Project Administration ------------------------------ M: Peter Maydell <peter.maydell@linaro.org> +All patches CC here +L: qemu-devel@nongnu.org +F: * +F: */ + Responsible Disclosure, Reporting Security Issues ------------------------------ W: http://wiki.qemu.org/SecurityProcess
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, 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. Signed-off-by: Stephen Warren <swarren@wwwdotorg.org> --- MAINTAINERS | 5 +++++ 1 file changed, 5 insertions(+)