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

From: Joe Perches
Date: Fri Sep 10 2010 - 11:12:44 EST


On Fri, 2010-09-10 at 13:11 +0200, Florian Mickler wrote:
> 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.
> Btw, why? I think it is a great way to show that it was done in an
> automated fashion.

More stuff than necessary to read, requires effort to find next.
Fewer names in visual field.

Also, it can affect systems that automatically add email addresses
to an address directory as it could store the entire content
before the < as the name.


--
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/