Re: [PATCH] get_maintainer.pl: append reason for cc to the name bydefault

From: Florian Mickler
Date: Fri Sep 10 2010 - 06:30:46 EST


On Fri, 10 Sep 2010 02:42:47 -0700
Joe Perches <joe@xxxxxxxxxxx> wrote:

> On Fri, 2010-09-10 at 11:33 +0200, florian@xxxxxxxxxxx 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/