Hi Leif, Sorry I didn't notice there's already patch out there. I don't work on edk2-platforms very often. I forgot to setup the git environment for edk2-platforms. Anyway, let's drop this patch.
Regards, Jian > -----Original Message----- > From: Leif Lindholm <[email protected]> > Sent: Tuesday, November 12, 2019 6:02 PM > To: Wang, Jian J <[email protected]> > Cc: [email protected]; Ard Biesheuvel <[email protected]>; Kinney, > Michael D <[email protected]>; Ni, Ray <[email protected]>; Gao, > Liming <[email protected]> > Subject: Re: [PATCH] Maintainers.txt: add file filter to let GetMaintainer.py > work > properly > > 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 <[email protected]> > > Cc: Leif Lindholm <[email protected]> > > Cc: Michael D Kinney <[email protected]> > > Cc: Ray Ni <[email protected]> > > Cc: Liming Gao <[email protected]> > > Signed-off-by: Jian J Wang <[email protected]> > > --- > > 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 <[email protected]> > > > > Drivers/DisplayLink > > +F: Drivers/DisplayLink/ > > M: Leif Lindholm <[email protected]> > > M: Ard Bieshuevel <[email protected]> > > R: Andy Hayes <[email protected]> > > > > Platform > > +F: Platform/ > > M: Ard Biesheuvel <[email protected]> > > M: Leif Lindholm <[email protected]> > > M: Michael D Kinney <[email protected]> > > > > Platform/Intel/QuarkPlatformPkg > > +F: Platform/Intel/QuarkPlatformPkg/ > > M: Michael D Kinney <[email protected]> > > M: Kelly Steele <[email protected]> > > > > Platform/Intel/Vlv2TbltDevicePkg > > +F: Platform/Intel/Vlv2TbltDevicePkg/ > > M: Zailiang Sun <[email protected]> > > M: Yi Qian <[email protected]> > > > > Platform/Intel/AdvancedFeaturePkg > > +F: Platform/Intel/AdvancedFeaturePkg/ > > M: Michael Kubacki <[email protected]> > > M: Sai Chaganty <[email protected]> > > R: Liming Gao <[email protected]> > > > > Platform/Intel/BoardModulePkg > > +F: Platform/Intel/BoardModulePkg/ > > M: Eric Dong <[email protected]> > > R: Liming Gao <[email protected]> > > > > Platform/Intel/DebugFeaturePkg > > +F: Platform/Intel/DebugFeaturePkg/ > > M: Eric Dong <[email protected]> > > R: Liming Gao <[email protected]> > > > > Platform/Intel/UserInterfaceFeaturePkg > > +F: Platform/Intel/UserInterfaceFeaturePkg/ > > M: Dandan Bi <[email protected]> > > R: Liming Gao <[email protected]> > > > > Platform/Intel/KabylakeOpenBoardPkg > > +F: Platform/Intel/KabylakeOpenBoardPkg/ > > M: Chasel Chiu <[email protected]> > > M: Michael Kubacki <[email protected]> > > M: Nate DeSimone <[email protected]> > > > > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3 > > +F: Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/ > > R: Jeremy Soller <[email protected]> > > > > Platform/Intel/MinPlatformPkg > > +F: Platform/Intel/MinPlatformPkg/ > > M: Michael Kubacki <[email protected]> > > M: Chasel Chiu <[email protected]> > > M: Nate DeSimone <[email protected]> > > R: Liming Gao <[email protected]> > > > > Platform/Intel/WhiskeylakeOpenBoardPkg > > +F: Platform/Intel/WhiskeylakeOpenBoardPkg/ > > M: Chasel Chiu <[email protected]> > > M: Michael Kubacki <[email protected]> > > M: Nate DeSimone <[email protected]> > > > > Platform/Intel/SimicsOpenBoardPkg > > +F: Platform/Intel/SimicsOpenBoardPkg/ > > M: Wei David Y <[email protected]> > > M: Agyeman Prince <[email protected]> > > > > Platform/Intel/Tools > > +F: Platform/Intel/Tools/ > > M: Bob Feng <[email protected]> > > M: Liming Gao <[email protected]> > > > > Platform/Marvell > > +F: Platform/Marvell/ > > R: Marcin Wojtas <[email protected]> > > > > Platform/SolidRun/Armada80x0McBin > > +F: Platform/SolidRun/Armada80x0McBin/ > > R: Marcin Wojtas <[email protected]> > > > > Silicon > > +F: Silicon/ > > M: Ard Biesheuvel <[email protected]> > > M: Leif Lindholm <[email protected]> > > M: Michael D Kinney <[email protected]> > > > > Silicon/Intel/IntelSiliconPkg > > +F: Silicon/Intel/IntelSiliconPkg/ > > M: Ray Ni <[email protected]> > > M: Rangasai V Chaganty <[email protected]> > > > > Silicon/Intel/QuarkSocPkg > > +F: Silicon/Intel/QuarkSocPkg/ > > M: Michael D Kinney <[email protected]> > > M: Kelly Steele <[email protected]> > > > > Silicon/Intel/Vlv2DeviceRefCodePkg > > +F: Silicon/Intel/Vlv2DeviceRefCodePkg/ > > M: Zailiang Sun <[email protected]> > > M: Yi Qian <[email protected]> > > > > Silicon/Intel/CoffeelakeSiliconPkg > > +F: Silicon/Intel/CoffeelakeSiliconPkg/ > > M: Chasel Chiu <[email protected]> > > M: Michael Kubacki <[email protected]> > > M: Sai Chaganty <[email protected]> > > > > Silicon/Intel/KabylakeSiliconPkg > > +F: Silicon/Intel/KabylakeSiliconPkg/ > > M: Chasel Chiu <[email protected]> > > M: Michael Kubacki <[email protected]> > > M: Sai Chaganty <[email protected]> > > > > Silicon/Intel/SimicsX58SktPkg > > +F: Silicon/Intel/SimicsX58SktPkg/ > > M: Wei David Y <[email protected]> > > M: Agyeman Prince <[email protected]> > > > > Silicon/Intel/SimicsIch10Pkg > > +F: Silicon/Intel/SimicsIch10Pkg/ > > M: Wei David Y <[email protected]> > > M: Agyeman Prince <[email protected]> > > > > Silicon/Intel/Tools > > +F: Silicon/Intel/Tools/ > > M: Bob Feng <[email protected]> > > M: Liming Gao <[email protected]> > > > > Silicon/Marvell > > +F: Silicon/Marvell/ > > R: Marcin Wojtas <[email protected]> > > -- > > 2.17.1.windows.2 > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#50464): https://edk2.groups.io/g/devel/message/50464 Mute This Topic: https://groups.io/mt/54125563/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
