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

Laszlo Ersek posted 4 patches 4 years, 9 months ago
Failed in applying to current master (apply log)
Maintainers.txt | 52 +++++++++++++++++---
1 file changed, 44 insertions(+), 8 deletions(-)
[edk2-devel] [PATCH 0/4] Maintainers.txt: formalize ArmVirtPkg and OvmfPkg subsystem pathnames
Posted by Laszlo Ersek 4 years, 9 months ago
Repo:   https://github.com/lersek/edk2.git
Branch: ovmfpkg_maintainers

The patches in this series should be inserted between Leif's patches

* [edk2-devel] [PATCH 2/3] Maintainers.txt: add wildcard path association for Arm/AArch64
  http://mid.mail-archive.com/20190712170128.6495-3-leif.lindholm@linaro.org
  https://edk2.groups.io/g/devel/message/43665

and

* [edk2-devel] [PATCH 3/3] BaseTools: add GetMaintainer.py script
  http://mid.mail-archive.com/20190712170128.6495-4-leif.lindholm@linaro.org
  https://edk2.groups.io/g/devel/message/43666

That's also how you will find them in my topic branch, referenced above
(in particular, look for the "Message-Id" tags on Leif's patches,
appended by my invocation of "git-am -m").

My understanding is that Leif would only like to pick up these patches
once they are ready for pushing. That is, if corrections are necessary,
I should post v2 (and I should post v2 of these four patches only) --
Leif's series and mine will only be unified for the pushing.

(Technically, Leif could incorporate these patches at once in his topic
branch, and post a unified v2; assuming corrections were necessary for
the OvmfPkg-related patches. That's because I expect minimal changes if
any, which Leif could implement on top "mindlessly". Of course, that's
also the argument against Leif writing the updates -- we shouldn't shove
mindless work over to other people. :) Anyway, I'm fine either way.)

Cc: Andrew Fish <afish@apple.com>
Cc: Anthony Perard <anthony.perard@citrix.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Julien Grall <julien.grall@arm.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Cc: Stefan Berger <stefanb@linux.ibm.com>

Thanks
Laszlo

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"

 Maintainers.txt | 52 +++++++++++++++++---
 1 file changed, 44 insertions(+), 8 deletions(-)

-- 
2.19.1.3.g30247aa5d201


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

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

Re: [edk2-devel] [PATCH 0/4] Maintainers.txt: formalize ArmVirtPkg and OvmfPkg subsystem pathnames
Posted by Philippe Mathieu-Daudé 4 years, 9 months ago
On 7/15/19 7:04 PM, Laszlo Ersek wrote:
> Repo:   https://github.com/lersek/edk2.git
> Branch: ovmfpkg_maintainers
> 
> The patches in this series should be inserted between Leif's patches
> 
> * [edk2-devel] [PATCH 2/3] Maintainers.txt: add wildcard path association for Arm/AArch64
>   http://mid.mail-archive.com/20190712170128.6495-3-leif.lindholm@linaro.org
>   https://edk2.groups.io/g/devel/message/43665
> 
> and
> 
> * [edk2-devel] [PATCH 3/3] BaseTools: add GetMaintainer.py script
>   http://mid.mail-archive.com/20190712170128.6495-4-leif.lindholm@linaro.org
>   https://edk2.groups.io/g/devel/message/43666
> 
> That's also how you will find them in my topic branch, referenced above
> (in particular, look for the "Message-Id" tags on Leif's patches,
> appended by my invocation of "git-am -m").
> 
> My understanding is that Leif would only like to pick up these patches
> once they are ready for pushing. That is, if corrections are necessary,
> I should post v2 (and I should post v2 of these four patches only) --
> Leif's series and mine will only be unified for the pushing.
> 
> (Technically, Leif could incorporate these patches at once in his topic
> branch, and post a unified v2; assuming corrections were necessary for
> the OvmfPkg-related patches. That's because I expect minimal changes if
> any, which Leif could implement on top "mindlessly". Of course, that's
> also the argument against Leif writing the updates -- we shouldn't shove
> mindless work over to other people. :) Anyway, I'm fine either way.)
> 
> Cc: Andrew Fish <afish@apple.com>
> Cc: Anthony Perard <anthony.perard@citrix.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Julien Grall <julien.grall@arm.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> Cc: Stefan Berger <stefanb@linux.ibm.com>
> 
> Thanks
> Laszlo
> 
> 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"
> 
>  Maintainers.txt | 52 +++++++++++++++++---
>  1 file changed, 44 insertions(+), 8 deletions(-)

FWIW, for this series:
Tested-by: Philippe Mathieu-Daude <philmd@redhat.com>
(with various paths from each new sections, and also from the previous
section, then comparating).

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

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

Re: [edk2-devel] [PATCH 0/4] Maintainers.txt: formalize ArmVirtPkg and OvmfPkg subsystem pathnames
Posted by Laszlo Ersek 4 years, 9 months ago
On 07/18/19 06:54, Philippe Mathieu-Daudé wrote:
> On 7/15/19 7:04 PM, Laszlo Ersek wrote:
>> Repo:   https://github.com/lersek/edk2.git
>> Branch: ovmfpkg_maintainers
>>
>> The patches in this series should be inserted between Leif's patches
>>
>> * [edk2-devel] [PATCH 2/3] Maintainers.txt: add wildcard path association for Arm/AArch64
>>   http://mid.mail-archive.com/20190712170128.6495-3-leif.lindholm@linaro.org
>>   https://edk2.groups.io/g/devel/message/43665
>>
>> and
>>
>> * [edk2-devel] [PATCH 3/3] BaseTools: add GetMaintainer.py script
>>   http://mid.mail-archive.com/20190712170128.6495-4-leif.lindholm@linaro.org
>>   https://edk2.groups.io/g/devel/message/43666
>>
>> That's also how you will find them in my topic branch, referenced above
>> (in particular, look for the "Message-Id" tags on Leif's patches,
>> appended by my invocation of "git-am -m").
>>
>> My understanding is that Leif would only like to pick up these patches
>> once they are ready for pushing. That is, if corrections are necessary,
>> I should post v2 (and I should post v2 of these four patches only) --
>> Leif's series and mine will only be unified for the pushing.
>>
>> (Technically, Leif could incorporate these patches at once in his topic
>> branch, and post a unified v2; assuming corrections were necessary for
>> the OvmfPkg-related patches. That's because I expect minimal changes if
>> any, which Leif could implement on top "mindlessly". Of course, that's
>> also the argument against Leif writing the updates -- we shouldn't shove
>> mindless work over to other people. :) Anyway, I'm fine either way.)
>>
>> Cc: Andrew Fish <afish@apple.com>
>> Cc: Anthony Perard <anthony.perard@citrix.com>
>> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> Cc: David Woodhouse <dwmw2@infradead.org>
>> Cc: Jordan Justen <jordan.l.justen@intel.com>
>> Cc: Julien Grall <julien.grall@arm.com>
>> Cc: Leif Lindholm <leif.lindholm@linaro.org>
>> Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
>> Cc: Stefan Berger <stefanb@linux.ibm.com>
>>
>> Thanks
>> Laszlo
>>
>> 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"
>>
>>  Maintainers.txt | 52 +++++++++++++++++---
>>  1 file changed, 44 insertions(+), 8 deletions(-)
> 
> FWIW, for this series:
> Tested-by: Philippe Mathieu-Daude <philmd@redhat.com>
> (with various paths from each new sections, and also from the previous
> section, then comparating).
> 

Thank you!
Laszlo

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

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