BaseTools/Source/Python/AutoGen/IncludesAutoGen.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
From: Mingyue Liang <mingyuex.liang@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.liang@intel.com>
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Yuwei Chen <yuwei.chen@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 (#65526): https://edk2.groups.io/g/devel/message/65526
Mute This Topic: https://groups.io/mt/77032313/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Reviewed-by: Yuwei Chen<yuwei.chen@intel.com> > -----Original Message----- > From: Feng, Bob C <bob.c.feng@intel.com> > Sent: Wednesday, September 23, 2020 6:52 PM > To: devel@edk2.groups.io > Cc: Liang, MingyueX <mingyuex.liang@intel.com>; Liming Gao > <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com> > Subject: [PATCH] BaseTools: Add included files to deps_target file. > > From: Mingyue Liang <mingyuex.liang@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.liang@intel.com> > Cc: Bob Feng <bob.c.feng@intel.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn> > Cc: Yuwei Chen <yuwei.chen@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/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.