Thread (49 messages) 49 messages, 8 authors, 2015-10-30

[PATCH] get_maintainer: Add subsystem to reviewer output

From: Lee Jones <hidden>
Date: 2015-10-29 16:17:24
Also in: lkml

On 29 October 2015 at 14:14, Joe Perches [off-list ref] wrote:
On Thu, 2015-10-29 at 09:20 +0000, Lee Jones wrote:
quoted
On Wed, 28 Oct 2015, Joe Perches wrote:
quoted
quoted
An issue for that will be how multiple subsystem section matching
affects the output for reviewers who are also maintainers.
I think that's okay, becuase the 'MAINTAINERS tag' will be different.

As an example:

  Lee Jones [off-list ref] (supporter:DRIVER SUBSYSTEM)
  Lee Jones [off-list ref] (reviewer:VENDOR DRIVER NAME)
I'm pretty sure I know a little more about the
behavior of the get_maintainers script than you do.
Instead of this childishness, I'm guessing what you meant to say was;
unfortunately the current behaviour of the script is such that, if a
person is marked as a Reviewer and a Maintainer, the final one
mentioned in MAINTAINERS will take precedence over each of the others.
In other words, only one (the final) reference will be printed.

If so, thanks Joe that's informative, and yes, I can see how that
might cause issues in some cases.

-- 
Lee Jones
Linaro ST Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help