Re: [edk2-devel] [Patch] BaseTools: Fix the ending character for Linux script BuildEnv

2021-02-10 Thread Bob Feng
Thank you Mike. Thanks, Bob -Original Message- From: Kinney, Michael D Sent: Wednesday, February 10, 2021 9:53 AM To: Feng, Bob C ; Leif Lindholm ; Kinney, Michael D Cc: devel@edk2.groups.io; Liming Gao ; Chen, Christine ; Matthew Carlson Subject: RE: [Patch] BaseTools: Fix the end

Re: [edk2-devel] [Patch] BaseTools: Fix the ending character for Linux script BuildEnv

2021-02-09 Thread Michael D Kinney
Pushed PR: https://github.com/tianocore/edk2/pull/1419 Commit: https://github.com/tianocore/edk2/pull/1419/commits/c10b513830e63fa4fc28f1edc43352de8d97b61f Mike > -Original Message- > From: Kinney, Michael D > Sent: Tuesday, February 9, 2021 5:21 PM > To: Feng, Bob C ; Leif Lindholm

Re: [edk2-devel] [Patch] BaseTools: Fix the ending character for Linux script BuildEnv

2021-02-09 Thread Michael D Kinney
Hi Bob, Reviewed-by: Michael D Kinney It is difficult to see the changes. I did verify that the file before the patch had mixed line endings and the one after the patch is LF only line endings. Minor issues with Author name with a ','. I fixed that when I added the rb to the commit message.

Re: [edk2-devel] [Patch] BaseTools: Fix the ending character for Linux script BuildEnv

2021-02-09 Thread Bob Feng
Hi Mike Could you help review this patch and push it? Liming may take leave now. Thanks, Bob -Original Message- From: Leif Lindholm Sent: Tuesday, February 9, 2021 11:01 PM To: Feng, Bob C Cc: devel@edk2.groups.io; Liming Gao ; Chen, Christine ; Matthew Carlson ; Kinney, Michael D

Re: [edk2-devel] [Patch] BaseTools: Fix the ending character for Linux script BuildEnv

2021-02-09 Thread Leif Lindholm
On Tue, Feb 09, 2021 at 22:58:08 +0800, Bob Feng wrote: > Signed-off-by: Bob Feng > Cc: Liming Gao > Cc: Yuwei Chen > Cc: Matthew Carlson > Cc: Michael D Kinney > Cc: Leif Lindholm SMTP strips the intersting aspect, but as long as the end result is a BuildEnv that has no characters: Review

[edk2-devel] [Patch] BaseTools: Fix the ending character for Linux script BuildEnv

2021-02-09 Thread Bob Feng
Signed-off-by: Bob Feng Cc: Liming Gao Cc: Yuwei Chen Cc: Matthew Carlson Cc: Michael D Kinney Cc: Leif Lindholm --- BaseTools/BuildEnv | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/BaseTools/BuildEnv b/BaseTools/BuildEnv index 8d283e10c0..27