Re: [tip:core/urgent] MAINTAINERS: Restore "L: linux-kernel@vger.kernel.org" entries

[Joe Perches <joe@perches.com>]

On Tue, 2014-01-21 at 07:49 -0800, tip-bot for Ingo Molnar wrote:

> MAINTAINERS: Restore "L: linux-kernel@vger.kernel.org" entries
> 
> A couple of years ago all the "L: lkml" email list entries in
> MAINTAINERS were removed and replaced with a 'the rest' entry
> at the end of the file - under the theory that this is unnecessary
> duplication and that people would find it intuitive:

You are politicking with the commentary.
Intuition had nothing to do with the removal,

The idea was to avoid having an L: lkml line for
each of more than 1200 sections so people wouldn't
think that any section that didn't have an lkml
entry shouldn't need to have lkml on the cc list
for a patch.

Also, SubmittingPatches had and still has:

	6) Select your CC (e-mail carbon copy) list.

	Unless you have a reason NOT to do so, CC linux-kernel@vger.kernel.org.

Arguably, that same info should be added to the
MAINTAINERS header instead of each section.

>   b5472cddbe2c MAINTAINERS: remove L: linux-kernel@vger.kernel.org from all but "THE REST"
> 
> So it turns out that it's all but intuitive, not all people
> use scripts/get_maintainer.pl to extract maintainer contact info,
> some people read the MAINTAINERS file and see the lack of 'L:' entries
> of various lkml-only subsystems and are sending patches to the
> maintainers only, without Cc:-ing lkml.  They arguably have a point.

shrug,  I think it would be better if people
used get_maintainer more often.

MAINTAINERS already says:

	PLEASE CC: the maintainers and mailing lists that are generated
	by scripts/get_maintainer.pl.

Maybe duplicating the SubmittingPatches block into
the MAINTAINERS intro is good enough.

> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -4655,6 +4657,7 @@ F:	net/irda/
>  IRQ SUBSYSTEM
>  M:	Thomas Gleixner <tglx@linutronix.de>
>  S:	Maintained
> +L:	linux-kernel@vger.kernel.org

If this is done, can you please standardize on
placing the L: line immediately after the M: line
and before the S: line as was done for all the
other added lines.

Thanks.