Re: [PATCH] Changed Maintainers.txt

[Laszlo Ersek <lersek@redhat.com>]

On 11/13/18 02:41, Wei, David wrote:
> Reviewed-by: David Wei <david.wei@intel.com>

(1) The original patch never reached the list. Is Zailiang Sun perhaps
not subscribed? I think maintainership requires that the maintainer be
subscribed.

(2) The subject line is a wreck. Zailiang, please resubmit the patch
with the following subject (or similar):

  Maintainers.txt: update Vlv2*Pkg maintainers

(44 characters)

One more comment below:

> -----Original Message-----
> From: Sun, Zailiang 
> Sent: Monday, November 12, 2018 1:05 PM
> To: edk2-devel@lists.01.org
> Cc: Sun, Zailiang <zailiang.sun@intel.com>; Wei, David <david.wei@intel.com>
> Subject: [PATCH] Changed Maintainers.txt
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Zailiang Sun <zailiang.sun@intel.com>
> CC: David Wei <david.wei@intel.com>
> ---
>  Maintainers.txt | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index b3048384c3..6ff80e76c2 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -3,8 +3,8 @@ MINNOWBOARD MAX/TURBOT PLATFORM Maintainers
>  
>  Vlv2DeviceRefCodePkg
>    David Wei <david.wei@intel.com>
> -  Mang Guo <mang.guo@intel.com>
> +  Zailiang Sun <zailiang.sun@intel.com>
>  
>  Vlv2TbltDevicePkg
>    David Wei <david.wei@intel.com>
> -  Mang Guo <mang.guo@intel.com>
> +  Zailiang Sun <zailiang.sun@intel.com>
> 

(3) I don't understand how this patch was formatted. The context doesn't
match at all: the lines don't start with "M:". Here's how
"Maintainers.txt" looks on my end, at commit da2c81ee96eb:

-------------
Vlv2DeviceRefCodePkg
M: David Wei <david.wei@intel.com>
M: Mang Guo <mang.guo@intel.com>

Vlv2TbltDevicePkg
M: David Wei <david.wei@intel.com>
M: Mang Guo <mang.guo@intel.com>
-------------

Ahh! Wait a second. In the hunk header above, I see:

  MINNOWBOARD MAX/TURBOT PLATFORM

That's totally not in the edk2 project. Is this patch perhaps for the
"edk2-platforms" tree? In that case:

- the subject prefix should not be [PATCH], but [PATCH edk2-platforms]

- I don't understand why edk2 and edk2-platforms *both* have
Vlv2DeviceRefCodePkg and Vlv2TbltDevicePkg directories!


Thanks
Laszlo