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

[Liming Gao]

Laszlo:


Thanks
Liming
> -----Original Message-----
> From: Laszlo Ersek <lersek at redhat.com>
> Sent: Thursday, April 30, 2020 2:04 AM
> To: Sean Brogan <spbrogan at outlook.com>; devel at edk2.groups.io; michael.kubacki at outlook.com
> Cc: Andrew Fish <afish at apple.com>; Ard Biesheuvel <ard.biesheuvel at arm.com>; Bret Barkelew <Bret.Barkelew at microsoft.com>;
> Justen, Jordan L <jordan.l.justen at intel.com>; Leif Lindholm <leif at nuviainc.com>; Gao, Liming <liming.gao at intel.com>; Kinney,
> Michael D <michael.d.kinney at intel.com>; Ni, Ray <ray.ni at intel.com>; Sean Brogan <sean.brogan at microsoft.com>
> Subject: Re: [edk2-devel] [PATCH v3 0/7] Edk2 Platform and Core CI for ArmVirtPkg, EmulatorPkg, and OvmfPkg
> 
> On 04/28/20 18:35, Sean Brogan wrote:
> > 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.
> 
> It depends on the maintainer, too.
> 
> Personally I give R-b if I carefully review the patch and am pleased
> with it.
> 
> I give A-b if I review the patch for general sanity, but don't dig into
> the details. I can also give A-b if someone I trust to do a good review
> in the subject technical area provides an R-b, regardless of whether
> they are an "R" or an otherwise un-designated contributor. With "R"
> folks the chance is higher for me to see such an R-b posted in the first
> place, of course.
> 
> I do think an "M" person should provide "at least" an A-b, even if they
> delegate the actual detailed review to someone else.
> 
I don't think there is such requirement to maintainer now. If you think this is required, 
You can give the proposal to add this requirement in Maintainers.txt.

Thanks
Liming
> So yes, I think your understanding "is correct" (meaning, selfishly,
> that it mostly matches mine, anyway :))
> 
> Thanks
> Laszlo


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

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