[edk2-devel] [Patch 1/1] BaseTools/PlatformAutoGen: MAKE_FLAGS and MAKE_PATH fixes
Bob Feng
bob.c.feng at intel.com
Tue Apr 13 07:36:38 UTC 2021
This patch looks good to me.
Reviewed-by: Bob Feng <bob.c.feng at intel.com>
Thanks,
Bob
-----Original Message-----
From: Kinney, Michael D <michael.d.kinney at intel.com>
Sent: Tuesday, April 13, 2021 10:40 AM
To: gaoliming <gaoliming at byosoft.com.cn>; devel at edk2.groups.io; Kinney, Michael D <michael.d.kinney at intel.com>
Cc: Feng, Bob C <bob.c.feng at intel.com>; Chen, Christine <yuwei.chen at intel.com>
Subject: RE: [edk2-devel] [Patch 1/1] BaseTools/PlatformAutoGen: MAKE_FLAGS and MAKE_PATH fixes
Liming,
The BZ contains an example of MAKE_FLAGS in DSC that breaks the build. With this change, that example passes and shows the additional make tool output.
Mike
> -----Original Message-----
> From: gaoliming <gaoliming at byosoft.com.cn>
> Sent: Monday, April 12, 2021 6:31 PM
> To: devel at edk2.groups.io; Kinney, Michael D
> <michael.d.kinney at intel.com>
> Cc: Feng, Bob C <bob.c.feng at intel.com>; Chen, Christine
> <yuwei.chen at intel.com>
> Subject: 回复: [edk2-devel] [Patch 1/1] BaseTools/PlatformAutoGen:
> MAKE_FLAGS and MAKE_PATH fixes
>
> Mike:
> I don't see the issue in the code change. I want to know what unit
> test has been done for this change.
>
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: devel at edk2.groups.io <devel at edk2.groups.io> 代表 Michael D Kinney
> > 发送时间: 2021年4月10日 5:49
> > 收件人: devel at edk2.groups.io
> > 抄送: Bob Feng <bob.c.feng at intel.com>; Liming Gao
> > <gaoliming at byosoft.com.cn>; Yuwei Chen <yuwei.chen at intel.com>
> > 主题: [edk2-devel] [Patch 1/1] BaseTools/PlatformAutoGen: MAKE_FLAGS
> > and MAKE_PATH fixes
> >
> > 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 at intel.com>
> > Cc: Liming Gao <gaoliming at byosoft.com.cn>
> > Cc: Yuwei Chen <yuwei.chen at intel.com>
> > Signed-off-by: Michael D Kinney <michael.d.kinney at intel.com>
> > ---
> > .../Source/Python/AutoGen/PlatformAutoGen.py | 36
> > +++++++++++--------
> > 1 file changed, 21 insertions(+), 15 deletions(-)
> >
> > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py
> > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py
> > index 7d8e7b3c7cc1..c16f2e4cd8b7 100644
> > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py
> > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py
> > @@ -1,7 +1,7 @@
> > ## @file
> > # 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> #
> > SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -805,20 +805,26
> > @@ class PlatformAutoGen(AutoGen):
> > #
> > @cached_property
> > def BuildCommand(self):
> > - RetVal = []
> > - if "MAKE" in self.ToolDefinition and "PATH" in
> > self.ToolDefinition["MAKE"]:
> > - RetVal += _SplitOption(self.ToolDefinition["MAKE"]["PATH"])
> > - if "FLAGS" in self.ToolDefinition["MAKE"]:
> > - NewOption =
> > self.ToolDefinition["MAKE"]["FLAGS"].strip()
> > - if NewOption != '':
> > - RetVal += _SplitOption(NewOption)
> > - if "MAKE" in self.EdkIIBuildOption:
> > - if "FLAGS" in self.EdkIIBuildOption["MAKE"]:
> > - Flags = self.EdkIIBuildOption["MAKE"]["FLAGS"]
> > - if Flags.startswith('='):
> > - RetVal = [RetVal[0]] + [Flags[1:]]
> > - else:
> > - RetVal.append(Flags)
> > + if "MAKE" in self.EdkIIBuildOption and "PATH" in
> > self.EdkIIBuildOption["MAKE"]:
> > + # MAKE_PATH in DSC [BuildOptions] section is higher priority
> > + Path = self.EdkIIBuildOption["MAKE"]["PATH"]
> > + 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()
> > + if NewOption != '':
> > + RetVal += _SplitOption(NewOption)
> > + if "MAKE" in self.EdkIIBuildOption and "FLAGS" in
> > self.EdkIIBuildOption["MAKE"]:
> > + Flags = self.EdkIIBuildOption["MAKE"]["FLAGS"]
> > + if Flags.startswith('='):
> > + RetVal = [RetVal[0]] + _SplitOption(Flags[1:].strip())
> > + else:
> > + RetVal = RetVal + _SplitOption(Flags.strip())
> > return RetVal
> >
> > ## Get tool chain definition
> > --
> > 2.31.1.windows.1
> >
> >
> >
> >
> >
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#74017): https://edk2.groups.io/g/devel/message/74017
Mute This Topic: https://groups.io/mt/82056259/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list