Re: [PATCH] Maintainers.txt: add file filter to let GetMaintainer.py work properly

["Leif Lindholm" <leif.lindholm@linaro.org>]

Hi Jian,

The subject line does not indicate that this refers to the
edk2-platorms repository. Have you executed the SetupGit.py script in
your tree?

Other than that, this is sort of a duplicate of
https://edk2.groups.io/g/devel/topic/34462371
which I sent out over a month ago (before I went out traveling for 2
weeks and forgot about it).

I will send out a v2 and cc you on it.

/
    Leif

On Tue, Nov 12, 2019 at 03:41:14PM +0800, Jian J Wang wrote:
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Signed-off-by: Jian J Wang <jian.j.wang@intel.com>
> ---
>  Maintainers.txt | 46 +++++++++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 45 insertions(+), 1 deletion(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 9599024e20..a4c39000c6 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -11,7 +11,8 @@ Descriptions of section entries:
>  
>    L: Mailing list that is relevant to this area (default is edk2-devel)
>       Patches and questions should be sent to the email list.
> -  M: Cc address for patches and questions (ie, the package maintainer)
> +  M: Package Maintainer: Cc address for patches and questions. Responsible
> +     for reviewing and pushing package changes to source control.
>    R: Package Reviewer: Cc address for patches and questions. Reviewers help
>       maintainers review code, but don't have push access. A designated Package
>       Reviewer is reasonably familiar with the Package (or some modules
> @@ -29,6 +30,22 @@ Descriptions of section entries:
>       Obsolete:   Old code. Something tagged obsolete generally means
>                   it has been replaced by a better system and you
>                   should be using that.
> +  F: Files and directories with wildcard patterns.
> +     A trailing slash includes all files and subdirectory files.
> +     F:   MdeModulePkg/   all files in and below MdeModulePkg
> +     F:   MdeModulePkg/*  all files in MdeModulePkg, but not below
> +     F:   */Pci/*         all files in a directory called Pci, at any depth in
> +                          the hierarchy, but not below
> +     One pattern per line.  Multiple F: lines per section acceptable.
> +  X: Files and directories that are NOT maintained, same rules as F:
> +     Files exclusions are tested after file matches.
> +     Can be useful for excluding a specific subdirectory, for instance:
> +     F:   NetworkPkg/
> +     X:   NetworkPkg/Ip6Dxe/
> +     matches all files in and below NetworkPkg excluding NetworkPkg/Ip6Dxe/
> +  Filenames not caught by any F: rule get matched as being located in the top-
> +  level directory. (Internally, the script looks for a match called '<default>',
> +  so please don't add a file called that in the top-level directory.)
>  
>  EDK II
>  ------
> @@ -44,115 +61,142 @@ EDK II Packages:
>  ----------------
>  
>  Drivers/OptionRomPkg
> +F: Drivers/OptionRomPkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
>  M: Ray Ni <ray.ni@intel.com>
>  
>  Drivers/DisplayLink
> +F: Drivers/DisplayLink/
>  M: Leif Lindholm <leif.lindholm@linaro.org>
>  M: Ard Bieshuevel <ard.bieshuevel@linaro.org>
>  R: Andy Hayes <andy.hayes@displaylink.com>
>  
>  Platform
> +F: Platform/
>  M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>  M: Leif Lindholm <leif.lindholm@linaro.org>
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  
>  Platform/Intel/QuarkPlatformPkg
> +F: Platform/Intel/QuarkPlatformPkg/
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  M: Kelly Steele <kelly.steele@intel.com>
>  
>  Platform/Intel/Vlv2TbltDevicePkg
> +F: Platform/Intel/Vlv2TbltDevicePkg/
>  M: Zailiang Sun <zailiang.sun@intel.com>
>  M: Yi Qian <yi.qian@intel.com>
>  
>  Platform/Intel/AdvancedFeaturePkg
> +F: Platform/Intel/AdvancedFeaturePkg/
>  M: Michael Kubacki <michael.a.kubacki@intel.com>
>  M: Sai Chaganty <rangasai.v.chaganty@intel.com>
>  R: Liming Gao <liming.gao@intel.com>
>  
>  Platform/Intel/BoardModulePkg
> +F: Platform/Intel/BoardModulePkg/
>  M: Eric Dong <eric.dong@intel.com>
>  R: Liming Gao <liming.gao@intel.com>
>  
>  Platform/Intel/DebugFeaturePkg
> +F: Platform/Intel/DebugFeaturePkg/
>  M: Eric Dong <eric.dong@intel.com>
>  R: Liming Gao <liming.gao@intel.com>
>  
>  Platform/Intel/UserInterfaceFeaturePkg
> +F: Platform/Intel/UserInterfaceFeaturePkg/
>  M: Dandan Bi <dandan.bi@intel.com>
>  R: Liming Gao <liming.gao@intel.com>
>  
>  Platform/Intel/KabylakeOpenBoardPkg
> +F: Platform/Intel/KabylakeOpenBoardPkg/
>  M: Chasel Chiu <chasel.chiu@intel.com>
>  M: Michael Kubacki <michael.a.kubacki@intel.com>
>  M: Nate DeSimone <nathaniel.l.desimone@intel.com>
>  
>  Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3
> +F: Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/
>  R: Jeremy Soller <jeremy@system76.com>
>  
>  Platform/Intel/MinPlatformPkg
> +F: Platform/Intel/MinPlatformPkg/
>  M: Michael Kubacki <michael.a.kubacki@intel.com>
>  M: Chasel Chiu <chasel.chiu@intel.com>
>  M: Nate DeSimone <nathaniel.l.desimone@intel.com>
>  R: Liming Gao <liming.gao@intel.com>
>  
>  Platform/Intel/WhiskeylakeOpenBoardPkg
> +F: Platform/Intel/WhiskeylakeOpenBoardPkg/
>  M: Chasel Chiu <chasel.chiu@intel.com>
>  M: Michael Kubacki <michael.a.kubacki@intel.com>
>  M: Nate DeSimone <nathaniel.l.desimone@intel.com>
>  
>  Platform/Intel/SimicsOpenBoardPkg
> +F: Platform/Intel/SimicsOpenBoardPkg/
>  M: Wei David Y <david.y.wei@intel.com>
>  M: Agyeman Prince <prince.agyeman@intel.com>
>  
>  Platform/Intel/Tools
> +F: Platform/Intel/Tools/
>  M: Bob Feng <bob.c.feng@intel.com>
>  M: Liming Gao <liming.gao@intel.com>
>  
>  Platform/Marvell
> +F: Platform/Marvell/
>  R: Marcin Wojtas <mw@semihalf.com>
>  
>  Platform/SolidRun/Armada80x0McBin
> +F: Platform/SolidRun/Armada80x0McBin/
>  R: Marcin Wojtas <mw@semihalf.com>
>  
>  Silicon
> +F: Silicon/
>  M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>  M: Leif Lindholm <leif.lindholm@linaro.org>
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  
>  Silicon/Intel/IntelSiliconPkg
> +F: Silicon/Intel/IntelSiliconPkg/
>  M: Ray Ni <ray.ni@intel.com>
>  M: Rangasai V Chaganty <rangasai.v.chaganty@intel.com>
>  
>  Silicon/Intel/QuarkSocPkg
> +F: Silicon/Intel/QuarkSocPkg/
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  M: Kelly Steele <kelly.steele@intel.com>
>  
>  Silicon/Intel/Vlv2DeviceRefCodePkg
> +F: Silicon/Intel/Vlv2DeviceRefCodePkg/
>  M: Zailiang Sun <zailiang.sun@intel.com>
>  M: Yi Qian <yi.qian@intel.com>
>  
>  Silicon/Intel/CoffeelakeSiliconPkg
> +F: Silicon/Intel/CoffeelakeSiliconPkg/
>  M: Chasel Chiu <chasel.chiu@intel.com>
>  M: Michael Kubacki <michael.a.kubacki@intel.com>
>  M: Sai Chaganty <rangasai.v.chaganty@intel.com>
>  
>  Silicon/Intel/KabylakeSiliconPkg
> +F: Silicon/Intel/KabylakeSiliconPkg/
>  M: Chasel Chiu <chasel.chiu@intel.com>
>  M: Michael Kubacki <michael.a.kubacki@intel.com>
>  M: Sai Chaganty <rangasai.v.chaganty@intel.com>
>  
>  Silicon/Intel/SimicsX58SktPkg
> +F: Silicon/Intel/SimicsX58SktPkg/
>  M: Wei David Y <david.y.wei@intel.com>
>  M: Agyeman Prince <prince.agyeman@intel.com>
>  
>  Silicon/Intel/SimicsIch10Pkg
> +F: Silicon/Intel/SimicsIch10Pkg/
>  M: Wei David Y <david.y.wei@intel.com>
>  M: Agyeman Prince <prince.agyeman@intel.com>
>  
>  Silicon/Intel/Tools
> +F: Silicon/Intel/Tools/
>  M: Bob Feng <bob.c.feng@intel.com>
>  M: Liming Gao <liming.gao@intel.com>
>  
>  Silicon/Marvell
> +F: Silicon/Marvell/
>  R: Marcin Wojtas <mw@semihalf.com>
> -- 
> 2.17.1.windows.2
>