Messages in this thread | | | Subject | Re: [bug] get_maintainer.pl incomplete output | From | Joe Perches <> | Date | Fri, 18 Oct 2013 17:25:57 -0700 |
| |
On Fri, 2013-10-18 at 15:58 -0700, David Rientjes wrote: > On Thu, 17 Oct 2013, Joe Perches wrote: > > Try this: > > It adds authored/lines_added/lines_deleted to rolestats > > For instance: > > $ ./scripts/get_maintainer.pl -f mm/vmpressure.c > > Tejun Heo <tj@kernel.org> (commit_signer:6/7=86%,authored:3/7=43%,removed_lines:15/21=71%) > > Michal Hocko <mhocko@suse.cz> (commit_signer:5/7=71%,authored:3/7=43%,added_lines:22/408=5%,removed_lines:6/21=29%) > > Andrew Morton <akpm@linux-foundation.org> (commit_signer:4/7=57%) > > Li Zefan <lizefan@huawei.com> (commit_signer:3/7=43%) > > "Kirill A. Shutemov" <kirill@shutemov.name> (commit_signer:1/7=14%) > > Anton Vorontsov <anton.vorontsov@linaro.org> (authored:1/7=14%,added_lines:374/408=92%) > > linux-mm@kvack.org (open list:MEMORY MANAGEMENT) > > linux-kernel@vger.kernel.org (open list) > > > > I haven't tested it much. > This looks good, thanks very much!
Oh sure, it's a trifle.
> I'm not sure how useful the > removed_lines stat is, but perhaps it can be useful for someone to chime > in if someone proposes a patch that includes support that had already been > removed.
The whole concept of lines +/- is kind of dubious. Quantity really doesn't judge value.
It makes some sense for patches where new files are added, but new files are a relatively a small percentage of the overall kernel source tree.
Let it stew for awhile in Andrew's tree and when -next opens up again, let's see if there are any other comments about it.
My guess is very few people will notice.
The best mechanism is to have MAINTAINERS entries.
| |