All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
To: Richard Weinberger
	<richard.weinberger-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Kalle Valo <kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
	Robin van der Gracht
	<robin-/Q/L1SwJa3aEVqv0pETR8A@public.gmane.org>,
	Greg Kroah-Hartman
	<gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>,
	LKML <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	Ian Campbell
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
	Miguel Ojeda Sandonis
	<miguel.ojeda.sandonis-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Jonathan Cameron <jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Matt Ranostay <mranostay-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Andrew Morton
	<akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>,
	"David S . Miller"
	<davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>,
	Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>,
	Jiri Slaby <jslaby-IBi9RG/b67k@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: get_maintainer.pl and MAINTAINERS file
Date: Thu, 19 May 2016 13:57:43 -0300	[thread overview]
Message-ID: <20160519135743.5fc661c4@recife.lan> (raw)
In-Reply-To: <CAFLxGvwU-2aE7_0Suht--DvWby3RT=mdkpRomHz25MgicRfC3w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

Em Thu, 19 May 2016 16:25:45 +0200
Richard Weinberger <richard.weinberger-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> escreveu:

> On Thu, May 19, 2016 at 2:57 PM, Kalle Valo <kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> wrote:
> > (Changed the subject from "Re: [PATCH v6 0/3] auxdisplay: Introduce ht16k33 driver")
> >
> > Robin van der Gracht <robin-/Q/L1SwJa3aEVqv0pETR8A@public.gmane.org> 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

WARNING: multiple messages have this Message-ID (diff)
From: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
To: Richard Weinberger <richard.weinberger@gmail.com>
Cc: Kalle Valo <kvalo@codeaurora.org>,
	Robin van der Gracht <robin@protonic.nl>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	LKML <linux-kernel@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Kumar Gala <galak@codeaurora.org>,
	Miguel Ojeda Sandonis <miguel.ojeda.sandonis@gmail.com>,
	Thierry Reding <treding@nvidia.com>,
	Shawn Guo <shawnguo@kernel.org>,
	Jonathan Cameron <jic23@kernel.org>,
	Matt Ranostay <mranostay@gmail.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	"David S . Miller" <davem@davemloft.net>,
	Guenter Roeck <linux@roeck-us.net>, Jiri Slaby <jslaby@suse.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: get_maintainer.pl and MAINTAINERS file
Date: Thu, 19 May 2016 13:57:43 -0300	[thread overview]
Message-ID: <20160519135743.5fc661c4@recife.lan> (raw)
In-Reply-To: <CAFLxGvwU-2aE7_0Suht--DvWby3RT=mdkpRomHz25MgicRfC3w@mail.gmail.com>

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

  parent reply	other threads:[~2016-05-19 16:57 UTC|newest]

Thread overview: 21+ 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
     [not found]   ` <1463574199-21077-2-git-send-email-robin-/Q/L1SwJa3aEVqv0pETR8A@public.gmane.org>
2016-05-18 23:17     ` Rob Herring
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
     [not found] ` <1463574199-21077-1-git-send-email-robin-/Q/L1SwJa3aEVqv0pETR8A@public.gmane.org>
2016-05-18 15:09   ` [PATCH v6 0/3] auxdisplay: Introduce " Greg Kroah-Hartman
2016-05-18 15:09     ` 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 12:57         ` Kalle Valo
     [not found]         ` <871t4yfbmy.fsf_-_-HodKDYzPHsUD5k0oWYwrnHL1okKdlPRT@public.gmane.org>
2016-05-19 14:25           ` Richard Weinberger
2016-05-19 14:25             ` Richard Weinberger
     [not found]             ` <CAFLxGvwU-2aE7_0Suht--DvWby3RT=mdkpRomHz25MgicRfC3w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-05-19 16:57               ` Mauro Carvalho Chehab [this message]
2016-05-19 16:57                 ` Mauro Carvalho Chehab
2016-05-20 13:14                 ` Kalle Valo
2016-05-20 13:14                   ` Kalle Valo
2016-05-19 19:53         ` Jiri Slaby
2016-05-19 22:32           ` Joe Perches

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=20160519135743.5fc661c4@recife.lan \
    --to=mchehab-jph+aebz4p+uejcrhfaqsw@public.gmane.org \
    --cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
    --cc=davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org \
    --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
    --cc=jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=jslaby-IBi9RG/b67k@public.gmane.org \
    --cc=kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=miguel.ojeda.sandonis-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=mranostay-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
    --cc=richard.weinberger-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=robin-/Q/L1SwJa3aEVqv0pETR8A@public.gmane.org \
    --cc=shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.