[edk2-devel] [PATCH v2 1/1] Maintainers.txt: Update reviewers of */RiscV64

[Laszlo Ersek]

Hi Abner,

On 09/04/20 10:51, Chang, Abner (HPS SW/FW Technologist) wrote:
> 
> 
>> -----Original Message-----
>> From: Laszlo Ersek [mailto:lersek at redhat.com]
>> Sent: Friday, September 04, 2020 4:25 PM
>> To: Chang, Abner (HPS SW/FW Technologist) <abner.chang at hpe.com>;
>> devel at edk2.groups.io
>> Cc: Andrew Fish <afish at apple.com>; Leif Lindholm <leif at nuviainc.com>;
>> Michael D Kinney <michael.d.kinney at intel.com>; Leif Lindholm
>> <leif.lindholm at linaro.org>
>> Subject: Re: [PATCH v2 1/1] Maintainers.txt: Update reviewers of */RiscV64
>>
>> On 09/04/20 09:19, Abner Chang wrote:
>>> Add reviewers for all /RiscV64 folders.
>>>
>>> Signed-off-by: Abner Chang <abner.chang at hpe.com>
>>> Cc: Andrew Fish <afish at apple.com>
>>> Cc: Laszlo Ersek <lersek at redhat.com>
>>> Cc: Leif Lindholm <leif at nuviainc.com>
>>> Cc: Michael D Kinney <michael.d.kinney at intel.com>
>>> Cc: Leif Lindholm <leif.lindholm at linaro.org>
>>> Acked-by: Laszlo Ersek <lersek at redhat.com>
>>> ---
>>>  Maintainers.txt | 5 +++++
>>>  1 file changed, 5 insertions(+)
>>>
>>> diff --git a/Maintainers.txt b/Maintainers.txt index
>>> 57cd2fc662..6a22a14796 100644
>>> --- a/Maintainers.txt
>>> +++ b/Maintainers.txt
>>> @@ -98,6 +98,11 @@ F: */Arm/
>>>  M: Leif Lindholm <leif at nuviainc.com>
>>>  M: Ard Biesheuvel <ard.biesheuvel at arm.com>
>>>
>>> +RISCV64
>>> +F: */RiscV64/
>>> +M: Abner Chang <abner.chang at hpe.com>
>>> +R: Daniel Schaefer <daniel.schaefer at hpe.com>
>>> +
>>>  EDK II Continuous Integration:
>>>  ------------------------------
>>>  .azurepipelines/
>>>
>>
>> What are the changes relative to v1?
> R: for Abner changed to M: for Abner
>>
>> (Also, I believe Leif is away at the moment, and I think we should have his
>> ACK on this patch, before we merge the patch.)
> Yes, I had few conversations with Leif and Mike for this in this.

Wait, that reminds me... OK, I've looked up the v1 discussion now.

Is there any particular reason you didn't pick up Leif's Reviewed-by
from the v1 thread? Leif gave his R-b conditional on the R->M change for
your name:

https://edk2.groups.io/g/devel/message/64807

and you have implemented that change. So I think Leif's R-b should have
been picked up.

Based on <https://edk2.groups.io/g/devel/message/64810> -- do you have
push access to edk2 at this time? Because if that's the case, then we
should merge this patch now.

Hmm let me see.

https://github.com/orgs/tianocore/teams/edk-ii-maintainers/members

Yes, you are a member of this group.

https://github.com/orgs/tianocore/teams?query=@changab

So I'm going to apply Leif's R-b from the v1 thread now, and then merge
this patch.

Thanks,
Laszlo

> Sure we can wait for his Ack.
> Thanks
>>
>> Thanks
>> Laszlo
> 


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

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