BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++ Maintainers.txt | 54 ++++++++ 2 files changed, 244 insertions(+) create mode 100644 BaseTools/Scripts/GetMaintainer.py
Changes are available directly from: https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstreaming/git-maintainer-v1 This series adds new tags to the Maintainers.txt format, making it possible to describe which filesystem paths are looked after by which people, and hence automating the extraction of a list over who should be cc:d on a patch submission. Remaining shorcomings in v1: - Will still be misparsing OvmfPkg and MdeModulePkg due to non-tag lines interspersed with the tag lines. These lines will be removed as areas of responsibility is formally rewritten as tags. - * Wildcard support is not fully filesystem compliant except in first or last position in the file pattern (it translates as regex .* elsewhere). However, actual cases of mismatch are expected to be unlikely, and they will be false positives rather than false negatives - so I think this is good enough at least for a start. - Provides no information of why certain people or meiling lists were picked - it just bundles all recipients up, deduplicates them, and prints them out. Using the script requires the gitpython module to be installed. Worthwhile mentioning outside the ChangeLog is the added -l flag, which lets you look up what a given path would return in the way of matches. E.g. "python BaseTools/Scripts/GetMaintainer.py -l Non/Existing/Path" would return: --- Non/Existing/Path "Non/Existing/Path": no maintainers found, looking for default Andrew Fish <afish@apple.com> Laszlo Ersek <lersek@redhat.com> Leif Lindholm <leif.lindholm@linaro.org> Michael D Kinney <michael.d.kinney@intel.com> devel@edk2.groups.io --- This series would still result in GetMaintainers.py missing some maintainers/reviewers due to descriptions in prose rather than filename patterns. My preferred way of handling this would be to merge 1-2/3 as soon as found acceptable, following up and merging patches to update ArmVirtPkg, MdeModulePkg, and OvmfPkg, and finally once the file is consistent, proceed to merge 3/3. Changelog: v1: - Rebase to current Maintainers.txt. - Fix typos and missed bits in Maintainers.txt. - Get rid of the magic '<default>' filename, let the single-char '*' wildcard resolve this (_using_ the magic '<default>' filename in the script, but treating it as if it was a file in the top-level directory). - Add -l flag to script to look up which maintainers would be returned for a given path (which need not exist). rfc: - Split patches up - one for new Maintainers.txt format (documentation and F: tags). - one for adding a new wilcards responsibility area for */Arm, */AArch64 - one for the GetMaintainer.py script - Reworked wildcard handling based on Laszlo's explanation - Trailing / covers everything under that directory - Trailing * does not cover subdirectories - Added support for X: tag - Added support for magic '<default>' pathname - Also prints mailing list addresses for matching L: tags Cc: Andrew Fish <afish@apple.com> Cc: Laszlo Ersek <lersek@redhat.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Cc: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Cc: Philippe Mathieu-Daude <philmd@redhat.com> Cc: "Wu, Hao A" <hao.a.wu@intel.com> Leif Lindholm (3): Maintainers.txt: update for filesystem area descriptions Maintainers.txt: add wildcard path association for Arm/AArch64 BaseTools: add GetMaintainer.py script BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++ Maintainers.txt | 54 ++++++++ 2 files changed, 244 insertions(+) create mode 100644 BaseTools/Scripts/GetMaintainer.py -- 2.20.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#43663): https://edk2.groups.io/g/devel/message/43663 Mute This Topic: https://groups.io/mt/32444044/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
On 07/12/19 19:01, Leif Lindholm wrote: > Changes are available directly from: > https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstreaming/git-maintainer-v1 > > This series adds new tags to the Maintainers.txt format, making it possible > to describe which filesystem paths are looked after by which people, and > hence automating the extraction of a list over who should be cc:d on a patch > submission. > > Remaining shorcomings in v1: > - Will still be misparsing OvmfPkg and MdeModulePkg due to non-tag lines > interspersed with the tag lines. These lines will be removed as areas of > responsibility is formally rewritten as tags. > - * Wildcard support is not fully filesystem compliant except in first or > last position in the file pattern (it translates as regex .* elsewhere). > However, actual cases of mismatch are expected to be unlikely, and they > will be false positives rather than false negatives - so I think this is > good enough at least for a start. > - Provides no information of why certain people or meiling lists were > picked - it just bundles all recipients up, deduplicates them, and > prints them out. > > Using the script requires the gitpython module to be installed. > > Worthwhile mentioning outside the ChangeLog is the added -l flag, which > lets you look up what a given path would return in the way of matches. > E.g. "python BaseTools/Scripts/GetMaintainer.py -l Non/Existing/Path" > would return: > --- > Non/Existing/Path > "Non/Existing/Path": no maintainers found, looking for default > Andrew Fish <afish@apple.com> > Laszlo Ersek <lersek@redhat.com> > Leif Lindholm <leif.lindholm@linaro.org> > Michael D Kinney <michael.d.kinney@intel.com> > devel@edk2.groups.io > --- > > This series would still result in GetMaintainers.py missing some > maintainers/reviewers due to descriptions in prose rather than filename > patterns. My preferred way of handling this would be to merge 1-2/3 as > soon as found acceptable, following up and merging patches to update > ArmVirtPkg, MdeModulePkg, and OvmfPkg, and finally once the file is > consistent, proceed to merge 3/3. Sounds like a plan. Thanks Laszlo > > Changelog: > v1: > - Rebase to current Maintainers.txt. > - Fix typos and missed bits in Maintainers.txt. > - Get rid of the magic '<default>' filename, let the single-char '*' > wildcard resolve this (_using_ the magic '<default>' filename in the > script, but treating it as if it was a file in the top-level directory). > - Add -l flag to script to look up which maintainers would be returned for > a given path (which need not exist). > rfc: > - Split patches up > - one for new Maintainers.txt format (documentation and F: tags). > - one for adding a new wilcards responsibility area for */Arm, */AArch64 > - one for the GetMaintainer.py script > - Reworked wildcard handling based on Laszlo's explanation > - Trailing / covers everything under that directory > - Trailing * does not cover subdirectories > - Added support for X: tag > - Added support for magic '<default>' pathname > - Also prints mailing list addresses for matching L: tags > > Cc: Andrew Fish <afish@apple.com> > Cc: Laszlo Ersek <lersek@redhat.com> > Cc: Michael D Kinney <michael.d.kinney@intel.com> > Cc: Bob Feng <bob.c.feng@intel.com> > Cc: Liming Gao <liming.gao@intel.com> > Cc: Philippe Mathieu-Daude <philmd@redhat.com> > Cc: "Wu, Hao A" <hao.a.wu@intel.com> > > Leif Lindholm (3): > Maintainers.txt: update for filesystem area descriptions > Maintainers.txt: add wildcard path association for Arm/AArch64 > BaseTools: add GetMaintainer.py script > > BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++ > Maintainers.txt | 54 ++++++++ > 2 files changed, 244 insertions(+) > create mode 100644 BaseTools/Scripts/GetMaintainer.py > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#43676): https://edk2.groups.io/g/devel/message/43676 Mute This Topic: https://groups.io/mt/32444044/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Laszlo, Hao - I have prepared a set for pushing as below.
(Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> for all of your
patches.)
*But*, I need a review from Bob and/or Liming before I push the final one:
"BaseTools: add GetMaintainer.py script" -
https://edk2.groups.io/g/devel/message/43666
I have verified reviews/acks by all affected reviewers, except for Stefan
Berger for Ovmf TCG/TPM2.
While I would prefer to have that too, this is a file format shift - not a
change in responsibility areas.
Laszlo, do you agree?
If so, I can push 1-7/8 as is.
/
Leif
Hao A Wu (1):
Maintainers.txt: Fine-grained review ownership for MdeModulePkg
Laszlo Ersek (4):
Maintainers.txt: split out section "ArmVirtPkg: modules used on Xen"
Maintainers.txt: split out section "OvmfPkg: Xen-related modules"
Maintainers.txt: split out section "OvmfPkg: TCG- and TPM2-related
modules"
Maintainers.txt: split out section "OvmfPkg: CSM modules"
Leif Lindholm (3):
Maintainers.txt: update for filesystem area descriptions
Maintainers.txt: add wildcard path association for Arm/AArch64
BaseTools: add GetMaintainer.py script
On Fri, 12 Jul 2019 at 18:01, Leif Lindholm <leif.lindholm@linaro.org>
wrote:
> Changes are available directly from:
>
> https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstreaming/git-maintainer-v1
>
> This series adds new tags to the Maintainers.txt format, making it possible
> to describe which filesystem paths are looked after by which people, and
> hence automating the extraction of a list over who should be cc:d on a
> patch
> submission.
>
> Remaining shorcomings in v1:
> - Will still be misparsing OvmfPkg and MdeModulePkg due to non-tag lines
> interspersed with the tag lines. These lines will be removed as areas of
> responsibility is formally rewritten as tags.
> - * Wildcard support is not fully filesystem compliant except in first or
> last position in the file pattern (it translates as regex .* elsewhere).
> However, actual cases of mismatch are expected to be unlikely, and they
> will be false positives rather than false negatives - so I think this is
> good enough at least for a start.
> - Provides no information of why certain people or meiling lists were
> picked - it just bundles all recipients up, deduplicates them, and
> prints them out.
>
> Using the script requires the gitpython module to be installed.
>
> Worthwhile mentioning outside the ChangeLog is the added -l flag, which
> lets you look up what a given path would return in the way of matches.
> E.g. "python BaseTools/Scripts/GetMaintainer.py -l Non/Existing/Path"
> would return:
> ---
> Non/Existing/Path
> "Non/Existing/Path": no maintainers found, looking for default
> Andrew Fish <afish@apple.com>
> Laszlo Ersek <lersek@redhat.com>
> Leif Lindholm <leif.lindholm@linaro.org>
> Michael D Kinney <michael.d.kinney@intel.com>
> devel@edk2.groups.io
> ---
>
> This series would still result in GetMaintainers.py missing some
> maintainers/reviewers due to descriptions in prose rather than filename
> patterns. My preferred way of handling this would be to merge 1-2/3 as
> soon as found acceptable, following up and merging patches to update
> ArmVirtPkg, MdeModulePkg, and OvmfPkg, and finally once the file is
> consistent, proceed to merge 3/3.
>
> Changelog:
> v1:
> - Rebase to current Maintainers.txt.
> - Fix typos and missed bits in Maintainers.txt.
> - Get rid of the magic '<default>' filename, let the single-char '*'
> wildcard resolve this (_using_ the magic '<default>' filename in the
> script, but treating it as if it was a file in the top-level directory).
> - Add -l flag to script to look up which maintainers would be returned for
> a given path (which need not exist).
> rfc:
> - Split patches up
> - one for new Maintainers.txt format (documentation and F: tags).
> - one for adding a new wilcards responsibility area for */Arm, */AArch64
> - one for the GetMaintainer.py script
> - Reworked wildcard handling based on Laszlo's explanation
> - Trailing / covers everything under that directory
> - Trailing * does not cover subdirectories
> - Added support for X: tag
> - Added support for magic '<default>' pathname
> - Also prints mailing list addresses for matching L: tags
>
> Cc: Andrew Fish <afish@apple.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Philippe Mathieu-Daude <philmd@redhat.com>
> Cc: "Wu, Hao A" <hao.a.wu@intel.com>
>
> Leif Lindholm (3):
> Maintainers.txt: update for filesystem area descriptions
> Maintainers.txt: add wildcard path association for Arm/AArch64
> BaseTools: add GetMaintainer.py script
>
> BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++
> Maintainers.txt | 54 ++++++++
> 2 files changed, 244 insertions(+)
> create mode 100644 BaseTools/Scripts/GetMaintainer.py
>
> --
> 2.20.1
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#43963): https://edk2.groups.io/g/devel/message/43963
Mute This Topic: https://groups.io/mt/32444044/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Hi Leif, On 7/18/19 6:27 PM, Leif Lindholm wrote: > Laszlo, Hao - I have prepared a set for pushing as below. > > (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org > <mailto:leif.lindholm@linaro.org>> for all of your patches.) > *But*, I need a review from Bob and/or Liming before I push the final one: > "BaseTools: add GetMaintainer.py script" - > https://edk2.groups.io/g/devel/message/43666 > > I have verified reviews/acks by all affected reviewers, except for > Stefan Berger for Ovmf TCG/TPM2. > While I would prefer to have that too, this is a file format shift - not > a change in responsibility areas. As Marc-André and Stefan have the same responsabilities, Marc-André's review for their common section should be sufficient, right? Now if you want to be kind and patient with Stefan, since this is not a logical change blocking anyone, it might be wise to wait over the weekend. My 2 cents ;) Phil. > Laszlo, do you agree? > If so, I can push 1-7/8 as is. > > / > Leif > > Hao A Wu (1): > Maintainers.txt: Fine-grained review ownership for MdeModulePkg > > Laszlo Ersek (4): > Maintainers.txt: split out section "ArmVirtPkg: modules used on Xen" > Maintainers.txt: split out section "OvmfPkg: Xen-related modules" > Maintainers.txt: split out section "OvmfPkg: TCG- and TPM2-related > modules" > Maintainers.txt: split out section "OvmfPkg: CSM modules" > > Leif Lindholm (3): > Maintainers.txt: update for filesystem area descriptions > Maintainers.txt: add wildcard path association for Arm/AArch64 > BaseTools: add GetMaintainer.py script > > > On Fri, 12 Jul 2019 at 18:01, Leif Lindholm <leif.lindholm@linaro.org > <mailto:leif.lindholm@linaro.org>> wrote: > > Changes are available directly from: > https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstreaming/git-maintainer-v1 > > This series adds new tags to the Maintainers.txt format, making it > possible > to describe which filesystem paths are looked after by which people, and > hence automating the extraction of a list over who should be cc:d on > a patch > submission. > > Remaining shorcomings in v1: > - Will still be misparsing OvmfPkg and MdeModulePkg due to non-tag lines > interspersed with the tag lines. These lines will be removed as > areas of > responsibility is formally rewritten as tags. > - * Wildcard support is not fully filesystem compliant except in > first or > last position in the file pattern (it translates as regex .* > elsewhere). > However, actual cases of mismatch are expected to be unlikely, and > they > will be false positives rather than false negatives - so I think > this is > good enough at least for a start. > - Provides no information of why certain people or meiling lists were > picked - it just bundles all recipients up, deduplicates them, and > prints them out. > > Using the script requires the gitpython module to be installed. > > Worthwhile mentioning outside the ChangeLog is the added -l flag, which > lets you look up what a given path would return in the way of matches. > E.g. "python BaseTools/Scripts/GetMaintainer.py -l Non/Existing/Path" > would return: > --- > Non/Existing/Path > "Non/Existing/Path": no maintainers found, looking for default > Andrew Fish <afish@apple.com <mailto:afish@apple.com>> > Laszlo Ersek <lersek@redhat.com <mailto:lersek@redhat.com>> > Leif Lindholm <leif.lindholm@linaro.org > <mailto:leif.lindholm@linaro.org>> > Michael D Kinney <michael.d.kinney@intel.com > <mailto:michael.d.kinney@intel.com>> > devel@edk2.groups.io <mailto:devel@edk2.groups.io> > --- > > This series would still result in GetMaintainers.py missing some > maintainers/reviewers due to descriptions in prose rather than filename > patterns. My preferred way of handling this would be to merge 1-2/3 as > soon as found acceptable, following up and merging patches to update > ArmVirtPkg, MdeModulePkg, and OvmfPkg, and finally once the file is > consistent, proceed to merge 3/3. > > Changelog: > v1: > - Rebase to current Maintainers.txt. > - Fix typos and missed bits in Maintainers.txt. > - Get rid of the magic '<default>' filename, let the single-char '*' > wildcard resolve this (_using_ the magic '<default>' filename in the > script, but treating it as if it was a file in the top-level > directory). > - Add -l flag to script to look up which maintainers would be > returned for > a given path (which need not exist). > rfc: > - Split patches up > - one for new Maintainers.txt format (documentation and F: tags). > - one for adding a new wilcards responsibility area for */Arm, > */AArch64 > - one for the GetMaintainer.py script > - Reworked wildcard handling based on Laszlo's explanation > - Trailing / covers everything under that directory > - Trailing * does not cover subdirectories > - Added support for X: tag > - Added support for magic '<default>' pathname > - Also prints mailing list addresses for matching L: tags > > Cc: Andrew Fish <afish@apple.com <mailto:afish@apple.com>> > Cc: Laszlo Ersek <lersek@redhat.com <mailto:lersek@redhat.com>> > Cc: Michael D Kinney <michael.d.kinney@intel.com > <mailto:michael.d.kinney@intel.com>> > Cc: Bob Feng <bob.c.feng@intel.com <mailto:bob.c.feng@intel.com>> > Cc: Liming Gao <liming.gao@intel.com <mailto:liming.gao@intel.com>> > Cc: Philippe Mathieu-Daude <philmd@redhat.com > <mailto:philmd@redhat.com>> > Cc: "Wu, Hao A" <hao.a.wu@intel.com <mailto:hao.a.wu@intel.com>> > > Leif Lindholm (3): > Maintainers.txt: update for filesystem area descriptions > Maintainers.txt: add wildcard path association for Arm/AArch64 > BaseTools: add GetMaintainer.py script > > BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++ > Maintainers.txt | 54 ++++++++ > 2 files changed, 244 insertions(+) > create mode 100644 BaseTools/Scripts/GetMaintainer.py > > -- > 2.20.1 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#43966): https://edk2.groups.io/g/devel/message/43966 Mute This Topic: https://groups.io/mt/32444044/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
On Thu, Jul 18, 2019 at 07:01:40PM +0200, Philippe Mathieu-Daudé wrote:
> Hi Leif,
>
> On 7/18/19 6:27 PM, Leif Lindholm wrote:
> > Laszlo, Hao - I have prepared a set for pushing as below.
> >
> > (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org
> > <mailto:leif.lindholm@linaro.org>> for all of your patches.)
> > *But*, I need a review from Bob and/or Liming before I push the final one:
> > "BaseTools: add GetMaintainer.py script" -
> > https://edk2.groups.io/g/devel/message/43666
> >
> > I have verified reviews/acks by all affected reviewers, except for
> > Stefan Berger for Ovmf TCG/TPM2.
> > While I would prefer to have that too, this is a file format shift - not
> > a change in responsibility areas.
>
> As Marc-André and Stefan have the same responsabilities, Marc-André's
> review for their common section should be sufficient, right?
Pretty much.
> Now if you want to be kind and patient with Stefan, since this is not a
> logical change blocking anyone, it might be wise to wait over the weekend.
>
> My 2 cents ;)
Yeah, only thing is it would be nice if we could get the script in
(which is dependent on the Maintainers.txt changes in order to not
give misleading results) with some time to spare (and for it to see
real-world use) before we start the freeze for edk2-stable201908.
/
Leif
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#43967): https://edk2.groups.io/g/devel/message/43967
Mute This Topic: https://groups.io/mt/32444044/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
On 7/18/19 7:40 PM, Leif Lindholm wrote: > On Thu, Jul 18, 2019 at 07:01:40PM +0200, Philippe Mathieu-Daudé wrote: >> Hi Leif, >> >> On 7/18/19 6:27 PM, Leif Lindholm wrote: >>> Laszlo, Hao - I have prepared a set for pushing as below. >>> >>> (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org >>> <mailto:leif.lindholm@linaro.org>> for all of your patches.) >>> *But*, I need a review from Bob and/or Liming before I push the final one: >>> "BaseTools: add GetMaintainer.py script" - >>> https://edk2.groups.io/g/devel/message/43666 >>> >>> I have verified reviews/acks by all affected reviewers, except for >>> Stefan Berger for Ovmf TCG/TPM2. >>> While I would prefer to have that too, this is a file format shift - not >>> a change in responsibility areas. >> >> As Marc-André and Stefan have the same responsabilities, Marc-André's >> review for their common section should be sufficient, right? > > Pretty much. > >> Now if you want to be kind and patient with Stefan, since this is not a >> logical change blocking anyone, it might be wise to wait over the weekend. >> >> My 2 cents ;) > > Yeah, only thing is it would be nice if we could get the script in > (which is dependent on the Maintainers.txt changes in order to not > give misleading results) with some time to spare (and for it to see > real-world use) before we start the freeze for edk2-stable201908. Then I insist, since Marc-André reviewed the patch, except for being particularly kind, I don't see any reason to not merge your series ;) Regards, Phil. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#43968): https://edk2.groups.io/g/devel/message/43968 Mute This Topic: https://groups.io/mt/32444044/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
On 07/18/19 18:27, Leif Lindholm wrote: > Laszlo, Hao - I have prepared a set for pushing as below. > > (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> for all of your > patches.) > *But*, I need a review from Bob and/or Liming before I push the final one: > "BaseTools: add GetMaintainer.py script" - > https://edk2.groups.io/g/devel/message/43666 > > I have verified reviews/acks by all affected reviewers, except for Stefan > Berger for Ovmf TCG/TPM2. Stefan's R-b has arrived as well: http://mid.mail-archive.com/7201b3fb-d84b-30e0-61c8-bf9bb9defa7a@linux.ibm.com So, I agree, those four patches regarding OvmfPkg reviewership are now fully approved. Please pick up Phil's feedback as well. > While I would prefer to have that too, this is a file format shift - not a > change in responsibility areas. > Laszlo, do you agree? > If so, I can push 1-7/8 as is. See above -- then I'm OK. Thank you! Laszlo > > / > Leif > > Hao A Wu (1): > Maintainers.txt: Fine-grained review ownership for MdeModulePkg > > Laszlo Ersek (4): > Maintainers.txt: split out section "ArmVirtPkg: modules used on Xen" > Maintainers.txt: split out section "OvmfPkg: Xen-related modules" > Maintainers.txt: split out section "OvmfPkg: TCG- and TPM2-related > modules" > Maintainers.txt: split out section "OvmfPkg: CSM modules" > > Leif Lindholm (3): > Maintainers.txt: update for filesystem area descriptions > Maintainers.txt: add wildcard path association for Arm/AArch64 > BaseTools: add GetMaintainer.py script > > > On Fri, 12 Jul 2019 at 18:01, Leif Lindholm <leif.lindholm@linaro.org> > wrote: > >> Changes are available directly from: >> >> https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstreaming/git-maintainer-v1 >> >> This series adds new tags to the Maintainers.txt format, making it possible >> to describe which filesystem paths are looked after by which people, and >> hence automating the extraction of a list over who should be cc:d on a >> patch >> submission. >> >> Remaining shorcomings in v1: >> - Will still be misparsing OvmfPkg and MdeModulePkg due to non-tag lines >> interspersed with the tag lines. These lines will be removed as areas of >> responsibility is formally rewritten as tags. >> - * Wildcard support is not fully filesystem compliant except in first or >> last position in the file pattern (it translates as regex .* elsewhere). >> However, actual cases of mismatch are expected to be unlikely, and they >> will be false positives rather than false negatives - so I think this is >> good enough at least for a start. >> - Provides no information of why certain people or meiling lists were >> picked - it just bundles all recipients up, deduplicates them, and >> prints them out. >> >> Using the script requires the gitpython module to be installed. >> >> Worthwhile mentioning outside the ChangeLog is the added -l flag, which >> lets you look up what a given path would return in the way of matches. >> E.g. "python BaseTools/Scripts/GetMaintainer.py -l Non/Existing/Path" >> would return: >> --- >> Non/Existing/Path >> "Non/Existing/Path": no maintainers found, looking for default >> Andrew Fish <afish@apple.com> >> Laszlo Ersek <lersek@redhat.com> >> Leif Lindholm <leif.lindholm@linaro.org> >> Michael D Kinney <michael.d.kinney@intel.com> >> devel@edk2.groups.io >> --- >> >> This series would still result in GetMaintainers.py missing some >> maintainers/reviewers due to descriptions in prose rather than filename >> patterns. My preferred way of handling this would be to merge 1-2/3 as >> soon as found acceptable, following up and merging patches to update >> ArmVirtPkg, MdeModulePkg, and OvmfPkg, and finally once the file is >> consistent, proceed to merge 3/3. >> >> Changelog: >> v1: >> - Rebase to current Maintainers.txt. >> - Fix typos and missed bits in Maintainers.txt. >> - Get rid of the magic '<default>' filename, let the single-char '*' >> wildcard resolve this (_using_ the magic '<default>' filename in the >> script, but treating it as if it was a file in the top-level directory). >> - Add -l flag to script to look up which maintainers would be returned for >> a given path (which need not exist). >> rfc: >> - Split patches up >> - one for new Maintainers.txt format (documentation and F: tags). >> - one for adding a new wilcards responsibility area for */Arm, */AArch64 >> - one for the GetMaintainer.py script >> - Reworked wildcard handling based on Laszlo's explanation >> - Trailing / covers everything under that directory >> - Trailing * does not cover subdirectories >> - Added support for X: tag >> - Added support for magic '<default>' pathname >> - Also prints mailing list addresses for matching L: tags >> >> Cc: Andrew Fish <afish@apple.com> >> Cc: Laszlo Ersek <lersek@redhat.com> >> Cc: Michael D Kinney <michael.d.kinney@intel.com> >> Cc: Bob Feng <bob.c.feng@intel.com> >> Cc: Liming Gao <liming.gao@intel.com> >> Cc: Philippe Mathieu-Daude <philmd@redhat.com> >> Cc: "Wu, Hao A" <hao.a.wu@intel.com> >> >> Leif Lindholm (3): >> Maintainers.txt: update for filesystem area descriptions >> Maintainers.txt: add wildcard path association for Arm/AArch64 >> BaseTools: add GetMaintainer.py script >> >> BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++ >> Maintainers.txt | 54 ++++++++ >> 2 files changed, 244 insertions(+) >> create mode 100644 BaseTools/Scripts/GetMaintainer.py >> >> -- >> 2.20.1 >> >> > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#43972): https://edk2.groups.io/g/devel/message/43972 Mute This Topic: https://groups.io/mt/32444044/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
On Thu, Jul 18, 2019 at 11:17:30PM +0200, Laszlo Ersek wrote: > On 07/18/19 18:27, Leif Lindholm wrote: > > Laszlo, Hao - I have prepared a set for pushing as below. > > > > (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> for all of your > > patches.) > > *But*, I need a review from Bob and/or Liming before I push the final one: > > "BaseTools: add GetMaintainer.py script" - > > https://edk2.groups.io/g/devel/message/43666 > > > > I have verified reviews/acks by all affected reviewers, except for Stefan > > Berger for Ovmf TCG/TPM2. > > Stefan's R-b has arrived as well: > > http://mid.mail-archive.com/7201b3fb-d84b-30e0-61c8-bf9bb9defa7a@linux.ibm.com > > So, I agree, those four patches regarding OvmfPkg reviewership are now > fully approved. > > Please pick up Phil's feedback as well. Have done. Staging branch in place at https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstreaming/git-maintainer-v2 I'll push 1-7 tomorrow morning my time even if I don't get the R-b from BaseTools Maintainers for 8/8 by then, and the whole set if I do. > > While I would prefer to have that too, this is a file format shift - not a > > change in responsibility areas. > > Laszlo, do you agree? > > If so, I can push 1-7/8 as is. > > See above -- then I'm OK. Thanks! Best Regards, Leif -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#43973): https://edk2.groups.io/g/devel/message/43973 Mute This Topic: https://groups.io/mt/32444044/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Sorry, I miss this mail. I am OK for this change. Ack-by: Liming Gao <liming.gao@intel.com> Thanks Liming > -----Original Message----- > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > Sent: Friday, July 19, 2019 5:57 AM > To: Laszlo Ersek <lersek@redhat.com> > Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Andrew Fish <afish@apple.com>; Kinney, Michael D <michael.d.kinney@intel.com>; > Philippe Mathieu-Daude <philmd@redhat.com>; Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <liming.gao@intel.com>; Wu, Hao A > <hao.a.wu@intel.com>; Stefan Berger <stefanb@linux.ibm.com> > Subject: Re: [PATCH 0/3] add GetMaintainer.py helper script > > On Thu, Jul 18, 2019 at 11:17:30PM +0200, Laszlo Ersek wrote: > > On 07/18/19 18:27, Leif Lindholm wrote: > > > Laszlo, Hao - I have prepared a set for pushing as below. > > > > > > (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> for all of your > > > patches.) > > > *But*, I need a review from Bob and/or Liming before I push the final one: > > > "BaseTools: add GetMaintainer.py script" - > > > https://edk2.groups.io/g/devel/message/43666 > > > > > > I have verified reviews/acks by all affected reviewers, except for Stefan > > > Berger for Ovmf TCG/TPM2. > > > > Stefan's R-b has arrived as well: > > > > http://mid.mail-archive.com/7201b3fb-d84b-30e0-61c8-bf9bb9defa7a@linux.ibm.com > > > > So, I agree, those four patches regarding OvmfPkg reviewership are now > > fully approved. > > > > Please pick up Phil's feedback as well. > > Have done. > Staging branch in place at > https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstreaming/git-maintainer-v2 > > I'll push 1-7 tomorrow morning my time even if I don't get the R-b > from BaseTools Maintainers for 8/8 by then, and the whole set if I do. > > > > While I would prefer to have that too, this is a file format shift - not a > > > change in responsibility areas. > > > Laszlo, do you agree? > > > If so, I can push 1-7/8 as is. > > > > See above -- then I'm OK. > > Thanks! > > Best Regards, > > Leif -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#44275): https://edk2.groups.io/g/devel/message/44275 Mute This Topic: https://groups.io/mt/32444044/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
On Thu, Jul 18, 2019 at 11:17:30PM +0200, Laszlo Ersek wrote:
> On 07/18/19 18:27, Leif Lindholm wrote:
> > Laszlo, Hao - I have prepared a set for pushing as below.
> >
> > (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> for all of your
> > patches.)
> > *But*, I need a review from Bob and/or Liming before I push the final one:
> > "BaseTools: add GetMaintainer.py script" -
> > https://edk2.groups.io/g/devel/message/43666
> >
> > I have verified reviews/acks by all affected reviewers, except for Stefan
> > Berger for Ovmf TCG/TPM2.
>
> Stefan's R-b has arrived as well:
>
> http://mid.mail-archive.com/7201b3fb-d84b-30e0-61c8-bf9bb9defa7a@linux.ibm.com
>
> So, I agree, those four patches regarding OvmfPkg reviewership are now
> fully approved.
>
> Please pick up Phil's feedback as well.
>
> > While I would prefer to have that too, this is a file format shift - not a
> > change in responsibility areas.
> > Laszlo, do you agree?
> > If so, I can push 1-7/8 as is.
>
> See above -- then I'm OK.
1-7/8 pushed as:
3dafa0382286..296c908c6968
Will push the GetMaintainer.py script once approved by BaseTools
maintainers.
Thanks!
/
Leif
> Thank you!
> Laszlo
>
> >
> > /
> > Leif
> >
> > Hao A Wu (1):
> > Maintainers.txt: Fine-grained review ownership for MdeModulePkg
> >
> > Laszlo Ersek (4):
> > Maintainers.txt: split out section "ArmVirtPkg: modules used on Xen"
> > Maintainers.txt: split out section "OvmfPkg: Xen-related modules"
> > Maintainers.txt: split out section "OvmfPkg: TCG- and TPM2-related
> > modules"
> > Maintainers.txt: split out section "OvmfPkg: CSM modules"
> >
> > Leif Lindholm (3):
> > Maintainers.txt: update for filesystem area descriptions
> > Maintainers.txt: add wildcard path association for Arm/AArch64
> > BaseTools: add GetMaintainer.py script
> >
> >
> > On Fri, 12 Jul 2019 at 18:01, Leif Lindholm <leif.lindholm@linaro.org>
> > wrote:
> >
> >> Changes are available directly from:
> >>
> >> https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstreaming/git-maintainer-v1
> >>
> >> This series adds new tags to the Maintainers.txt format, making it possible
> >> to describe which filesystem paths are looked after by which people, and
> >> hence automating the extraction of a list over who should be cc:d on a
> >> patch
> >> submission.
> >>
> >> Remaining shorcomings in v1:
> >> - Will still be misparsing OvmfPkg and MdeModulePkg due to non-tag lines
> >> interspersed with the tag lines. These lines will be removed as areas of
> >> responsibility is formally rewritten as tags.
> >> - * Wildcard support is not fully filesystem compliant except in first or
> >> last position in the file pattern (it translates as regex .* elsewhere).
> >> However, actual cases of mismatch are expected to be unlikely, and they
> >> will be false positives rather than false negatives - so I think this is
> >> good enough at least for a start.
> >> - Provides no information of why certain people or meiling lists were
> >> picked - it just bundles all recipients up, deduplicates them, and
> >> prints them out.
> >>
> >> Using the script requires the gitpython module to be installed.
> >>
> >> Worthwhile mentioning outside the ChangeLog is the added -l flag, which
> >> lets you look up what a given path would return in the way of matches.
> >> E.g. "python BaseTools/Scripts/GetMaintainer.py -l Non/Existing/Path"
> >> would return:
> >> ---
> >> Non/Existing/Path
> >> "Non/Existing/Path": no maintainers found, looking for default
> >> Andrew Fish <afish@apple.com>
> >> Laszlo Ersek <lersek@redhat.com>
> >> Leif Lindholm <leif.lindholm@linaro.org>
> >> Michael D Kinney <michael.d.kinney@intel.com>
> >> devel@edk2.groups.io
> >> ---
> >>
> >> This series would still result in GetMaintainers.py missing some
> >> maintainers/reviewers due to descriptions in prose rather than filename
> >> patterns. My preferred way of handling this would be to merge 1-2/3 as
> >> soon as found acceptable, following up and merging patches to update
> >> ArmVirtPkg, MdeModulePkg, and OvmfPkg, and finally once the file is
> >> consistent, proceed to merge 3/3.
> >>
> >> Changelog:
> >> v1:
> >> - Rebase to current Maintainers.txt.
> >> - Fix typos and missed bits in Maintainers.txt.
> >> - Get rid of the magic '<default>' filename, let the single-char '*'
> >> wildcard resolve this (_using_ the magic '<default>' filename in the
> >> script, but treating it as if it was a file in the top-level directory).
> >> - Add -l flag to script to look up which maintainers would be returned for
> >> a given path (which need not exist).
> >> rfc:
> >> - Split patches up
> >> - one for new Maintainers.txt format (documentation and F: tags).
> >> - one for adding a new wilcards responsibility area for */Arm, */AArch64
> >> - one for the GetMaintainer.py script
> >> - Reworked wildcard handling based on Laszlo's explanation
> >> - Trailing / covers everything under that directory
> >> - Trailing * does not cover subdirectories
> >> - Added support for X: tag
> >> - Added support for magic '<default>' pathname
> >> - Also prints mailing list addresses for matching L: tags
> >>
> >> Cc: Andrew Fish <afish@apple.com>
> >> Cc: Laszlo Ersek <lersek@redhat.com>
> >> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> >> Cc: Bob Feng <bob.c.feng@intel.com>
> >> Cc: Liming Gao <liming.gao@intel.com>
> >> Cc: Philippe Mathieu-Daude <philmd@redhat.com>
> >> Cc: "Wu, Hao A" <hao.a.wu@intel.com>
> >>
> >> Leif Lindholm (3):
> >> Maintainers.txt: update for filesystem area descriptions
> >> Maintainers.txt: add wildcard path association for Arm/AArch64
> >> BaseTools: add GetMaintainer.py script
> >>
> >> BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++
> >> Maintainers.txt | 54 ++++++++
> >> 2 files changed, 244 insertions(+)
> >> create mode 100644 BaseTools/Scripts/GetMaintainer.py
> >>
> >> --
> >> 2.20.1
> >>
> >>
> >
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#44033): https://edk2.groups.io/g/devel/message/44033
Mute This Topic: https://groups.io/mt/32524902/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Patch looks fine for me.
Reviewed-by: Bob Feng <bob.c.feng@intel.com>
-----Original Message-----
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Leif Lindholm
Sent: Friday, July 19, 2019 5:00 PM
To: Laszlo Ersek <lersek@redhat.com>
Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Andrew Fish <afish@apple.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Philippe Mathieu-Daude <philmd@redhat.com>; Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <liming.gao@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Stefan Berger <stefanb@linux.ibm.com>
Subject: Re: [edk2-devel] [PATCH 0/3] add GetMaintainer.py helper script (and Maintainers.txt format change)
On Thu, Jul 18, 2019 at 11:17:30PM +0200, Laszlo Ersek wrote:
> On 07/18/19 18:27, Leif Lindholm wrote:
> > Laszlo, Hao - I have prepared a set for pushing as below.
> >
> > (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> for all of
> > your
> > patches.)
> > *But*, I need a review from Bob and/or Liming before I push the final one:
> > "BaseTools: add GetMaintainer.py script" -
> > https://edk2.groups.io/g/devel/message/43666
> >
> > I have verified reviews/acks by all affected reviewers, except for
> > Stefan Berger for Ovmf TCG/TPM2.
>
> Stefan's R-b has arrived as well:
>
> http://mid.mail-archive.com/7201b3fb-d84b-30e0-61c8-bf9bb9defa7a@linux
> .ibm.com
>
> So, I agree, those four patches regarding OvmfPkg reviewership are now
> fully approved.
>
> Please pick up Phil's feedback as well.
>
> > While I would prefer to have that too, this is a file format shift -
> > not a change in responsibility areas.
> > Laszlo, do you agree?
> > If so, I can push 1-7/8 as is.
>
> See above -- then I'm OK.
1-7/8 pushed as:
3dafa0382286..296c908c6968
Will push the GetMaintainer.py script once approved by BaseTools maintainers.
Thanks!
/
Leif
> Thank you!
> Laszlo
>
> >
> > /
> > Leif
> >
> > Hao A Wu (1):
> > Maintainers.txt: Fine-grained review ownership for MdeModulePkg
> >
> > Laszlo Ersek (4):
> > Maintainers.txt: split out section "ArmVirtPkg: modules used on Xen"
> > Maintainers.txt: split out section "OvmfPkg: Xen-related modules"
> > Maintainers.txt: split out section "OvmfPkg: TCG- and TPM2-related
> > modules"
> > Maintainers.txt: split out section "OvmfPkg: CSM modules"
> >
> > Leif Lindholm (3):
> > Maintainers.txt: update for filesystem area descriptions
> > Maintainers.txt: add wildcard path association for Arm/AArch64
> > BaseTools: add GetMaintainer.py script
> >
> >
> > On Fri, 12 Jul 2019 at 18:01, Leif Lindholm
> > <leif.lindholm@linaro.org>
> > wrote:
> >
> >> Changes are available directly from:
> >>
> >> https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstrea
> >> ming/git-maintainer-v1
> >>
> >> This series adds new tags to the Maintainers.txt format, making it
> >> possible to describe which filesystem paths are looked after by
> >> which people, and hence automating the extraction of a list over
> >> who should be cc:d on a patch submission.
> >>
> >> Remaining shorcomings in v1:
> >> - Will still be misparsing OvmfPkg and MdeModulePkg due to non-tag lines
> >> interspersed with the tag lines. These lines will be removed as areas of
> >> responsibility is formally rewritten as tags.
> >> - * Wildcard support is not fully filesystem compliant except in first or
> >> last position in the file pattern (it translates as regex .* elsewhere).
> >> However, actual cases of mismatch are expected to be unlikely, and they
> >> will be false positives rather than false negatives - so I think this is
> >> good enough at least for a start.
> >> - Provides no information of why certain people or meiling lists were
> >> picked - it just bundles all recipients up, deduplicates them, and
> >> prints them out.
> >>
> >> Using the script requires the gitpython module to be installed.
> >>
> >> Worthwhile mentioning outside the ChangeLog is the added -l flag,
> >> which lets you look up what a given path would return in the way of matches.
> >> E.g. "python BaseTools/Scripts/GetMaintainer.py -l Non/Existing/Path"
> >> would return:
> >> ---
> >> Non/Existing/Path
> >> "Non/Existing/Path": no maintainers found, looking for default
> >> Andrew Fish <afish@apple.com>
> >> Laszlo Ersek <lersek@redhat.com>
> >> Leif Lindholm <leif.lindholm@linaro.org>
> >> Michael D Kinney <michael.d.kinney@intel.com>
> >> devel@edk2.groups.io
> >> ---
> >>
> >> This series would still result in GetMaintainers.py missing some
> >> maintainers/reviewers due to descriptions in prose rather than
> >> filename patterns. My preferred way of handling this would be to
> >> merge 1-2/3 as soon as found acceptable, following up and merging
> >> patches to update ArmVirtPkg, MdeModulePkg, and OvmfPkg, and
> >> finally once the file is consistent, proceed to merge 3/3.
> >>
> >> Changelog:
> >> v1:
> >> - Rebase to current Maintainers.txt.
> >> - Fix typos and missed bits in Maintainers.txt.
> >> - Get rid of the magic '<default>' filename, let the single-char '*'
> >> wildcard resolve this (_using_ the magic '<default>' filename in the
> >> script, but treating it as if it was a file in the top-level directory).
> >> - Add -l flag to script to look up which maintainers would be returned for
> >> a given path (which need not exist).
> >> rfc:
> >> - Split patches up
> >> - one for new Maintainers.txt format (documentation and F: tags).
> >> - one for adding a new wilcards responsibility area for */Arm, */AArch64
> >> - one for the GetMaintainer.py script
> >> - Reworked wildcard handling based on Laszlo's explanation
> >> - Trailing / covers everything under that directory
> >> - Trailing * does not cover subdirectories
> >> - Added support for X: tag
> >> - Added support for magic '<default>' pathname
> >> - Also prints mailing list addresses for matching L: tags
> >>
> >> Cc: Andrew Fish <afish@apple.com>
> >> Cc: Laszlo Ersek <lersek@redhat.com>
> >> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> >> Cc: Bob Feng <bob.c.feng@intel.com>
> >> Cc: Liming Gao <liming.gao@intel.com>
> >> Cc: Philippe Mathieu-Daude <philmd@redhat.com>
> >> Cc: "Wu, Hao A" <hao.a.wu@intel.com>
> >>
> >> Leif Lindholm (3):
> >> Maintainers.txt: update for filesystem area descriptions
> >> Maintainers.txt: add wildcard path association for Arm/AArch64
> >> BaseTools: add GetMaintainer.py script
> >>
> >> BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++
> >> Maintainers.txt | 54 ++++++++
> >> 2 files changed, 244 insertions(+) create mode 100644
> >> BaseTools/Scripts/GetMaintainer.py
> >>
> >> --
> >> 2.20.1
> >>
> >>
> >
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#44280): https://edk2.groups.io/g/devel/message/44280
Mute This Topic: https://groups.io/mt/32524902/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Thanks Bob, Liming. GetMaintainer.py patch pushed as 7d0a56c4a125 On Wed, Jul 24, 2019 at 02:03:34AM +0000, Feng, Bob C wrote: > Patch looks fine for me. > > Reviewed-by: Bob Feng <bob.c.feng@intel.com> > > -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Leif Lindholm > Sent: Friday, July 19, 2019 5:00 PM > To: Laszlo Ersek <lersek@redhat.com> > Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Andrew Fish <afish@apple.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Philippe Mathieu-Daude <philmd@redhat.com>; Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <liming.gao@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Stefan Berger <stefanb@linux.ibm.com> > Subject: Re: [edk2-devel] [PATCH 0/3] add GetMaintainer.py helper script (and Maintainers.txt format change) > > On Thu, Jul 18, 2019 at 11:17:30PM +0200, Laszlo Ersek wrote: > > On 07/18/19 18:27, Leif Lindholm wrote: > > > Laszlo, Hao - I have prepared a set for pushing as below. > > > > > > (Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> for all of > > > your > > > patches.) > > > *But*, I need a review from Bob and/or Liming before I push the final one: > > > "BaseTools: add GetMaintainer.py script" - > > > https://edk2.groups.io/g/devel/message/43666 > > > > > > I have verified reviews/acks by all affected reviewers, except for > > > Stefan Berger for Ovmf TCG/TPM2. > > > > Stefan's R-b has arrived as well: > > > > http://mid.mail-archive.com/7201b3fb-d84b-30e0-61c8-bf9bb9defa7a@linux > > .ibm.com > > > > So, I agree, those four patches regarding OvmfPkg reviewership are now > > fully approved. > > > > Please pick up Phil's feedback as well. > > > > > While I would prefer to have that too, this is a file format shift - > > > not a change in responsibility areas. > > > Laszlo, do you agree? > > > If so, I can push 1-7/8 as is. > > > > See above -- then I'm OK. > > 1-7/8 pushed as: > 3dafa0382286..296c908c6968 > > Will push the GetMaintainer.py script once approved by BaseTools maintainers. > > Thanks! > > / > Leif > > > Thank you! > > Laszlo > > > > > > > > / > > > Leif > > > > > > Hao A Wu (1): > > > Maintainers.txt: Fine-grained review ownership for MdeModulePkg > > > > > > Laszlo Ersek (4): > > > Maintainers.txt: split out section "ArmVirtPkg: modules used on Xen" > > > Maintainers.txt: split out section "OvmfPkg: Xen-related modules" > > > Maintainers.txt: split out section "OvmfPkg: TCG- and TPM2-related > > > modules" > > > Maintainers.txt: split out section "OvmfPkg: CSM modules" > > > > > > Leif Lindholm (3): > > > Maintainers.txt: update for filesystem area descriptions > > > Maintainers.txt: add wildcard path association for Arm/AArch64 > > > BaseTools: add GetMaintainer.py script > > > > > > > > > On Fri, 12 Jul 2019 at 18:01, Leif Lindholm > > > <leif.lindholm@linaro.org> > > > wrote: > > > > > >> Changes are available directly from: > > >> > > >> https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=upstrea > > >> ming/git-maintainer-v1 > > >> > > >> This series adds new tags to the Maintainers.txt format, making it > > >> possible to describe which filesystem paths are looked after by > > >> which people, and hence automating the extraction of a list over > > >> who should be cc:d on a patch submission. > > >> > > >> Remaining shorcomings in v1: > > >> - Will still be misparsing OvmfPkg and MdeModulePkg due to non-tag lines > > >> interspersed with the tag lines. These lines will be removed as areas of > > >> responsibility is formally rewritten as tags. > > >> - * Wildcard support is not fully filesystem compliant except in first or > > >> last position in the file pattern (it translates as regex .* elsewhere). > > >> However, actual cases of mismatch are expected to be unlikely, and they > > >> will be false positives rather than false negatives - so I think this is > > >> good enough at least for a start. > > >> - Provides no information of why certain people or meiling lists were > > >> picked - it just bundles all recipients up, deduplicates them, and > > >> prints them out. > > >> > > >> Using the script requires the gitpython module to be installed. > > >> > > >> Worthwhile mentioning outside the ChangeLog is the added -l flag, > > >> which lets you look up what a given path would return in the way of matches. > > >> E.g. "python BaseTools/Scripts/GetMaintainer.py -l Non/Existing/Path" > > >> would return: > > >> --- > > >> Non/Existing/Path > > >> "Non/Existing/Path": no maintainers found, looking for default > > >> Andrew Fish <afish@apple.com> > > >> Laszlo Ersek <lersek@redhat.com> > > >> Leif Lindholm <leif.lindholm@linaro.org> > > >> Michael D Kinney <michael.d.kinney@intel.com> > > >> devel@edk2.groups.io > > >> --- > > >> > > >> This series would still result in GetMaintainers.py missing some > > >> maintainers/reviewers due to descriptions in prose rather than > > >> filename patterns. My preferred way of handling this would be to > > >> merge 1-2/3 as soon as found acceptable, following up and merging > > >> patches to update ArmVirtPkg, MdeModulePkg, and OvmfPkg, and > > >> finally once the file is consistent, proceed to merge 3/3. > > >> > > >> Changelog: > > >> v1: > > >> - Rebase to current Maintainers.txt. > > >> - Fix typos and missed bits in Maintainers.txt. > > >> - Get rid of the magic '<default>' filename, let the single-char '*' > > >> wildcard resolve this (_using_ the magic '<default>' filename in the > > >> script, but treating it as if it was a file in the top-level directory). > > >> - Add -l flag to script to look up which maintainers would be returned for > > >> a given path (which need not exist). > > >> rfc: > > >> - Split patches up > > >> - one for new Maintainers.txt format (documentation and F: tags). > > >> - one for adding a new wilcards responsibility area for */Arm, */AArch64 > > >> - one for the GetMaintainer.py script > > >> - Reworked wildcard handling based on Laszlo's explanation > > >> - Trailing / covers everything under that directory > > >> - Trailing * does not cover subdirectories > > >> - Added support for X: tag > > >> - Added support for magic '<default>' pathname > > >> - Also prints mailing list addresses for matching L: tags > > >> > > >> Cc: Andrew Fish <afish@apple.com> > > >> Cc: Laszlo Ersek <lersek@redhat.com> > > >> Cc: Michael D Kinney <michael.d.kinney@intel.com> > > >> Cc: Bob Feng <bob.c.feng@intel.com> > > >> Cc: Liming Gao <liming.gao@intel.com> > > >> Cc: Philippe Mathieu-Daude <philmd@redhat.com> > > >> Cc: "Wu, Hao A" <hao.a.wu@intel.com> > > >> > > >> Leif Lindholm (3): > > >> Maintainers.txt: update for filesystem area descriptions > > >> Maintainers.txt: add wildcard path association for Arm/AArch64 > > >> BaseTools: add GetMaintainer.py script > > >> > > >> BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++ > > >> Maintainers.txt | 54 ++++++++ > > >> 2 files changed, 244 insertions(+) create mode 100644 > > >> BaseTools/Scripts/GetMaintainer.py > > >> > > >> -- > > >> 2.20.1 > > >> > > >> > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#44321): https://edk2.groups.io/g/devel/message/44321 Mute This Topic: https://groups.io/mt/32524902/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.