[edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt

Leif Lindholm posted 5 patches 4 years, 4 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
Maintainers.txt | 153 ++++++++++++++++++++++++++++++++++++++++--------
1 file changed, 127 insertions(+), 26 deletions(-)
[edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt
Posted by Leif Lindholm 4 years, 4 months ago
This set does a few things:
- Imports the documentation header from edk2/Maintainers.txt
- Reformats some maintainer entries to get rid of ',' characters
  in the names.
- Adds F: entries for all sections, and a default fallback section
  for those where there is no match.
- Merges the Marvell sections (designed to work with this format)
  into a single area of responsibility.
- Adds sections for those that now ended up having none other than
  the fallback one (mostly ARM platforms).
- Remove Ard from the default section.

I am intentionally not Cc:ing all affected maintainers, since this
is a reformatting exercise. All roles actually affected are Cc:d.

Changes since v1:
- 2/6 ("Maintainers.txt: drop ',' from maintainer names") dropped,
  since commit 3adcb58818ca ("Maintainers.txt: Update Intel package
  maintainers") dropped the affected sections.
- Fixed the spelling of Ard's surname :|
- Fixed other reported typos.
- Added R-b/A-b.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Marcin Wojtas <mw@semihalf.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>

Leif Lindholm (5):
  Maintainers.txt: Update documentation for new format
  Maintainers.txt: revamp file to machine-readable format
  Maintainers.txt: merge Marvell platforms and silicon sections
  Maintainers.txt: create explicit sections for remaining modules
  Maintainers.txt: drop Ard from default section

 Maintainers.txt | 153 ++++++++++++++++++++++++++++++++++++++++--------
 1 file changed, 127 insertions(+), 26 deletions(-)

-- 
2.20.1


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

View/Reply Online (#50587): https://edk2.groups.io/g/devel/message/50587
Mute This Topic: https://groups.io/mt/56099426/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt
Posted by Michael D Kinney 4 years, 4 months ago
Series 

Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On
> Behalf Of Leif Lindholm
> Sent: Wednesday, November 13, 2019 10:00 AM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Marcin
> Wojtas <mw@semihalf.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>
> Subject: [edk2-devel] [PATCH edk2-platforms v2 0/5]
> Update Maintainers.txt
> 
> This set does a few things:
> - Imports the documentation header from
> edk2/Maintainers.txt
> - Reformats some maintainer entries to get rid of ','
> characters
>   in the names.
> - Adds F: entries for all sections, and a default
> fallback section
>   for those where there is no match.
> - Merges the Marvell sections (designed to work with
> this format)
>   into a single area of responsibility.
> - Adds sections for those that now ended up having none
> other than
>   the fallback one (mostly ARM platforms).
> - Remove Ard from the default section.
> 
> I am intentionally not Cc:ing all affected maintainers,
> since this is a reformatting exercise. All roles
> actually affected are Cc:d.
> 
> Changes since v1:
> - 2/6 ("Maintainers.txt: drop ',' from maintainer
> names") dropped,
>   since commit 3adcb58818ca ("Maintainers.txt: Update
> Intel package
>   maintainers") dropped the affected sections.
> - Fixed the spelling of Ard's surname :|
> - Fixed other reported typos.
> - Added R-b/A-b.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Marcin Wojtas <mw@semihalf.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> 
> Leif Lindholm (5):
>   Maintainers.txt: Update documentation for new format
>   Maintainers.txt: revamp file to machine-readable
> format
>   Maintainers.txt: merge Marvell platforms and silicon
> sections
>   Maintainers.txt: create explicit sections for
> remaining modules
>   Maintainers.txt: drop Ard from default section
> 
>  Maintainers.txt | 153
> ++++++++++++++++++++++++++++++++++++++++--------
>  1 file changed, 127 insertions(+), 26 deletions(-)
> 
> --
> 2.20.1
> 
> 
> 


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

View/Reply Online (#50594): https://edk2.groups.io/g/devel/message/50594
Mute This Topic: https://groups.io/mt/56099426/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt
Posted by Leif Lindholm 4 years, 4 months ago
Thanks Mike (and other reviewers).

Pushed as cee28ab1c8da..f7271a06170b.

/
    Leif

On Wed, Nov 13, 2019 at 06:42:16PM +0000, Kinney, Michael D wrote:
> Series 
> 
> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> 
> Mike
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On
> > Behalf Of Leif Lindholm
> > Sent: Wednesday, November 13, 2019 10:00 AM
> > To: devel@edk2.groups.io
> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Marcin
> > Wojtas <mw@semihalf.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Wang, Jian J
> > <jian.j.wang@intel.com>
> > Subject: [edk2-devel] [PATCH edk2-platforms v2 0/5]
> > Update Maintainers.txt
> > 
> > This set does a few things:
> > - Imports the documentation header from
> > edk2/Maintainers.txt
> > - Reformats some maintainer entries to get rid of ','
> > characters
> >   in the names.
> > - Adds F: entries for all sections, and a default
> > fallback section
> >   for those where there is no match.
> > - Merges the Marvell sections (designed to work with
> > this format)
> >   into a single area of responsibility.
> > - Adds sections for those that now ended up having none
> > other than
> >   the fallback one (mostly ARM platforms).
> > - Remove Ard from the default section.
> > 
> > I am intentionally not Cc:ing all affected maintainers,
> > since this is a reformatting exercise. All roles
> > actually affected are Cc:d.
> > 
> > Changes since v1:
> > - 2/6 ("Maintainers.txt: drop ',' from maintainer
> > names") dropped,
> >   since commit 3adcb58818ca ("Maintainers.txt: Update
> > Intel package
> >   maintainers") dropped the affected sections.
> > - Fixed the spelling of Ard's surname :|
> > - Fixed other reported typos.
> > - Added R-b/A-b.
> > 
> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > Cc: Marcin Wojtas <mw@semihalf.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Jian J Wang <jian.j.wang@intel.com>
> > 
> > Leif Lindholm (5):
> >   Maintainers.txt: Update documentation for new format
> >   Maintainers.txt: revamp file to machine-readable
> > format
> >   Maintainers.txt: merge Marvell platforms and silicon
> > sections
> >   Maintainers.txt: create explicit sections for
> > remaining modules
> >   Maintainers.txt: drop Ard from default section
> > 
> >  Maintainers.txt | 153
> > ++++++++++++++++++++++++++++++++++++++++--------
> >  1 file changed, 127 insertions(+), 26 deletions(-)
> > 
> > --
> > 2.20.1
> > 
> > 
> > 
> 

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

View/Reply Online (#50597): https://edk2.groups.io/g/devel/message/50597
Mute This Topic: https://groups.io/mt/56099426/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-