Reviewed-by: Yuwei Chen<[email protected]> > -----Original Message----- > From: Feng, Bob C <[email protected]> > Sent: Wednesday, September 23, 2020 6:52 PM > To: [email protected] > Cc: Liang, MingyueX <[email protected]>; Liming Gao > <[email protected]>; Chen, Christine <[email protected]> > Subject: [PATCH] BaseTools: Add included files to deps_target file. > > From: Mingyue Liang <[email protected]> > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2882 > > After changing the name of the include source file, when doing incremental > build, the previous source file is not covered in the. DEPs file, and a build > error occurs. > > The root cause is that the build tools filter out some dependency files, which > are listed in inf source section, from the deps_target file. > Add those files back to deps_target file to resolve the above problem. > > Signed-off-by: Mingyue Liang <[email protected]> > Cc: Bob Feng <[email protected]> > Cc: Liming Gao <[email protected]> > Cc: Yuwei Chen <[email protected]> > --- > BaseTools/Source/Python/AutoGen/IncludesAutoGen.py | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/BaseTools/Source/Python/AutoGen/IncludesAutoGen.py > b/BaseTools/Source/Python/AutoGen/IncludesAutoGen.py > index 720d93395a..c3e6333217 100644 > --- a/BaseTools/Source/Python/AutoGen/IncludesAutoGen.py > +++ b/BaseTools/Source/Python/AutoGen/IncludesAutoGen.py > @@ -103,7 +103,7 @@ ${END} > if os.path.normpath(dependency_file +".deps") == abspath: > continue > filename = os.path.basename(dependency_file).strip() > - if filename not in self.SourceFileList and filename not > in > targetname: > + if filename not in targetname: > includes.add(dependency_file.strip()) > > for item in lines[1:]: > @@ -116,8 +116,6 @@ ${END} > if os.path.normpath(dependency_file +".deps") == abspath: > continue > filename = os.path.basename(dependency_file).strip() > - if filename in self.SourceFileList: > - continue > if filename in targetname: > continue > includes.add(dependency_file.strip()) > -- > 2.28.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#65559): https://edk2.groups.io/g/devel/message/65559 Mute This Topic: https://groups.io/mt/77032313/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
