From: Joe Perches <joe@perches.com>
To: Jiri Slaby <jslaby@suse.cz>, Kalle Valo <kvalo@codeaurora.org>,
Robin van der Gracht <robin@protonic.nl>
Cc: Mark Rutland <mark.rutland@arm.com>,
Pawel Moll <pawel.moll@arm.com>,
Kumar Gala <galak@codeaurora.org>,
"David S . Miller" <davem@davemloft.net>,
Miguel Ojeda Sandonis <miguel.ojeda.sandonis@gmail.com>,
Matt Ranostay <mranostay@gmail.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Jonathan Cameron <jic23@kernel.org>,
Rob Herring <robh+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
Andrew Morton <akpm@linux-foundation.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Thierry Reding <treding@nvidia.com>,
Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
Guenter Roeck <linux@roeck-us.net>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: get_maintainer.pl and MAINTAINERS file
Date: Thu, 19 May 2016 15:32:53 -0700 [thread overview]
Message-ID: <1463697173.14098.46.camel@perches.com> (raw)
In-Reply-To: <573E19B5.1050800@suse.cz>
On Thu, 2016-05-19 at 21:53 +0200, Jiri Slaby wrote:
> Cc: joe
>
> On 05/19/2016, 02:57 PM, Kalle Valo wrote:
> >
> > (Changed the subject from "Re: [PATCH v6 0/3] auxdisplay: Introduce ht16k33 driver")
> >
> > Robin van der Gracht <robin@protonic.nl> writes:
> >
> > >
> > > >
> > > > And 4th, what is with the insane number of people on cc:? Use
> > > > get_maintainer.pl correctly please, and don't just hit everyone you
> > > > can possibly think of with a cc: for no good reason.
> > > I did use the get_maintainer script. I solely passed the patch set as
> > > command line argument.
> > I think get_maintainer script should ignore MAINTAINERS file altogether.
Maybe, maybe not.
The list of MAINTAINERS names returned by get_maintainer.pl is:
Andrew Morton <akpm@linux-foundation.org> (commit_signer:39/504=8%)
"David S. Miller" <davem@davemloft.net> (commit_signer:38/504=8%)
Greg Kroah-Hartman <gregkh@linuxfoundation.org> (commit_signer:36/504=7%)
Kalle Valo <kvalo@codeaurora.org> (commit_signer:28/504=6%)
Mauro Carvalho Chehab <mchehab@osg.samsung.com> (added_lines:109/2073=5%,removed_lines:108/779=14%)
Guenter Roeck <linux@roeck-us.net> (removed_lines:48/779=6%)
Jiri Slaby <jslaby@suse.com> (removed_lines:46/779=6%)
> > After I did drivers/net/wireless reorganisation, and had to make
> > numerous changes to MAINTAINERS in the process, I have been CCed to all
> > sorts of patchsets to which I have nothing to do with (this one
> > included). I suspect get_maintainer.pl is to blame here.
Doubtful but possible.
I have no idea how many files were modified by whatever
patch was checked, but more files, more maintainers.
Unmaintained files have even more "not quite maintainers",
but get_maintainer.pl lists the names of people that recently
modified those specific files.
> Last time I even tried to fix this up, but failed due to time
> limitations as the code is a bit complex.
>
> Joe, you seem to be the one who touched the script many times recently.
> May we ask you to fix this up?
I'm not touching stuff right now. Maybe next month.
prev parent reply other threads:[~2016-05-19 22:33 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-18 12:23 [PATCH v6 0/3] auxdisplay: Introduce ht16k33 driver Robin van der Gracht
2016-05-18 12:23 ` [PATCH v6 1/3] of: add vendor prefix for Holtek Semiconductor Robin van der Gracht
2016-05-18 23:17 ` Rob Herring
2016-05-18 12:23 ` [PATCH v6 2/3] auxdisplay: ht16k33: Driver for LED controller Robin van der Gracht
2016-05-18 23:23 ` Rob Herring
2016-05-18 12:23 ` [PATCH v6 3/3] MAINTAINERS: auxdisplay: Added myself as maintainer for ht16k33 driver Robin van der Gracht
2016-05-18 14:32 ` Linus Walleij
2016-05-18 15:09 ` [PATCH v6 0/3] auxdisplay: Introduce " Greg Kroah-Hartman
2016-05-19 8:54 ` Robin van der Gracht
2016-05-19 12:57 ` get_maintainer.pl and MAINTAINERS file Kalle Valo
2016-05-19 14:25 ` Richard Weinberger
2016-05-19 16:57 ` Mauro Carvalho Chehab
2016-05-20 13:14 ` Kalle Valo
2016-05-19 19:53 ` Jiri Slaby
2016-05-19 22:32 ` Joe Perches [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1463697173.14098.46.camel@perches.com \
--to=joe@perches.com \
--cc=akpm@linux-foundation.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=gregkh@linuxfoundation.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=jic23@kernel.org \
--cc=jslaby@suse.cz \
--cc=kvalo@codeaurora.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=mark.rutland@arm.com \
--cc=mchehab@osg.samsung.com \
--cc=miguel.ojeda.sandonis@gmail.com \
--cc=mranostay@gmail.com \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=robin@protonic.nl \
--cc=shawnguo@kernel.org \
--cc=treding@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome