From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mauro Carvalho Chehab Subject: Re: get_maintainer.pl and MAINTAINERS file Date: Thu, 19 May 2016 13:57:43 -0300 Message-ID: <20160519135743.5fc661c4@recife.lan> References: <1463574199-21077-1-git-send-email-robin@protonic.nl> <20160518150924.GA3871@kroah.com> <20160519105418.5df3c8f7@erd979> <871t4yfbmy.fsf_-_@kamboji.qca.qualcomm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Richard Weinberger Cc: Kalle Valo , Robin van der Gracht , Greg Kroah-Hartman , LKML , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Miguel Ojeda Sandonis , Thierry Reding , Shawn Guo , Jonathan Cameron , Matt Ranostay , Andrew Morton , "David S . Miller" , Guenter Roeck , Jiri Slaby , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org Em Thu, 19 May 2016 16:25:45 +0200 Richard Weinberger escreveu: > On Thu, May 19, 2016 at 2:57 PM, Kalle Valo wrote: > > (Changed the subject from "Re: [PATCH v6 0/3] auxdisplay: Introduce ht16k33 driver") > > > > Robin van der Gracht 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. > > 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. > > Are you sure it is because you are listed as maintainer for specific files? > If you are maintain something you have to be CC'ed. > It is rather because get_maintainer.pl tries to be smart and uses git to > find people who have touched a file... > > I guess instead you want something like: > http://permalink.gmane.org/gmane.linux.kernel.commits.head/538561 I guess what Kalle said is that the get_maintainer.pl script should disable, by default, "--git-blame" behavior for some specific files that everyone touches like /MAINTAINERS, Documentation/kernel-parameters.txt, /CREDITS, etc, as random people that touch on those files aren't really interested on being c/c to every change there. Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html