MAINTAINERS/get_maintainers: add support for reviewers

[Qemu-devel] [PATCH 0/5] MAINTAINERS/get_maintainers: add support for reviewers

Paolo Bonzini posted 5 patches 6 years, 11 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170503094351.5040-1-pbonzini@redhat.com

Test checkpatch passed

Test docker passed

Test s390x passed

MAINTAINERS               |  2 ++
scripts/get_maintainer.pl | 53 +++++++++++++++++++++++++++++++++--------------
2 files changed, 39 insertions(+), 16 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/5] MAINTAINERS/get_maintainers: add support for reviewers

Posted by Paolo Bonzini 6 years, 11 months ago

I would like to use "R" so that people can delegate maintenance of some
code while keeping some degree of oversight.  In my case, this can be
appropriate for record/replay and cpus.c.

Other people are free to use "R" as in Linux, where it is used to
designate non-maintainers for reviewing patches in a given area.

Paolo

Brian Norris (2):
  get_maintainer: it's '--pattern-depth', not '-pattern-depth'
  get_maintainer: --r (list reviewer) is on by default

Joe Perches (2):
  get_maintainer: Teach get_maintainer.pl about the new "R:" tag
  get_maintainer: add subsystem to reviewer output

Paul E. McKenney (1):
  MAINTAINERS: Add "R:" tag for self-appointed reviewers

 MAINTAINERS               |  2 ++
 scripts/get_maintainer.pl | 53 +++++++++++++++++++++++++++++++++--------------
 2 files changed, 39 insertions(+), 16 deletions(-)

-- 
2.12.2

Re: [Qemu-devel] [PATCH 0/5] MAINTAINERS/get_maintainers: add support for reviewers

Posted by Thomas Huth 6 years, 11 months ago

On 03.05.2017 11:43, Paolo Bonzini wrote:
> I would like to use "R" so that people can delegate maintenance of some
> code while keeping some degree of oversight.  In my case, this can be
> appropriate for record/replay and cpus.c.
> 
> Other people are free to use "R" as in Linux, where it is used to
> designate non-maintainers for reviewing patches in a given area.
> 
> Paolo
> 
> Brian Norris (2):
>   get_maintainer: it's '--pattern-depth', not '-pattern-depth'
>   get_maintainer: --r (list reviewer) is on by default
> 
> Joe Perches (2):
>   get_maintainer: Teach get_maintainer.pl about the new "R:" tag
>   get_maintainer: add subsystem to reviewer output
> 
> Paul E. McKenney (1):
>   MAINTAINERS: Add "R:" tag for self-appointed reviewers
> 
>  MAINTAINERS               |  2 ++
>  scripts/get_maintainer.pl | 53 +++++++++++++++++++++++++++++++++--------------
>  2 files changed, 39 insertions(+), 16 deletions(-)

Patches look fine to me, and I think this is a good idea, so:

Reviewed-by: Thomas Huth <thuth@redhat.com>