Re: [PATCH] MAINTAINERS: correct entry for LVM

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

 



On Tuesday 12 April 2016 12:20 AM, Wols Lists wrote:
On 11/04/16 17:39, Sudip Mukherjee wrote:
On Monday 11 April 2016 09:53 PM, Alasdair G Kergon wrote:
On Mon, Apr 11, 2016 at 09:45:01PM +0530, Sudip Mukherjee wrote:
L stands for "Mailing list that is relevant to this area", and this is a
mailing list. :)

Your proposed patch isn't changing the L entry, so this is of no
relevance.

Sorry, I am not understanding.

The current entry in MAINTAINERS is:
DEVICE-MAPPER  (LVM)
M:      Alasdair Kergon <agk@xxxxxxxxxx>
M:      Mike Snitzer <snitzer@xxxxxxxxxx>
M:      dm-devel@xxxxxxxxxx
L:      dm-devel@xxxxxxxxxx
...

So my patch just removed the line : "M:      dm-devel@xxxxxxxxxx"

So now the entry becomes :
DEVICE-MAPPER  (LVM)
M:      Alasdair Kergon <agk@xxxxxxxxxx>
M:      Mike Snitzer <snitzer@xxxxxxxxxx>
L:      dm-devel@xxxxxxxxxx
...

So, now it correctly shows dm-devel@xxxxxxxxxx as a mailing list which
should have cc to all the patches related to LVM.

Or am I understanding this wrong?

Yes. Because (I guess M stands for maintainer) this list has maintainer
status. As all patches should be sent to the maintainers therefore all
patches should be sent to this list.

The same person can appear twice in a phone book, once under their name
and once under their job title. This is exactly the same situation -
this list should appear once as a list to tell people that it's a list,
AND ALSO as a maintainer to tell people that patches must be sent to the
list.

I guess English is not your first language, but the important point is
that M and L are not mutually exclusive.

Don't worry, English is my first language. Have you tried with getmaintainer.pl and seen the result? It only shows dm-devel@xxxxxxxxxx as a Maintainer and not as a list. (I noticed because I was sending a patch, and hence this patch again). But I believe a mailing list can not be a Maintainer ( have you seen any patch with a Signed-off-by: from a mailing list? ).
Anyway, I think this thread has become too long for an unimportant patch.

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



[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux