[edk2] [Patch] BaseTools: Fix the build report issue for VPD
From: BobCF Make sure the PlatformAutoGen.AllPcdList always be latest, or build report will fail sometimes. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng Cc: Liming Gao --- BaseTools/Source/Python/AutoGen/AutoGen.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen/AutoGen.py index cfe2d29099..85d966c62f 100644 --- a/BaseTools/Source/Python/AutoGen/AutoGen.py +++ b/BaseTools/Source/Python/AutoGen/AutoGen.py @@ -1029,11 +1029,10 @@ class PlatformAutoGen(AutoGen): self.Arch = Arch self.SourceDir = PlatformFile.SubDir self.SourceOverrideDir = None self.FdTargetList = self.Workspace.FdTargetList self.FvTargetList = self.Workspace.FvTargetList -self.AllPcdList = [] # get the original module/package/platform objects self.BuildDatabase = Workspace.BuildDatabase self.DscBuildDataObj = Workspace.Platform # flag indicating if the makefile/C-code file has been created or not @@ -1110,10 +1109,13 @@ class PlatformAutoGen(AutoGen): self.LibraryBuildDirectoryList = Makefile.GetLibraryBuildDirectoryList() self.ModuleBuildDirectoryList = Makefile.GetModuleBuildDirectoryList() self.IsMakeFileCreated = True +@property +def AllPcdList(self): +return self.DynamicPcdList + self.NonDynamicPcdList ## Deal with Shared FixedAtBuild Pcds # def CollectFixedAtBuildPcds(self): for LibAuto in self.LibraryAutoGenList: FixedAtBuildPcds = {} @@ -1624,11 +1626,10 @@ class PlatformAutoGen(AutoGen): if type(SkuId) in (str, unicode) and eval(SkuId) == 0 or SkuId == 0: continue pcd.SkuInfoList[SkuName] = copy.deepcopy(pcd.SkuInfoList[TAB_DEFAULT]) pcd.SkuInfoList[SkuName].SkuId = SkuId pcd.SkuInfoList[SkuName].SkuIdName = SkuName -self.AllPcdList = self._NonDynamicPcdList + self._DynamicPcdList def FixVpdOffset(self, VpdFile ): FvPath = os.path.join(self.BuildDir, TAB_FV_DIRECTORY) if not os.path.exists(FvPath): try: -- 2.19.1.windows.1 ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [Patch] BaseTools: Fix the build report issue for VPD
Bob: Please submit BZ for this issue. Could we merge these two patches? Seemly, they are both build report issue. Thanks Liming > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Feng, > Bob C > Sent: Wednesday, January 16, 2019 11:30 PM > To: edk2-devel@lists.01.org > Cc: Gao, Liming > Subject: [edk2] [Patch] BaseTools: Fix the build report issue for VPD > > From: BobCF > > Make sure the PlatformAutoGen.AllPcdList always be latest, > or build report will fail sometimes. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Bob Feng > Cc: Liming Gao > --- > BaseTools/Source/Python/AutoGen/AutoGen.py | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py > b/BaseTools/Source/Python/AutoGen/AutoGen.py > index cfe2d29099..85d966c62f 100644 > --- a/BaseTools/Source/Python/AutoGen/AutoGen.py > +++ b/BaseTools/Source/Python/AutoGen/AutoGen.py > @@ -1029,11 +1029,10 @@ class PlatformAutoGen(AutoGen): > self.Arch = Arch > self.SourceDir = PlatformFile.SubDir > self.SourceOverrideDir = None > self.FdTargetList = self.Workspace.FdTargetList > self.FvTargetList = self.Workspace.FvTargetList > -self.AllPcdList = [] > # get the original module/package/platform objects > self.BuildDatabase = Workspace.BuildDatabase > self.DscBuildDataObj = Workspace.Platform > > # flag indicating if the makefile/C-code file has been created or not > @@ -1110,10 +1109,13 @@ class PlatformAutoGen(AutoGen): > self.LibraryBuildDirectoryList = > Makefile.GetLibraryBuildDirectoryList() > self.ModuleBuildDirectoryList = > Makefile.GetModuleBuildDirectoryList() > > self.IsMakeFileCreated = True > > +@property > +def AllPcdList(self): > +return self.DynamicPcdList + self.NonDynamicPcdList > ## Deal with Shared FixedAtBuild Pcds > # > def CollectFixedAtBuildPcds(self): > for LibAuto in self.LibraryAutoGenList: > FixedAtBuildPcds = {} > @@ -1624,11 +1626,10 @@ class PlatformAutoGen(AutoGen): > if type(SkuId) in (str, unicode) and eval(SkuId) == 0 or > SkuId == 0: > continue > pcd.SkuInfoList[SkuName] = > copy.deepcopy(pcd.SkuInfoList[TAB_DEFAULT]) > pcd.SkuInfoList[SkuName].SkuId = SkuId > pcd.SkuInfoList[SkuName].SkuIdName = SkuName > -self.AllPcdList = self._NonDynamicPcdList + self._DynamicPcdList > > def FixVpdOffset(self, VpdFile ): > FvPath = os.path.join(self.BuildDir, TAB_FV_DIRECTORY) > if not os.path.exists(FvPath): > try: > -- > 2.19.1.windows.1 > > ___ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [Patch] BaseTools: Fix the build report issue for VPD
OK. I'll submit BZ. The two patches are to fix two issues that have different root cause, so I think two patches would be better. -Original Message- From: Gao, Liming Sent: Thursday, January 17, 2019 12:03 AM To: Feng, Bob C ; edk2-devel@lists.01.org Subject: RE: [edk2] [Patch] BaseTools: Fix the build report issue for VPD Bob: Please submit BZ for this issue. Could we merge these two patches? Seemly, they are both build report issue. Thanks Liming > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Feng, Bob C > Sent: Wednesday, January 16, 2019 11:30 PM > To: edk2-devel@lists.01.org > Cc: Gao, Liming > Subject: [edk2] [Patch] BaseTools: Fix the build report issue for VPD > > From: BobCF > > Make sure the PlatformAutoGen.AllPcdList always be latest, or build > report will fail sometimes. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Bob Feng > Cc: Liming Gao > --- > BaseTools/Source/Python/AutoGen/AutoGen.py | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py > b/BaseTools/Source/Python/AutoGen/AutoGen.py > index cfe2d29099..85d966c62f 100644 > --- a/BaseTools/Source/Python/AutoGen/AutoGen.py > +++ b/BaseTools/Source/Python/AutoGen/AutoGen.py > @@ -1029,11 +1029,10 @@ class PlatformAutoGen(AutoGen): > self.Arch = Arch > self.SourceDir = PlatformFile.SubDir > self.SourceOverrideDir = None > self.FdTargetList = self.Workspace.FdTargetList > self.FvTargetList = self.Workspace.FvTargetList > -self.AllPcdList = [] > # get the original module/package/platform objects > self.BuildDatabase = Workspace.BuildDatabase > self.DscBuildDataObj = Workspace.Platform > > # flag indicating if the makefile/C-code file has been > created or not @@ -1110,10 +1109,13 @@ class PlatformAutoGen(AutoGen): > self.LibraryBuildDirectoryList = > Makefile.GetLibraryBuildDirectoryList() > self.ModuleBuildDirectoryList = > Makefile.GetModuleBuildDirectoryList() > > self.IsMakeFileCreated = True > > +@property > +def AllPcdList(self): > +return self.DynamicPcdList + self.NonDynamicPcdList > ## Deal with Shared FixedAtBuild Pcds > # > def CollectFixedAtBuildPcds(self): > for LibAuto in self.LibraryAutoGenList: > FixedAtBuildPcds = {} > @@ -1624,11 +1626,10 @@ class PlatformAutoGen(AutoGen): > if type(SkuId) in (str, unicode) and eval(SkuId) == 0 or > SkuId == 0: > continue > pcd.SkuInfoList[SkuName] = > copy.deepcopy(pcd.SkuInfoList[TAB_DEFAULT]) > pcd.SkuInfoList[SkuName].SkuId = SkuId > pcd.SkuInfoList[SkuName].SkuIdName = SkuName > -self.AllPcdList = self._NonDynamicPcdList + self._DynamicPcdList > > def FixVpdOffset(self, VpdFile ): > FvPath = os.path.join(self.BuildDir, TAB_FV_DIRECTORY) > if not os.path.exists(FvPath): > try: > -- > 2.19.1.windows.1 > > ___ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel