Even the five people claims to be the maintainers of all TPM2 modules, I still 
think
the maintainers.txt is not very human readable.

I suggest a txt file as below:
MdePkg
......
F: MdePkg/Include/IndustryStandard/Tpm2*
R: Marc-André Lureau <marcandre.lur...@redhat.com>
R: Stefan Berger <stef...@linux.ibm.com>

SecurityPkg
......
F: SecurityPkg/*/Tpm2*
R: Marc-André Lureau <marcandre.lur...@redhat.com>
R: Stefan Berger <stef...@linux.ibm.com>


So developers who change a certain package can firstly find the
correct package section, then check F-tag to understand who
are the maintainers and reviewers.

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray
> Sent: Tuesday, June 18, 2019 11:31 AM
> To: devel@edk2.groups.io; 'phi...@redhat.com' <phi...@redhat.com>;
> leif.lindh...@linaro.org; Justen, Jordan L <jordan.l.jus...@intel.com>;
> Marc-André Lureau <marcandre.lur...@redhat.com>; Stefan Berger
> <stef...@linux.ibm.com>; Anthony Perard <anthony.per...@citrix.com>;
> Julien Grall <julien.gr...@arm.com>
> Cc: Feng, Bob C <bob.c.f...@intel.com>; Gao, Liming
> <liming....@intel.com>; Andrew Fish <af...@apple.com>; Laszlo Ersek
> <ler...@redhat.com>; Kinney, Michael D <michael.d.kin...@intel.com>;
> Wu, Hao A <hao.a...@intel.com>
> Subject: Re: [edk2-devel] [RFC PATCH 1/3] Maintainers.txt: update for
> filesystem area descriptions
> 
> > +
> > +OvmfPkg (TPM2 modules)
> 
> How about just say "TPM2 modules"?
> 
> > +F: MdePkg/Include/IndustryStandard/Tpm2*
> > +F: SecurityPkg/*/Tpm2*
> > +M: Jordan Justen <jordan.l.jus...@intel.com>
> > +M: Laszlo Ersek <ler...@redhat.com>
> > +M: Ard Biesheuvel <ard.biesheu...@linaro.org>
> >  R: Marc-André Lureau <marcandre.lur...@redhat.com>
> > -   (TPM2 modules)
> >  R: Stefan Berger <stef...@linux.ibm.com>
> > -   (TPM2 modules)
> >  S: Maintained
> > ---
> >
> > Although TPM2 deserves its own section IMHO.
> > (Ditto with ArmVirtPkg Xen).
> 
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#42531): https://edk2.groups.io/g/devel/message/42531
Mute This Topic: https://groups.io/mt/32067940/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to