Re: [PATCH] MAINTAINERS: Clarify the meaning of nested maintainership

[Lars Kurth <lars.kurth@citrix.com>]

On 21/04/2016 17:03, "George Dunlap" <george.dunlap@citrix.com> wrote:

>Clarify the meaning of nested maintainership.
>
>Signed-off-by: George Dunlap <george.dunlap@citrix.com>
>---
>
> MAINTAINERS | 34 ++++++++++++++++++++++++++++++++++
> 1 file changed, 34 insertions(+)
>
>diff --git a/MAINTAINERS b/MAINTAINERS
>index a34685d..be901d5 100644
>--- a/MAINTAINERS
>+++ b/MAINTAINERS
>@@ -94,6 +94,40 @@ Descriptions of section entries:
> 	      printk, pr_info or pr_err
> 	   One regex pattern per line.  Multiple K: lines acceptable.
> 
>+
>+The meaning of nesting:
>+
>
>[snip]

>+ - In the case of a disagreement between maintainers, THE REST can
>+ vote to settle the matter.  (This should be very exceptional indeed.)

This creates a slight conflict with the project governance, which states
that committers act as referees in case of disagreements. As REST
maintainers are a superset of committers, this could create a potential
mismatch.

Given, that the text says "can", I don't have an issue with it. It merely
adds an extra level of complexity when it comes to resolving disagreements
and maybe could be confusing.

Regards
Lars

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel