[edk2-devel] [PATCH 3/3] BaseTools: add GetMaintainer.py script

[Laszlo Ersek]

On 07/13/19 12:41, Leif Lindholm wrote:
> On Sat, Jul 13, 2019 at 12:58:18AM +0200, Laszlo Ersek wrote:
>> On 07/12/19 19:01, Leif Lindholm wrote:
>>> Add a new script GetMaintainer.py that uses the new Maintainer.txt format
>>> to determine which addresses to cc on patch submission.
>>>
>>> Signed-off-by: Leif Lindholm <leif.lindholm at linaro.org>
>>> Reviewed-by: Philippe Mathieu-Daude <philmd at redhat.com>
>>> Tested-by: Philippe Mathieu-Daude <philmd at redhat.com>
>>> ---
>>>  BaseTools/Scripts/GetMaintainer.py | 190 +++++++++++++++++++++++++++++
>>>  1 file changed, 190 insertions(+)
>>>  create mode 100644 BaseTools/Scripts/GetMaintainer.py
>>
>> I should really re-review this from zero, but it's 0:55 AM again. :/
>>
>> Anyway, I think this script is a good candidate for gradual improvements
>> (should such ever become necessary, that is). The script doesn't have to
>> be perfect to be incredibly useful.
>>
>> Acked-by: Laszlo Ersek <lersek at redhat.com>
> 
> Thanks!
> 
>> ... Did you have me in mind for the ArmVirtPkg and OvmfPkg pathname
>> patterns, for replacing the natural language pointers? :)
> 
> That would be ideal, thanks. I might miss something if I did it.
> I would be happy to push them together with 1-2/3 once ready.

Posted:

  [edk2-devel] [PATCH 0/4]
  Maintainers.txt: formalize ArmVirtPkg and OvmfPkg subsystem pathnames

  https://edk2.groups.io/g/devel/message/43728
  http://mid.mail-archive.com/20190715170431.13955-1-lersek@redhat.com

(The mail-archive.com link will start working after mail-archive.com
indexes the message. Oh, the beauty of stable message identifiers! :) )

Thanks
Laszlo

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

View/Reply Online (#43735): https://edk2.groups.io/g/devel/message/43735
Mute This Topic: https://groups.io/mt/32444053/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-