Date
1 - 3 of 3
[PATCH v4 0/2] BaseTools: Rationalise makefile generation
PierreGondois
v4:
Remove caret in NASM_INC macro: I could not reproduce the build fail Bob Feng had when issuing the following command: build -p UefiCpuPkg\UefiCpuPkg.dsc -a IA32 -t VS2015x86 Nonetheless, this v4 should only have an impact on the builds made on Windows platforms using nmake. The changes can be seen at https://github.com/PierreARM/edk2/tree/720_BaseTools_Rationalise_makefile_generation_v4 Pierre Gondois (2): BaseTools: Rationalise makefile generation BaseTools: Remove caret in NASM_INC macro BaseTools/Source/Python/AutoGen/GenMake.py | 145 +++++++++++--------- BaseTools/Source/Python/AutoGen/IncludesAutoGen.py | 34 +++-- BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 13 +- BaseTools/Source/Python/build/build.py | 25 ++-- 4 files changed, 120 insertions(+), 97 deletions(-) -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' |
|
Bob Feng
This patch set is good to me.
toggle quoted message
Show quoted text
Reviewed-by: Bob Feng <bob.c.feng@...> -----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of PierreGondois Sent: Monday, February 10, 2020 6:49 PM To: devel@edk2.groups.io Cc: Pierre Gondois <Pierre.Gondois@...>; Feng, Bob C <bob.c.feng@...>; Gao, Liming <liming.gao@...>; Sami.Mujawar@...; pierre.gondois@...; nd@... Subject: [edk2-devel] [PATCH v4 0/2] BaseTools: Rationalise makefile generation v4: Remove caret in NASM_INC macro: I could not reproduce the build fail Bob Feng had when issuing the following command: build -p UefiCpuPkg\UefiCpuPkg.dsc -a IA32 -t VS2015x86 Nonetheless, this v4 should only have an impact on the builds made on Windows platforms using nmake. The changes can be seen at https://github.com/PierreARM/edk2/tree/720_BaseTools_Rationalise_makefile_generation_v4 Pierre Gondois (2): BaseTools: Rationalise makefile generation BaseTools: Remove caret in NASM_INC macro BaseTools/Source/Python/AutoGen/GenMake.py | 145 +++++++++++--------- BaseTools/Source/Python/AutoGen/IncludesAutoGen.py | 34 +++-- BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 13 +- BaseTools/Source/Python/build/build.py | 25 ++-- 4 files changed, 120 insertions(+), 97 deletions(-) -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' |
|
Bob Feng
Pushed.
toggle quoted message
Show quoted text
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Bob Feng Sent: Wednesday, February 12, 2020 10:07 AM To: devel@edk2.groups.io; pierre.gondois@... Cc: Gao, Liming <liming.gao@...>; Sami.Mujawar@...; nd@... Subject: Re: [edk2-devel] [PATCH v4 0/2] BaseTools: Rationalise makefile generation This patch set is good to me. Reviewed-by: Bob Feng <bob.c.feng@...> -----Original Message----- From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of PierreGondois Sent: Monday, February 10, 2020 6:49 PM To: devel@edk2.groups.io Cc: Pierre Gondois <Pierre.Gondois@...>; Feng, Bob C <bob.c.feng@...>; Gao, Liming <liming.gao@...>; Sami.Mujawar@...; pierre.gondois@...; nd@... Subject: [edk2-devel] [PATCH v4 0/2] BaseTools: Rationalise makefile generation v4: Remove caret in NASM_INC macro: I could not reproduce the build fail Bob Feng had when issuing the following command: build -p UefiCpuPkg\UefiCpuPkg.dsc -a IA32 -t VS2015x86 Nonetheless, this v4 should only have an impact on the builds made on Windows platforms using nmake. The changes can be seen at https://github.com/PierreARM/edk2/tree/720_BaseTools_Rationalise_makefile_generation_v4 Pierre Gondois (2): BaseTools: Rationalise makefile generation BaseTools: Remove caret in NASM_INC macro BaseTools/Source/Python/AutoGen/GenMake.py | 145 +++++++++++--------- BaseTools/Source/Python/AutoGen/IncludesAutoGen.py | 34 +++-- BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 13 +- BaseTools/Source/Python/build/build.py | 25 ++-- 4 files changed, 120 insertions(+), 97 deletions(-) -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' |
|