Acked-by: Eric Dong <eric.dong@intel.com>

 

Hi Laszlo,

 

I saw you already provided Review-by tag and I can’t verify the change, so I don’t add comments for this patch.

 

I will provide Acked-by tag for the similar case next time.

 

I have pushed this change.

 

Thanks,

Eric

From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Laszlo Ersek
Sent: Wednesday, May 6, 2020 7:22 AM
To: devel@edk2.groups.io; Dong, Eric <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
Cc: Leo Duran <leo.duran@amd.com>
Subject: Re: [edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Remove Executable attribute from MpLib.h

 

Eric, Ray,

can one of you please approve, and also merge, this patch?

It's been on the list for two weeks now (and you were CC'd). I didn't
want to go ahead and merge it myself without at least an ACK from one of
you.

Thanks
Laszlo

On 04/22/20 18:55, Laszlo Ersek wrote:
> On 04/22/20 00:05, Leo Duran wrote:
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2678
>>
>> This patch fixes a file permission issue introduced by accident.
>>
>> Cc: Eric Dong <eric.dong@intel.com>
>> Cc: Ray Ni <ray.ni@intel.com>
>> Cc: Laszlo Ersek <lersek@redhat.com>
>> Signed-off-by: Leo Duran <leo.duran@amd.com>
>> ---
>>  UefiCpuPkg/Library/MpInitLib/MpLib.h | 0
>>  1 file changed, 0 insertions(+), 0 deletions(-)
>>  mode change 100755 => 100644 UefiCpuPkg/Library/MpInitLib/MpLib.h
>>
>> diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.h b/UefiCpuPkg/Library/MpInitLib/MpLib.h
>> old mode 100755
>> new mode 100644
>>

> Reviewed-by: Laszlo Ersek <lersek@redhat.com>

> Thanks, Leo!
> Laszlo