Re: [edk2-devel] [PATCH v3 0/7] Edk2 Platform and Core CI for ArmVirtPkg, EmulatorPkg, and OvmfPkg

["Sean" <spbrogan@outlook.com>]

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.kubacki@outlook.com wrote:
>> From: Michael Kubacki <michael.kubacki@microsoft.com>
>>
>> Attention: Reviewed-by is still needed from some package maintainers.
>> * 0002-ArmVirtPkg-Add-Platform-CI-and-configuration-for-Cor.patch
>>    * Laszlo Ersek <lersek@redhat.com>
>>    * Ard Biesheuvel <ard.biesheuvel@arm.com>
>>    * Leif Lindholm <leif@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 <lersek@redhat.com>
> Acked-by: Ard Biesheuvel <ard.biesheuvel@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.justen@intel.com>
>>    * Laszlo Ersek <lersek@redhat.com>
>>    * Ard Biesheuvel <ard.biesheuvel@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
> 
> 
> 
>