Reviewed-by: Liming Gao <[email protected]> > -----Original Message----- > From: Feng, Bob C <[email protected]> > Sent: Thursday, September 19, 2019 10:57 AM > To: [email protected] > Cc: Gao, Liming <[email protected]>; Shi, Steven <[email protected]>; > Feng, Bob C <[email protected]> > Subject: [Patch] BaseTools: init DependencyHeaderFileSet for each MakeFile > class > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2203 > > This patch is going to fix a regression issue that > build breaks when the custom makefile exist. > > Cc: Liming Gao <[email protected]> > Cc: Steven Shi <[email protected]> > Signed-off-by: Bob Feng <[email protected]> > --- > BaseTools/Source/Python/AutoGen/GenMake.py | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/BaseTools/Source/Python/AutoGen/GenMake.py > b/BaseTools/Source/Python/AutoGen/GenMake.py > index 940136248f..e447e43ec3 100755 > --- a/BaseTools/Source/Python/AutoGen/GenMake.py > +++ b/BaseTools/Source/Python/AutoGen/GenMake.py > @@ -447,10 +447,11 @@ cleanlib: > self.Macros["WORKSPACE" ] = > self._AutoGenObject.Macros["WORKSPACE"] > self.Macros["FFS_OUTPUT_DIR" ] = > self._AutoGenObject.Macros["FFS_OUTPUT_DIR"] > self.GenFfsList = ModuleAutoGen.GenFfsList > self.MacroList = ['FFS_OUTPUT_DIR', 'MODULE_GUID', 'OUTPUT_DIR'] > self.FfsOutputFileList = [] > + self.DependencyHeaderFileSet = set() > > # Compose a dict object containing information used to do replacement in > template > @property > def _TemplateDict(self): > if self._FileType not in self._SEP_: > @@ -908,11 +909,11 @@ cleanlib: > SourceFileList, > ForceIncludedFile, > self._AutoGenObject.IncludePathList + > self._AutoGenObject.BuildOptionIncPathList > ) > > - self.DependencyHeaderFileSet = set() > + > if FileDependencyDict: > for Dependency in FileDependencyDict.values(): > self.DependencyHeaderFileSet.update(set(Dependency)) > > # Get a set of unique package includes from MetaFile > @@ -1242,10 +1243,11 @@ ${BEGIN}\t-@${create_directory_command}\n${END}\ > # > def __init__(self, ModuleAutoGen): > BuildFile.__init__(self, ModuleAutoGen) > self.PlatformInfo = self._AutoGenObject.PlatformInfo > self.IntermediateDirectoryList = ["$(DEBUG_DIR)", "$(OUTPUT_DIR)"] > + self.DependencyHeaderFileSet = set() > > # Compose a dict object containing information used to do replacement in > template > @property > def _TemplateDict(self): > Separator = self._SEP_[self._FileType] > @@ -1432,10 +1434,11 @@ cleanlib: > self.ModuleMakefileList = [] > self.IntermediateDirectoryList = [] > self.ModuleBuildDirectoryList = [] > self.LibraryBuildDirectoryList = [] > self.LibraryMakeCommandList = [] > + self.DependencyHeaderFileSet = set() > > # Compose a dict object containing information used to do replacement in > template > @property > def _TemplateDict(self): > Separator = self._SEP_[self._FileType] > @@ -1537,10 +1540,11 @@ class TopLevelMakefile(BuildFile): > # @param Workspace Object of WorkspaceAutoGen class > # > def __init__(self, Workspace): > BuildFile.__init__(self, Workspace) > self.IntermediateDirectoryList = [] > + self.DependencyHeaderFileSet = set() > > # Compose a dict object containing information used to do replacement in > template > @property > def _TemplateDict(self): > Separator = self._SEP_[self._FileType] > -- > 2.20.1.windows.1
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#47852): https://edk2.groups.io/g/devel/message/47852 Mute This Topic: https://groups.io/mt/34195983/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
