[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