Re: get_maintainer.pl and MAINTAINERS file

From	Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Newsgroups	linux.kernel
Subject	Re: get_maintainer.pl and MAINTAINERS file
Date	2016-05-19 19:00 +0200
Message-ID	<rAzsJ-2iP-1@gated-at.bofh.it> (permalink)
References	<rA8LT-25R-3@gated-at.bofh.it> <rAbgK-3KP-25@gated-at.bofh.it> <rArYd-5S6-7@gated-at.bofh.it> <rAvIt-8m9-3@gated-at.bofh.it> <rAx7F-Wa-21@gated-at.bofh.it>
Organization	Samsung

Show all headers | View raw

Em Thu, 19 May 2016 16:25:45 +0200
Richard Weinberger <richard.weinberger@gmail.com> escreveu:

> On Thu, May 19, 2016 at 2:57 PM, Kalle Valo <kvalo@codeaurora.org> 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.
> > 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

Thread

[PATCH v6 0/3] auxdisplay: Introduce ht16k33 driver Robin van der Gracht <robin@protonic.nl> - 2016-05-18 14:30 +0200
  [PATCH v6 1/3] of: add vendor prefix for Holtek Semiconductor Robin van der Gracht <robin@protonic.nl> - 2016-05-18 14:30 +0200
    Re: [PATCH v6 1/3] of: add vendor prefix for Holtek Semiconductor Rob Herring <robh@kernel.org> - 2016-05-19 01:20 +0200
  Re: [PATCH v6 0/3] auxdisplay: Introduce ht16k33 driver Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-05-18 17:10 +0200
    Re: [PATCH v6 0/3] auxdisplay: Introduce ht16k33 driver Robin van der Gracht <robin@protonic.nl> - 2016-05-19 11:00 +0200
      get_maintainer.pl and MAINTAINERS file Kalle Valo <kvalo@codeaurora.org> - 2016-05-19 15:00 +0200
        Re: get_maintainer.pl and MAINTAINERS file Richard Weinberger <richard.weinberger@gmail.com> - 2016-05-19 16:30 +0200
          Re: get_maintainer.pl and MAINTAINERS file Mauro Carvalho Chehab <mchehab@osg.samsung.com> - 2016-05-19 19:00 +0200
            Re: get_maintainer.pl and MAINTAINERS file Kalle Valo <kvalo@codeaurora.org> - 2016-05-20 15:20 +0200
        Re: get_maintainer.pl and MAINTAINERS file Jiri Slaby <jslaby@suse.cz> - 2016-05-19 22:00 +0200
          Re: get_maintainer.pl and MAINTAINERS file Joe Perches <joe@perches.com> - 2016-05-20 00:40 +0200

csiph-web