[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20100910123040.5a6f0128@schatten.dmk.lab>
Date: Fri, 10 Sep 2010 12:30:40 +0200
From: Florian Mickler <florian@...kler.org>
To: Joe Perches <joe@...ches.com>
Cc: Andrew Morton <akpm@...ux-foundation.org>,
"Stephen Hemminger (role:commit_signer)" <shemminger@...tta.com>,
"Wolfram Sang (role:commit_signer)" <w.sang@...gutronix.de>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] get_maintainer.pl: append reason for cc to the name by
default
On Fri, 10 Sep 2010 02:42:47 -0700
Joe Perches <joe@...ches.com> wrote:
> On Fri, 2010-09-10 at 11:33 +0200, florian@...kler.org wrote:
> > The script get_maintainer.pl is a very useful tool for deploying changes
> > made to the kernel. Among others it searches not only the MAINTAINERS
> > file but also the git history for people to send patches to.\
> []
> > By appending a a note of the reason for the cc in the name, the reason
> > becomes clear and the receiving side is relieved from feeling obliged to
> > check the patch while the sending side has a chance to adapt the
> > cc'list to their liking.
>
> I don't like reading annotated email names myself.
>
> As the coverage of file patterns in MAINTAINERS is
> now pretty good, the --git results are less useful.
>
> I'd much rather turn off --git as a default, or change
> get_maintainers to add git signers only when there's no
> specifically listed maintainer for a particular file.
>
>
Of course, that is also an option.
But I personally would love to be cc'd on files I committed stuff to,
as long as it is clear, that it was done in an automated way.
People that get angered by being cc'd a lot as commit_signers can always
filter their mail for that.
But I don't think it is the major part of kernel hackers that get angry
when they are cc'd. Most kernel hackers _love_ getting mail, or not? :)
At least they should.
Your mileage may vary of course, but I can't see any harm done by this
change. Anyone familiar with get_maintainer.pl can always just put
"--nogit" in his .get_maintainer.conf file.
I think the --git thing is great, as it really get's patches into the
kernel vs letting them rot on some mailinglist.
But
Cheers,
Flo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists