[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.

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.

-----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)'