I think this was my fault.
I was under the impression that a patch needed one of developers listed
in the (m) or (r) section of maintainers.txt to provide a reviewed-by.
My new understanding is an ack from the (m) plus anyone providing a
reviewed-by is enough.
Thanks
Sean
On 4/28/2020 5:59 AM, Laszlo Ersek wrote:
On 04/24/20 23:31, michael.kuba...@outlook.com wrote:
From: Michael Kubacki <michael.kuba...@microsoft.com>
Attention: Reviewed-by is still needed from some package maintainers.
* 0002-ArmVirtPkg-Add-Platform-CI-and-configuration-for-Cor.patch
* Laszlo Ersek <ler...@redhat.com>
* Ard Biesheuvel <ard.biesheu...@arm.com>
* Leif Lindholm <l...@nuviainc.com>
I don't understand. Your posting
[edk2-devel] [PATCH v3 2/7] ArmVirtPkg: Add Platform CI and
configuration for Core CI
already -- and, to my understanding, correctly -- includes
Acked-by: Laszlo Ersek <ler...@redhat.com>
Acked-by: Ard Biesheuvel <ard.biesheu...@arm.com>
That means both Ard and myself are OK with the patch. What else is
needed from us?
* 0004-OvmfPkg-Add-Platform-CI-and-configuration-for-Core-C.patch
* Jordan Justen <jordan.l.jus...@intel.com>
* Laszlo Ersek <ler...@redhat.com>
* Ard Biesheuvel <ard.biesheu...@arm.com>
Same here:
[edk2-devel] [PATCH v3 4/7] OvmfPkg: Add Platform CI and configuration
for Core CI
I think these patches are ready to be merged.
Who's going to merge the series?
Thanks!
Laszlo
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#58259): https://edk2.groups.io/g/devel/message/58259
Mute This Topic: https://groups.io/mt/73251653/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
