Re: [edk2-devel] [RFC PATCH 1/3] Maintainers.txt: update for filesystem area descriptions

["Philippe Mathieu-Daudé" <philmd@redhat.com>]

On 6/14/19 10:21 PM, Leif Lindholm wrote:
> Add comment describing new F: and X: tags for associating maintainership
> sections with specific filesystem paths, including wildcards.
> 
> Add global section associating *all* code with devel@edk2.groups.io,
> and add a magic '<default>' F: tag directing all modifications that
> do not hit a rule to the stewards.
> 
> Also tag all files in top directory as maintained by the stewards.
> 
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>  Maintainers.txt | 50 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 50 insertions(+)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 762a68465979..cd32f9b00170 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -36,6 +36,20 @@ 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:   drivers/net/    all files in and below drivers/net
> +     F:   drivers/net/*   all files in drivers/net, but not below
> +     F:   */net/*         all files in "any top level directory"/net
> +     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 NetworkPkgnet/Ip6Dxe/
> +  Magic filename "<default>" denotes all files not caught by any F: rule should
> +  be sent.
>  
[...]
>  
>  OvmfPkg
> +F: OvmfPkg/
>  W: http://www.tianocore.org/ovmf/
>  M: Jordan Justen <jordan.l.justen@intel.com>
>  M: Laszlo Ersek <lersek@redhat.com>
> @@ -191,16 +233,19 @@ R: Stefan Berger <stefanb@linux.ibm.com>
>  S: Maintained

Possible follow-up patch:

-- >8 --
 OvmfPkg
 F: OvmfPkg/
 W: http://www.tianocore.org/ovmf/
 M: Jordan Justen <jordan.l.justen@intel.com>
 M: Laszlo Ersek <lersek@redhat.com>
 M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+S: Maintained
+
+OvmfPkg (Xen modules)
+F: OvmfPkg/*/Xen*
+M: Jordan Justen <jordan.l.justen@intel.com>
+M: Laszlo Ersek <lersek@redhat.com>
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
 R: Anthony Perard <anthony.perard@citrix.com>
-   (Xen modules)
 R: Julien Grall <julien.grall@arm.com>
-   (Xen modules)
+S: Maintained
+
+OvmfPkg (TPM2 modules)
+F: MdePkg/Include/IndustryStandard/Tpm2*
+F: SecurityPkg/*/Tpm2*
+M: Jordan Justen <jordan.l.justen@intel.com>
+M: Laszlo Ersek <lersek@redhat.com>
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
 R: Marc-André Lureau <marcandre.lureau@redhat.com>
-   (TPM2 modules)
 R: Stefan Berger <stefanb@linux.ibm.com>
-   (TPM2 modules)
 S: Maintained
---

Although TPM2 deserves its own section IMHO.
(Ditto with ArmVirtPkg Xen).