Re: [edk2-devel] [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format

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

On Wed, Oct 09, 2019 at 02:51:38PM +0200, Philippe Mathieu-Daudé wrote:
> On 10/9/19 2:00 PM, Leif Lindholm wrote:
> > Synchronize header with edk2/Maintainers.txt and change references
> > from "EDK II to EDK II Platforms".
> > 
> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> > ---
> >   Maintainers.txt | 42 +++++++++++++++++++++++++++++++++---------
> >   1 file changed, 33 insertions(+), 9 deletions(-)
> > 
> > diff --git a/Maintainers.txt b/Maintainers.txt
> > index 1260c8d2465b..f824e2b05de1 100644
> > --- a/Maintainers.txt
> > +++ b/Maintainers.txt
> > @@ -1,17 +1,24 @@
> > -EDK II Maintainers
> > -==================
> > +EDK II Platforms Maintainers
> > +============================
> >   This file provides information about the primary maintainers for
> > -EDK II Platforms.
> > +EDK Platforms II.
> 
> This one looks odd :)

I might have been getting a bit cross-eyed at this point.

Thanks!

/
    Leif

> >   In general, you should not privately email the maintainer. You should
> > -email the edk2-devel list, but you can also Cc the maintainer.
> > +email the edk2-devel list, and Cc the package maintainers and
> > +reviewers.
> > +
> > +If the package maintainer wants to hand over the role to other people,
> > +the package maintainer should send the patch to update Maintainers.txt
> > +with new maintainer, and the new maintainer should follow up with
> > +an Acked-by or a Reviewed-by.
> >   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,9 +36,26 @@ 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
> > -------
> > +
> > +EDK II Platforms
> > +----------------
> >   W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II
> >   L: https://edk2.groups.io/g/devel/
> >   T: git - https://github.com/tianocore/edk2-platforms.git
> > @@ -40,8 +64,8 @@ Responsible Disclosure, Reporting Security Issues
> >   -------------------------------------------------
> >   W: https://github.com/tianocore/tianocore.github.io/wiki/Security
> > -EDK II Packages:
> > -----------------
> > +EDK II Platforms Packages:
> > +--------------------------
> >   Drivers/OptionRomPkg
> >   W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
> > 
> 
> 
> 
>