Re: [PATCH] Maintainers.txt: Update mail address

[Laszlo Ersek <lersek@redhat.com>]

On 01/09/19 10:59, Ruiyu Ni wrote:
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ray Ni <ruiyu.ni@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> ---
>  Maintainers.txt | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)

And I'll update my address book in Thunderbird :)

> diff --git a/Maintainers.txt b/Maintainers.txt
> index 7f2f9de79f..3b2676bc32 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -124,12 +124,12 @@ EmulatorPkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/EmulatorPkg
>  M: Jordan Justen <jordan.l.justen@intel.com>
>  M: Andrew Fish <afish@apple.com>
> -M: Ruiyu Ni <ruiyu.ni@intel.com>
> +M: Ray Ni <ray.ni@intel.com>
>  S: Maintained
>  
>  FatPkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/Edk2-fat-driver
> -M: Ruiyu Ni <ruiyu.ni@intel.com>
> +M: Ray Ni <ray.ni@intel.com>
>  T: svn - https://svn.code.sf.net/p/edk2-fatdriver2/code/trunk/EnhancedFat
>  T: git - https://github.com/tianocore/edk2-FatPkg.git
>  
> @@ -173,7 +173,7 @@ R: Star Zeng <star.zeng@intel.com>
>  
>  IntelSiliconPkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/IntelSiliconPkg
> -M: Ruiyu Ni <ruiyu.ni@intel.com>
> +M: Ray Ni <ray.ni@intel.com>
>  M: Rangasai V Chaganty <rangasai.v.chaganty@intel.com>
>  
>  MdeModulePkg
> @@ -182,7 +182,7 @@ M: Jian J Wang <jian.j.wang@intel.com>
>  M: Hao Wu <hao.a.wu@intel.com>
>  M: NetworkPkg maintainers
>    (Universal/Network and related libraries, header files)
> -R: Ruiyu Ni <ruiyu.ni@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
>    (especially for Bus, Universal/Console, Universal/Disk,
>     Universal/BdsDxe and related libraries, header files)
>  R: Star Zeng <star.zeng@intel.com>
> @@ -199,7 +199,7 @@ M: Jiaxin Wu <jiaxin.wu@intel.com>
>  
>  Nt32Pkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/Nt32Pkg
> -M: Ruiyu Ni <ruiyu.ni@intel.com>
> +M: Ray Ni <ray.ni@intel.com>
>  M: Hao Wu <hao.a.wu@intel.com>
>  
>  Omap35xxPkg
> @@ -209,7 +209,7 @@ M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>  
>  OptionRomPkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
> -M: Ruiyu Ni <ruiyu.ni@intel.com>
> +M: Ray Ni <ray.ni@intel.com>
>  
>  OvmfPkg
>  W: http://www.tianocore.org/ovmf/
> @@ -222,7 +222,7 @@ S: Maintained
>  
>  PcAtChipsetPkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/PcAtChipsetPkg
> -M: Ruiyu Ni <ruiyu.ni@intel.com>
> +M: Ray Ni <ray.ni@intel.com>
>  
>  QuarkPlatformPkg, QuarkSocPkg
>  M: Michael D Kinney <michael.d.kinney@intel.com>
> @@ -236,14 +236,14 @@ M: Jiewen Yao <jiewen.yao@intel.com>
>  ShellBinPkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
>  M: Jaben Carsey <jaben.carsey@intel.com>      (Ia32/X64)
> -M: Ruiyu Ni <ruiyu.ni@intel.com>          (Ia32/X64)
> +M: Ray Ni <ray.ni@intel.com>          (Ia32/X64)
>  M: Leif Lindholm <leif.lindholm@linaro.org>   (ARM/AArch64)
>  M: Ard Biesheuvel <ard.biesheuvel@linaro.org> (ARM/AArch64)
>  
>  ShellPkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
>  M: Jaben Carsey <jaben.carsey@intel.com>
> -M: Ruiyu Ni <ruiyu.ni@intel.com>
> +M: Ray Ni <ray.ni@intel.com>
>  
>  SignedCapsulePkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
> @@ -262,7 +262,7 @@ M: Jaben Carsey <jaben.carsey@intel.com>
>  UefiCpuPkg
>  W: https://github.com/tianocore/tianocore.github.io/wiki/UefiCpuPkg
>  M: Eric Dong <eric.dong@intel.com>
> -M: Ruiyu Ni <ruiyu.ni@intel.com>
> +M: Ray Ni <ray.ni@intel.com>
>  R: Laszlo Ersek <lersek@redhat.com>
>  
>  Vlv2DeviceRefCodePkg
> 

Ugh, don't you have a bit too much on your plate? Co-maintaining 9
packages, and being a reviewer for another one. That can't be easy!

(Side topic: here's a list of maintainers/reviewers (lumped together)
with responsibility for more than one "area" (not just package), ordered
descending by area count:

$ grep '^[MR]: ' Maintainers.txt \
| cut -c 4- \
| sort \
| uniq -c \
| sort -rn

      9 Ruiyu Ni <ruiyu.ni@intel.com>
      7 Liming Gao <liming.gao@intel.com>
      7 Ard Biesheuvel <ard.biesheuvel@linaro.org>
      6 Leif Lindholm <leif.lindholm@linaro.org>
      5 Star Zeng <star.zeng@intel.com>
      5 Michael D Kinney <michael.d.kinney@intel.com>
      4 Nate DeSimone <nathaniel.l.desimone@intel.com>
      4 Laszlo Ersek <lersek@redhat.com>
      4 Chasel Chiu <chasel.chiu@intel.com>
      3 Jiewen Yao <jiewen.yao@intel.com>
      3 Jaben Carsey <jaben.carsey@intel.com>
      3 Hao Wu <hao.a.wu@intel.com>
      2 Zailiang Sun <zailiang.sun@intel.com>
      2 Yi Qian <yi.qian@intel.com>
      2 Julien Grall <julien.grall@linaro.org>
      2 Jordan Justen <jordan.l.justen@intel.com>
      2 Daryl McDaniel <edk2-lists@mc2research.org>
      2 Chao Zhang <chao.b.zhang@intel.com>
      2 Andrew Fish <afish@apple.com>
)

Anyway it looks like all occurrences in the text file are covered.

Reviewed-by: Laszlo Ersek <lersek@redhat.com>

Thanks,
Laszlo