Messages in this thread | | | Date | Thu, 6 May 2010 08:42:51 -0700 (PDT) | From | Davide Libenzi <> | Subject | Re: indiscriminate get_maintainer.pl usage |
| |
On Thu, 6 May 2010, Stefan Richter wrote:
> > I should add --nogit option. > > The script does get some useful information out of git (but obviously > also some questionable). > > --roles and --rolestats give away some more data... > > $ scripts/get_maintainer.pl --rolestats -f fs/eventpoll.c > Alexander Viro <viro@zeniv.linux.org.uk> (maintainer:FILESYSTEMS (VFS...) > Andrew Morton <akpm@linux-foundation.org> (commit_signer:2/6=33%) > "Eric W. Biederman" <ebiederm@xmission.com> (commit_signer:2/6=33%) > Davide Libenzi <davidel@xmailserver.org> (commit_signer:2/6=33%) > Roland Dreier <rolandd@cisco.com> (commit_signer:1/6=17%) > Stefan Richter <stefanr@s5r6.in-berlin.de> (commit_signer:1/6=17%) > linux-fsdevel@vger.kernel.org (open list:FILESYSTEMS (VFS...) > linux-kernel@vger.kernel.org (open list) > > ...but still don't distinguish between signed-off-by or reviewed-by on > one hand, and reported-by and tested-by on the other hand. Cc'ing a > reporter or tester would only make sense if the patch changes something > that directly affects the reporter's/ tester's setup, e.g. reworks a > provisional fix.
I have always maintained that code w/out the need of explicit entry in the MAINTAINERS file. For isolated files people usually include the copyright owners and the ones listed in the comment header in general. No problem in being explicitly listed if this causes noise during bug reporting.
- Davide
| |