Reviewed-by: Yuwei Chen<yuwei.c...@intel.com> > -----Original Message----- > From: Feng, Bob C <bob.c.f...@intel.com> > Sent: Wednesday, September 23, 2020 6:52 PM > To: devel@edk2.groups.io > Cc: Liang, MingyueX <mingyuex.li...@intel.com>; Liming Gao > <gaolim...@byosoft.com.cn>; Chen, Christine <yuwei.c...@intel.com> > Subject: [PATCH] BaseTools: Add included files to deps_target file. > > From: Mingyue Liang <mingyuex.li...@intel.com> > > 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 <mingyuex.li...@intel.com> > Cc: Bob Feng <bob.c.f...@intel.com> > Cc: Liming Gao <gaolim...@byosoft.com.cn> > Cc: Yuwei Chen <yuwei.c...@intel.com> > --- > 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: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-