tyler.durden/audk
1
0
Fork 0
mirror of https://github.com/acidanthera/audk.git synced 2025-07-28 16:14:04 +02:00
Code Issues Packages Projects Releases Wiki Activity

BaseTools/PlatformAutoGen: MAKE_FLAGS and MAKE_PATH fixes

Browse Source 
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3312

Update parsing of MAKE_FLAGS in DSC [BuildOptions] sections
to split the flags into a list to be compatible with
running the make command using Popen().  Parsing MAKE_FLAGS
from tools_def.txt already uses _SplitOption().  This change
uses the same _SplitOption() method for MAKE_FLAGS from a
DSC [BuildOptions] section.

Also update the parsing of MAKE_PATH to support MAKE_PATH
from tools_def.txt or the DSC [BuildOptions] section.  MAKE_PATH
in DSC [BuildOptions] section is higher priority than MAKE_PATH
in tools_def.txt.

Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Yuwei Chen <yuwei.chen@intel.com>
Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Yuwei Chen <yuwei.chen@intel.com>
Reviewed-by: Bob Feng <bob.c.feng@intel.com>
This commit is contained in:
Michael D Kinney 2021-04-08 21:35:38 -07:00 committed by mergify[bot]
parent 037090cb7c
commit c055be5b82
1 changed files with 21 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
BaseTools/Source/Python/AutoGen/PlatformAutoGen.py
View File

@ -1,7 +1,7 @@
## @file ## @file
# Create makefile for MS nmake and GNU make # Create makefile for MS nmake and GNU make
# #
# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> # Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
# Copyright (c) 2020, ARM Limited. All rights reserved.<BR> # Copyright (c) 2020, ARM Limited. All rights reserved.<BR>
# SPDX-License-Identifier: BSD-2-Clause-Patent # SPDX-License-Identifier: BSD-2-Clause-Patent
# #
@ -805,20 +805,26 @@ class PlatformAutoGen(AutoGen):
# #
@cached_property @cached_property
def BuildCommand(self): def BuildCommand(self):
RetVal = [] if "MAKE" in self.EdkIIBuildOption and "PATH" in self.EdkIIBuildOption["MAKE"]:
if "MAKE" in self.ToolDefinition and "PATH" in self.ToolDefinition["MAKE"]: # MAKE_PATH in DSC [BuildOptions] section is higher priority
RetVal += _SplitOption(self.ToolDefinition["MAKE"]["PATH"]) Path = self.EdkIIBuildOption["MAKE"]["PATH"]
if "FLAGS" in self.ToolDefinition["MAKE"]: if Path.startswith('='):
Path = Path[1:].strip()
RetVal = _SplitOption(Path)
elif "MAKE" in self.ToolDefinition and "PATH" in self.ToolDefinition["MAKE"]:
RetVal = _SplitOption(self.ToolDefinition["MAKE"]["PATH"])
else:
return []
if "MAKE" in self.ToolDefinition and "FLAGS" in self.ToolDefinition["MAKE"]:
NewOption = self.ToolDefinition["MAKE"]["FLAGS"].strip() NewOption = self.ToolDefinition["MAKE"]["FLAGS"].strip()
if NewOption != '': if NewOption != '':
RetVal += _SplitOption(NewOption) RetVal += _SplitOption(NewOption)
if "MAKE" in self.EdkIIBuildOption: if "MAKE" in self.EdkIIBuildOption and "FLAGS" in self.EdkIIBuildOption["MAKE"]:
if "FLAGS" in self.EdkIIBuildOption["MAKE"]:
Flags = self.EdkIIBuildOption["MAKE"]["FLAGS"] Flags = self.EdkIIBuildOption["MAKE"]["FLAGS"]
if Flags.startswith('='): if Flags.startswith('='):
RetVal = [RetVal[0]] + [Flags[1:]] RetVal = [RetVal[0]] + _SplitOption(Flags[1:].strip())
else: else:
RetVal.append(Flags) RetVal = RetVal + _SplitOption(Flags.strip())
return RetVal return RetVal
## Get tool chain definition ## Get tool chain definition