* [PATCH] Fix build error of OpenBoard caused by change of AdvancedFeaturePkg @ 2020-03-30 4:29 Heng Luo 2020-03-30 4:38 ` Ni, Ray 0 siblings, 1 reply; 3+ messages in thread From: Heng Luo @ 2020-03-30 4:29 UTC (permalink / raw) To: devel; +Cc: Dandan Bi, Liming Gao, Eric Dong, Ray Ni Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2644 1. Add all pathes of feature domains to package path. 2. Add LogoFeaturePkg to TemporaryBuildWorkaround. 3. Correct wrong codes and remove unnecessary codes in LogoFeaturePkg Cc: Dandan Bi <dandan.bi@intel.com> Cc: Liming Gao <liming.gao@intel.com> Cc: Eric Dong <eric.dong@intel.com> Cc: Ray Ni <ray.ni@intel.com> Signed-off-by: Heng Luo <heng.luo@intel.com> --- Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.dsc | 4 +++- Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.inf | 5 ++++- Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc | 9 --------- Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf | 2 +- Platform/Intel/build_bios.py | 7 ++++++- 5 files changed, 14 insertions(+), 13 deletions(-) diff --git a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.dsc b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.dsc index 227ae00908..c62f9ecc6e 100644 --- a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.dsc +++ b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.dsc @@ -13,7 +13,7 @@ # When the BaseTools update is complete, this file can entirely be removed # from this package. # -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2019 - 2020, Intel Corporation. All rights reserved.<BR> # # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -49,6 +49,8 @@ gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosFeatureEnable |FALSE gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugFeatureEnable |FALSE gUserAuthFeaturePkgTokenSpaceGuid.PcdUserAuthenticationFeatureEnable |FALSE + gLogoFeaturePkgTokenSpaceGuid.PcdLogoFeatureEnable |FALSE + gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable |FALSE !endif # diff --git a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.inf b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.inf index 74176d1989..00818fbe0a 100644 --- a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.inf +++ b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaround.inf @@ -13,7 +13,7 @@ # When the BaseTools update is complete, this file can entirely be removed # from this package. # -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2019 - 2020, Intel Corporation. All rights reserved.<BR> # # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -40,6 +40,7 @@ PowerManagement/S3FeaturePkg/S3FeaturePkg.dec SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec UserInterface/UserAuthFeaturePkg/UserAuthFeaturePkg.dec + UserInterface/LogoFeaturePkg/LogoFeaturePkg.dec [FeaturePcd] gAcpiDebugFeaturePkgTokenSpaceGuid.PcdAcpiDebugFeatureEnable @@ -49,6 +50,8 @@ gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosFeatureEnable gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugFeatureEnable gUserAuthFeaturePkgTokenSpaceGuid.PcdUserAuthenticationFeatureEnable + gLogoFeaturePkgTokenSpaceGuid.PcdLogoFeatureEnable + gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable [Sources] TemporaryBuildWorkaround.c diff --git a/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc b/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc index fca0bfd540..d2dcdeb36a 100644 --- a/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc +++ b/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc @@ -25,15 +25,6 @@ !error "DXE_ARCH must be specified to build this feature!" !endif -################################################################################ -# -# Packages Section - Make sure PCD can be directly used in a conditional statement -# in a DSC which includes this DSC file. -# -################################################################################ -[Packages] - LogoFeaturePkg/LogoFeaturePkg.dec - ################################################################################ # # Library Class section - list of all Library Classes needed by this feature. diff --git a/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf b/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf index 080c87223c..fead9f3b02 100644 --- a/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf +++ b/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf @@ -6,7 +6,7 @@ # SPDX-License-Identifier: BSD-2-Clause-Patent # ## -!if gSmbiosFeaturePkgTokenSpaceGuid.PcdJpgEnable == TRUE +!if gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable == TRUE INF LogoFeaturePkg/LogoDxe/JpegLogoDxe.inf !else INF LogoFeaturePkg/LogoDxe/LogoDxe.inf diff --git a/Platform/Intel/build_bios.py b/Platform/Intel/build_bios.py index 1ef35aca0a..b9ad980510 100644 --- a/Platform/Intel/build_bios.py +++ b/Platform/Intel/build_bios.py @@ -3,7 +3,7 @@ # Builds BIOS using configuration files and dynamically # imported functions from board directory # -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2019 - 2020, Intel Corporation. All rights reserved.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -120,6 +120,11 @@ def pre_build(build_config, build_type="DEBUG", silent=False, toolchain=None): config["PACKAGES_PATH"] += os.pathsep + config["WORKSPACE_SILICON"] config["PACKAGES_PATH"] += os.pathsep + config["WORKSPACE_SILICON_BIN"] config["PACKAGES_PATH"] += os.pathsep + config["WORKSPACE_FEATURES"] + # add all feature domains in WORKSPACE_FEATURES to package path + for filename in os.listdir(config["WORKSPACE_FEATURES"]): + filepath = os.path.join(config["WORKSPACE_FEATURES"], filename) + if os.path.isdir(filepath): + config["PACKAGES_PATH"] += os.pathsep + filepath config["PACKAGES_PATH"] += os.pathsep + config["WORKSPACE_DRIVERS"] config["PACKAGES_PATH"] += os.pathsep + \ os.path.join(config["WORKSPACE"], "FSP") -- 2.24.0.windows.2 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] Fix build error of OpenBoard caused by change of AdvancedFeaturePkg 2020-03-30 4:29 [PATCH] Fix build error of OpenBoard caused by change of AdvancedFeaturePkg Heng Luo @ 2020-03-30 4:38 ` Ni, Ray 2020-03-30 5:55 ` Heng Luo 0 siblings, 1 reply; 3+ messages in thread From: Ni, Ray @ 2020-03-30 4:38 UTC (permalink / raw) To: Luo, Heng, devel@edk2.groups.io; +Cc: Bi, Dandan, Gao, Liming, Dong, Eric Heng, Can you please separate the patch to 3 so each patch fixes one issue specifically? I am fine with one Bugzilla to cover the 3 issues. Thanks, Ray > -----Original Message----- > From: Luo, Heng <heng.luo@intel.com> > Sent: Monday, March 30, 2020 12:30 PM > To: devel@edk2.groups.io > Cc: Bi, Dandan <dandan.bi@intel.com>; Gao, Liming <liming.gao@intel.com>; > Dong, Eric <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com> > Subject: [PATCH] Fix build error of OpenBoard caused by change of > AdvancedFeaturePkg > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2644 > > 1. Add all pathes of feature domains to package path. > 2. Add LogoFeaturePkg to TemporaryBuildWorkaround. > 3. Correct wrong codes and remove unnecessary codes in LogoFeaturePkg > > Cc: Dandan Bi <dandan.bi@intel.com> > Cc: Liming Gao <liming.gao@intel.com> > Cc: Eric Dong <eric.dong@intel.com> > Cc: Ray Ni <ray.ni@intel.com> > Signed-off-by: Heng Luo <heng.luo@intel.com> > --- > > Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryB > uildWorkaround.dsc | 4 +++- > > Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryB > uildWorkaround.inf | 5 ++++- > Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > | 9 --------- > Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > | 2 +- > Platform/Intel/build_bios.py | 7 ++++++- > 5 files changed, 14 insertions(+), 13 deletions(-) > > diff --git > a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporary > BuildWorkaround.dsc > b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporary > BuildWorkaround.dsc > index 227ae00908..c62f9ecc6e 100644 > --- > a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporary > BuildWorkaround.dsc > +++ > b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporary > BuildWorkaround.dsc > @@ -13,7 +13,7 @@ > # When the BaseTools update is complete, this file can entirely be removed > > # from this package. > > # > > -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > > +# Copyright (c) 2019 - 2020, Intel Corporation. All rights reserved.<BR> > > # > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > # > > @@ -49,6 +49,8 @@ > gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosFeatureEnable > |FALSE > > gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugFeatureEnable > |FALSE > > gUserAuthFeaturePkgTokenSpaceGuid.PcdUserAuthenticationFeatureEnable > |FALSE > > + gLogoFeaturePkgTokenSpaceGuid.PcdLogoFeatureEnable > |FALSE > > + gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable |FALSE > > !endif > > > > # > > diff --git > a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporary > BuildWorkaround.inf > b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporary > BuildWorkaround.inf > index 74176d1989..00818fbe0a 100644 > --- > a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporary > BuildWorkaround.inf > +++ > b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporary > BuildWorkaround.inf > @@ -13,7 +13,7 @@ > # When the BaseTools update is complete, this file can entirely be removed > > # from this package. > > # > > -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > > +# Copyright (c) 2019 - 2020, Intel Corporation. All rights reserved.<BR> > > # > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > # > > @@ -40,6 +40,7 @@ > PowerManagement/S3FeaturePkg/S3FeaturePkg.dec > > SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec > > UserInterface/UserAuthFeaturePkg/UserAuthFeaturePkg.dec > > + UserInterface/LogoFeaturePkg/LogoFeaturePkg.dec > > > > [FeaturePcd] > > gAcpiDebugFeaturePkgTokenSpaceGuid.PcdAcpiDebugFeatureEnable > > @@ -49,6 +50,8 @@ > gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosFeatureEnable > > gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugFeatureEnable > > gUserAuthFeaturePkgTokenSpaceGuid.PcdUserAuthenticationFeatureEnable > > + gLogoFeaturePkgTokenSpaceGuid.PcdLogoFeatureEnable > > + gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable > > > > [Sources] > > TemporaryBuildWorkaround.c > > diff --git > a/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > b/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > index fca0bfd540..d2dcdeb36a 100644 > --- a/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > +++ b/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > @@ -25,15 +25,6 @@ > !error "DXE_ARCH must be specified to build this feature!" > > !endif > > > > - > ################################################################ > ################ > > -# > > -# Packages Section - Make sure PCD can be directly used in a conditional > statement > > -# in a DSC which includes this DSC file. > > -# > > - > ################################################################ > ################ > > -[Packages] > > - LogoFeaturePkg/LogoFeaturePkg.dec > > - > > > ################################################################ > ################ > > # > > # Library Class section - list of all Library Classes needed by this feature. > > diff --git > a/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > b/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > index 080c87223c..fead9f3b02 100644 > --- a/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > +++ b/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > @@ -6,7 +6,7 @@ > # SPDX-License-Identifier: BSD-2-Clause-Patent > > # > > ## > > -!if gSmbiosFeaturePkgTokenSpaceGuid.PcdJpgEnable == TRUE > > +!if gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable == TRUE > > INF LogoFeaturePkg/LogoDxe/JpegLogoDxe.inf > > !else > > INF LogoFeaturePkg/LogoDxe/LogoDxe.inf > > diff --git a/Platform/Intel/build_bios.py b/Platform/Intel/build_bios.py > index 1ef35aca0a..b9ad980510 100644 > --- a/Platform/Intel/build_bios.py > +++ b/Platform/Intel/build_bios.py > @@ -3,7 +3,7 @@ > # Builds BIOS using configuration files and dynamically > > # imported functions from board directory > > # > > -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > > +# Copyright (c) 2019 - 2020, Intel Corporation. All rights reserved.<BR> > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > # > > > > @@ -120,6 +120,11 @@ def pre_build(build_config, build_type="DEBUG", > silent=False, toolchain=None): > config["PACKAGES_PATH"] += os.pathsep + config["WORKSPACE_SILICON"] > > config["PACKAGES_PATH"] += os.pathsep + > config["WORKSPACE_SILICON_BIN"] > > config["PACKAGES_PATH"] += os.pathsep + config["WORKSPACE_FEATURES"] > > + # add all feature domains in WORKSPACE_FEATURES to package path > > + for filename in os.listdir(config["WORKSPACE_FEATURES"]): > > + filepath = os.path.join(config["WORKSPACE_FEATURES"], filename) > > + if os.path.isdir(filepath): > > + config["PACKAGES_PATH"] += os.pathsep + filepath > > config["PACKAGES_PATH"] += os.pathsep + config["WORKSPACE_DRIVERS"] > > config["PACKAGES_PATH"] += os.pathsep + \ > > os.path.join(config["WORKSPACE"], "FSP") > > -- > 2.24.0.windows.2 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Fix build error of OpenBoard caused by change of AdvancedFeaturePkg 2020-03-30 4:38 ` Ni, Ray @ 2020-03-30 5:55 ` Heng Luo 0 siblings, 0 replies; 3+ messages in thread From: Heng Luo @ 2020-03-30 5:55 UTC (permalink / raw) To: Ni, Ray, devel@edk2.groups.io; +Cc: Bi, Dandan, Gao, Liming, Dong, Eric OK, I will send the separated patches. Best Regards Heng > -----Original Message----- > From: Ni, Ray <ray.ni@intel.com> > Sent: Monday, March 30, 2020 12:38 PM > To: Luo, Heng <heng.luo@intel.com>; devel@edk2.groups.io > Cc: Bi, Dandan <dandan.bi@intel.com>; Gao, Liming <liming.gao@intel.com>; > Dong, Eric <eric.dong@intel.com> > Subject: RE: [PATCH] Fix build error of OpenBoard caused by change of > AdvancedFeaturePkg > > Heng, > Can you please separate the patch to 3 so each patch fixes one issue > specifically? > > I am fine with one Bugzilla to cover the 3 issues. > > Thanks, > Ray > > > -----Original Message----- > > From: Luo, Heng <heng.luo@intel.com> > > Sent: Monday, March 30, 2020 12:30 PM > > To: devel@edk2.groups.io > > Cc: Bi, Dandan <dandan.bi@intel.com>; Gao, Liming > > <liming.gao@intel.com>; Dong, Eric <eric.dong@intel.com>; Ni, Ray > > <ray.ni@intel.com> > > Subject: [PATCH] Fix build error of OpenBoard caused by change of > > AdvancedFeaturePkg > > > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2644 > > > > 1. Add all pathes of feature domains to package path. > > 2. Add LogoFeaturePkg to TemporaryBuildWorkaround. > > 3. Correct wrong codes and remove unnecessary codes in LogoFeaturePkg > > > > Cc: Dandan Bi <dandan.bi@intel.com> > > Cc: Liming Gao <liming.gao@intel.com> > > Cc: Eric Dong <eric.dong@intel.com> > > Cc: Ray Ni <ray.ni@intel.com> > > Signed-off-by: Heng Luo <heng.luo@intel.com> > > --- > > > > > Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporar > yB > > uildWorkaround.dsc | 4 +++- > > > > > Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Temporar > yB > > uildWorkaround.inf | 5 ++++- > > Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > > | 9 --------- > > Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > > | 2 +- > > Platform/Intel/build_bios.py | 7 ++++++- > > 5 files changed, 14 insertions(+), 13 deletions(-) > > > > diff --git > > > a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Tempor > ary > > BuildWorkaround.dsc > > > b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Tempor > ary > > BuildWorkaround.dsc > > index 227ae00908..c62f9ecc6e 100644 > > --- > > > a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Tempor > ary > > BuildWorkaround.dsc > > +++ > > > b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Tempor > ary > > BuildWorkaround.dsc > > @@ -13,7 +13,7 @@ > > # When the BaseTools update is complete, this file can entirely be > > removed > > > > # from this package. > > > > # > > > > -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > > > > +# Copyright (c) 2019 - 2020, Intel Corporation. All rights > > +reserved.<BR> > > > > # > > > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > > > # > > > > @@ -49,6 +49,8 @@ > > gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosFeatureEnable > > |FALSE > > > > gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugFeatureEnable > > |FALSE > > > > > > > gUserAuthFeaturePkgTokenSpaceGuid.PcdUserAuthenticationFeatureEnable > > |FALSE > > > > + gLogoFeaturePkgTokenSpaceGuid.PcdLogoFeatureEnable > > |FALSE > > > > + gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable |FALSE > > > > !endif > > > > > > > > # > > > > diff --git > > > a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Tempor > ary > > BuildWorkaround.inf > > > b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Tempor > ary > > BuildWorkaround.inf > > index 74176d1989..00818fbe0a 100644 > > --- > > > a/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Tempor > ary > > BuildWorkaround.inf > > +++ > > > b/Features/Intel/AdvancedFeaturePkg/TemporaryBuildWorkaround/Tempor > ary > > BuildWorkaround.inf > > @@ -13,7 +13,7 @@ > > # When the BaseTools update is complete, this file can entirely be > > removed > > > > # from this package. > > > > # > > > > -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > > > > +# Copyright (c) 2019 - 2020, Intel Corporation. All rights > > +reserved.<BR> > > > > # > > > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > > > # > > > > @@ -40,6 +40,7 @@ > > PowerManagement/S3FeaturePkg/S3FeaturePkg.dec > > > > SystemInformation/SmbiosFeaturePkg/SmbiosFeaturePkg.dec > > > > UserInterface/UserAuthFeaturePkg/UserAuthFeaturePkg.dec > > > > + UserInterface/LogoFeaturePkg/LogoFeaturePkg.dec > > > > > > > > [FeaturePcd] > > > > gAcpiDebugFeaturePkgTokenSpaceGuid.PcdAcpiDebugFeatureEnable > > > > @@ -49,6 +50,8 @@ > > gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosFeatureEnable > > > > gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugFeatureEnable > > > > > > > gUserAuthFeaturePkgTokenSpaceGuid.PcdUserAuthenticationFeatureEnable > > > > + gLogoFeaturePkgTokenSpaceGuid.PcdLogoFeatureEnable > > > > + gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable > > > > > > > > [Sources] > > > > TemporaryBuildWorkaround.c > > > > diff --git > > a/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > > b/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > > index fca0bfd540..d2dcdeb36a 100644 > > --- > > a/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc > > +++ b/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature. > > +++ dsc > > @@ -25,15 +25,6 @@ > > !error "DXE_ARCH must be specified to build this feature!" > > > > !endif > > > > > > > > - > > > ################################################################ > > ################ > > > > -# > > > > -# Packages Section - Make sure PCD can be directly used in a > > conditional statement > > > > -# in a DSC which includes this DSC file. > > > > -# > > > > - > > > ################################################################ > > ################ > > > > -[Packages] > > > > - LogoFeaturePkg/LogoFeaturePkg.dec > > > > - > > > > > > > ################################################################ > > ################ > > > > # > > > > # Library Class section - list of all Library Classes needed by this feature. > > > > diff --git > > a/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > > b/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > > index 080c87223c..fead9f3b02 100644 > > --- > > a/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.fdf > > +++ b/Features/Intel/UserInterface/LogoFeaturePkg/Include/PostMemory.f > > +++ df > > @@ -6,7 +6,7 @@ > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > > > # > > > > ## > > > > -!if gSmbiosFeaturePkgTokenSpaceGuid.PcdJpgEnable == TRUE > > > > +!if gLogoFeaturePkgTokenSpaceGuid.PcdJpgEnable == TRUE > > > > INF LogoFeaturePkg/LogoDxe/JpegLogoDxe.inf > > > > !else > > > > INF LogoFeaturePkg/LogoDxe/LogoDxe.inf > > > > diff --git a/Platform/Intel/build_bios.py > > b/Platform/Intel/build_bios.py index 1ef35aca0a..b9ad980510 100644 > > --- a/Platform/Intel/build_bios.py > > +++ b/Platform/Intel/build_bios.py > > @@ -3,7 +3,7 @@ > > # Builds BIOS using configuration files and dynamically > > > > # imported functions from board directory > > > > # > > > > -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > > > > +# Copyright (c) 2019 - 2020, Intel Corporation. All rights > > +reserved.<BR> > > > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > > > # > > > > > > > > @@ -120,6 +120,11 @@ def pre_build(build_config, build_type="DEBUG", > > silent=False, toolchain=None): > > config["PACKAGES_PATH"] += os.pathsep + > > config["WORKSPACE_SILICON"] > > > > config["PACKAGES_PATH"] += os.pathsep + > > config["WORKSPACE_SILICON_BIN"] > > > > config["PACKAGES_PATH"] += os.pathsep + > > config["WORKSPACE_FEATURES"] > > > > + # add all feature domains in WORKSPACE_FEATURES to package path > > > > + for filename in os.listdir(config["WORKSPACE_FEATURES"]): > > > > + filepath = os.path.join(config["WORKSPACE_FEATURES"], > > + filename) > > > > + if os.path.isdir(filepath): > > > > + config["PACKAGES_PATH"] += os.pathsep + filepath > > > > config["PACKAGES_PATH"] += os.pathsep + > > config["WORKSPACE_DRIVERS"] > > > > config["PACKAGES_PATH"] += os.pathsep + \ > > > > os.path.join(config["WORKSPACE"], "FSP") > > > > -- > > 2.24.0.windows.2 ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-03-30 5:55 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-03-30 4:29 [PATCH] Fix build error of OpenBoard caused by change of AdvancedFeaturePkg Heng Luo 2020-03-30 4:38 ` Ni, Ray 2020-03-30 5:55 ` Heng Luo
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox