Re: [PATCH 2/2] MAINTAINERS: Mark VMware mailing list entries as private

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

 




> On Nov 10, 2021, at 9:20 AM, Srivatsa S. Bhat <srivatsa@xxxxxxxxxxxxx> wrote:
> 
> On Tue, Nov 09, 2021 at 01:57:31PM -0800, Joe Perches wrote:
>> On Tue, 2021-11-09 at 00:58 +0000, Nadav Amit wrote:
>>>> On Nov 8, 2021, at 4:37 PM, Joe Perches <joe@xxxxxxxxxxx> wrote:
>>>> On Mon, 2021-11-08 at 16:22 -0800, Srivatsa S. Bhat wrote:
>>>> 
>>>> So it's an exploder not an actual maintainer and it likely isn't
>>>> publically archived with any normal list mechanism.
>>>> 
>>>> So IMO "private" isn't appropriate.  Neither is "L:"
>>>> Perhaps just mark it as what it is as an "exploder".
>>>> 
>>>> Or maybe these blocks should be similar to:
>>>> 
>>>> M:	Name of Lead Developer <somebody@xxxxxxxxxx>
>>>> M:	VMware <foo> maintainers <linux-<foo>-maintainers@xxxxxxxxxxx>
>> 
>> Maybe adding entries like
>> 
>> M:	Named maintainer <whoever@xxxxxxxxxx>
>> R:	VMware <foo> reviewers <linux-<foo>-maintainers@xxxxxxxxxx>
>> 
>> would be best/simplest.
>> 
> 
> Sure, that sounds good to me. I also considered adding "(email alias)"
> like Juergen suggested, but I think the R: entry is clear enough.
> Please find the updated patch below.
> 
> ---
> 
> From f66faa238facf504cfc66325912ce7af8cbf79ec Mon Sep 17 00:00:00 2001
> From: "Srivatsa S. Bhat (VMware)" <srivatsa@xxxxxxxxxxxxx>
> Date: Mon, 8 Nov 2021 11:46:57 -0800
> Subject: [PATCH v2 2/2] MAINTAINERS: Mark VMware mailing list entries as email
> aliases
> 
> VMware mailing lists in the MAINTAINERS file are private lists meant
> for VMware-internal review/notification for patches to the respective
> subsystems. Anyone can post to these addresses, but there is no public
> read access like open mailing lists, which makes them more like email
> aliases instead (to reach out to reviewers).
> 
> So update all the VMware mailing list references in the MAINTAINERS
> file to mark them as such, using "R: email-alias@xxxxxxxxxx".
> 
> Signed-off-by: Srivatsa S. Bhat (VMware) <srivatsa@xxxxxxxxxxxxx>
> Cc: Zack Rusin <zackr@xxxxxxxxxx>
> Cc: Nadav Amit <namit@xxxxxxxxxx>
> Cc: Vivek Thampi <vithampi@xxxxxxxxxx>
> Cc: Vishal Bhakta <vbhakta@xxxxxxxxxx>
> Cc: Ronak Doshi <doshir@xxxxxxxxxx>
> Cc: pv-drivers@xxxxxxxxxx
> Cc: linux-graphics-maintainer@xxxxxxxxxx
> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: linux-rdma@xxxxxxxxxxxxxxx
> Cc: linux-scsi@xxxxxxxxxxxxxxx
> Cc: netdev@xxxxxxxxxxxxxxx
> Cc: linux-input@xxxxxxxxxxxxxxx
> ---
> MAINTAINERS | 22 +++++++++++-----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 118cf8170d02..4372d79027e9 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -6134,8 +6134,8 @@ T:	git git://anongit.freedesktop.org/drm/drm-misc
> F:	drivers/gpu/drm/vboxvideo/
> 
> DRM DRIVER FOR VMWARE VIRTUAL GPU
> -M:	"VMware Graphics" <linux-graphics-maintainer@xxxxxxxxxx>
> M:	Zack Rusin <zackr@xxxxxxxxxx>
> +R:	VMware Graphics Reviewers <linux-graphics-maintainer@xxxxxxxxxx>
> L:	dri-devel@xxxxxxxxxxxxxxxxxxxxx
> S:	Supported
> T:	git git://anongit.freedesktop.org/drm/drm-misc
> @@ -14189,7 +14189,7 @@ F:	include/uapi/linux/ppdev.h
> PARAVIRT_OPS INTERFACE
> M:	Juergen Gross <jgross@xxxxxxxx>
> M:	Srivatsa S. Bhat (VMware) <srivatsa@xxxxxxxxxxxxx>
> -L:	pv-drivers@xxxxxxxxxx (private)
> +R:	VMware PV-Drivers Reviewers <pv-drivers@xxxxxxxxxx>

This patch that you just sent seems to go on top of the previous patches
(as it removes "L: pv-drivers@xxxxxxxxxx (private)”).

Since the patches were still not merged, I would presume you should squash
the old 2/2 with this new patch and send v3 of these patches.





[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux