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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

> MAINTAINERS: Restore "L: linux-kernel@xxxxxxxxxxxxxxx" 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@xxxxxxxxxxxxxxx.

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

>   b5472cddbe2c MAINTAINERS: remove L: linux-kernel@xxxxxxxxxxxxxxx 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@xxxxxxxxxxxxx>
>  S:	Maintained
> +L:	linux-kernel@xxxxxxxxxxxxxxx

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.

--
To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Stable Commits]     [Linux Stable Kernel]     [Linux Kernel]     [Linux USB Devel]     [Linux Video &Media]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux