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 <liming....@intel.com> > Subject: [edk2] [Patch] BaseTools: Fix the build report issue for VPD > > From: BobCF <bob.c.f...@intel.com> > > 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 <bob.c.f...@intel.com> > Cc: Liming Gao <liming....@intel.com> > --- > 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