public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.
@ 2020-11-09  2:51 Mingyue Liang
  2020-11-09  6:19 ` Bob Feng
       [not found] ` <1645C293A820CE46.8542@groups.io>
  0 siblings, 2 replies; 5+ messages in thread
From: Mingyue Liang @ 2020-11-09  2:51 UTC (permalink / raw)
  To: devel; +Cc: Bob Feng, Liming Gao, Yuwei Chen

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2972

When using incremental build to delete an included xxx.asi file from
the ASL file, the xxx.asl.trim.deps file generated by previous build
process will not be deleted from the OUTPUT directory, which caused
the dependency file still include the xxx.asl.trim.deps file.

If the include file is deleted and DEPs is empty.

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>
---
 V3:
  Modify Version Description location.
  
 BaseTools/Source/Python/Trim/Trim.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/BaseTools/Source/Python/Trim/Trim.py b/BaseTools/Source/Python/Trim/Trim.py
index 776474b20349..c479f7d2b2e7 100644
--- a/BaseTools/Source/Python/Trim/Trim.py
+++ b/BaseTools/Source/Python/Trim/Trim.py
@@ -372,8 +372,7 @@ def TrimAslFile(Source, Target, IncludePathFile,AslDeps = False):
     AslIncludes = []
     Lines = DoInclude(Source, '', IncludePathList,IncludeFileList=AslIncludes,filetype='ASL')
     AslIncludes = [item for item in AslIncludes if item !=Source]
-    if AslDeps and AslIncludes:
-        SaveFileOnChange(os.path.join(os.path.dirname(Target),os.path.basename(Source))+".trim.deps", " \\\n".join([Source+":"] +AslIncludes),False)
+    SaveFileOnChange(os.path.join(os.path.dirname(Target),os.path.basename(Source))+".trim.deps", " \\\n".join([Source+":"] +AslIncludes),False)
 
     #
     # Undef MIN and MAX to avoid collision in ASL source code
-- 
2.29.2.windows.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.
  2020-11-09  2:51 [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion Mingyue Liang
@ 2020-11-09  6:19 ` Bob Feng
       [not found] ` <1645C293A820CE46.8542@groups.io>
  1 sibling, 0 replies; 5+ messages in thread
From: Bob Feng @ 2020-11-09  6:19 UTC (permalink / raw)
  To: Liang, MingyueX, devel@edk2.groups.io; +Cc: Liming Gao, Chen, Christine

Reviewed-by: Bob Feng <bob.c.feng@intel.com>

-----Original Message-----
From: Mingyue Liang <mingyuex.liang@intel.com> 
Sent: Monday, November 9, 2020 10:51 AM
To: devel@edk2.groups.io
Cc: Feng, Bob C <bob.c.feng@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>
Subject: [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2972

When using incremental build to delete an included xxx.asi file from the ASL file, the xxx.asl.trim.deps file generated by previous build process will not be deleted from the OUTPUT directory, which caused the dependency file still include the xxx.asl.trim.deps file.

If the include file is deleted and DEPs is empty.

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>
---
 V3:
  Modify Version Description location.
  
 BaseTools/Source/Python/Trim/Trim.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/BaseTools/Source/Python/Trim/Trim.py b/BaseTools/Source/Python/Trim/Trim.py
index 776474b20349..c479f7d2b2e7 100644
--- a/BaseTools/Source/Python/Trim/Trim.py
+++ b/BaseTools/Source/Python/Trim/Trim.py
@@ -372,8 +372,7 @@ def TrimAslFile(Source, Target, IncludePathFile,AslDeps = False):
     AslIncludes = []
     Lines = DoInclude(Source, '', IncludePathList,IncludeFileList=AslIncludes,filetype='ASL')
     AslIncludes = [item for item in AslIncludes if item !=Source]
-    if AslDeps and AslIncludes:
-        SaveFileOnChange(os.path.join(os.path.dirname(Target),os.path.basename(Source))+".trim.deps", " \\\n".join([Source+":"] +AslIncludes),False)
+    
+ SaveFileOnChange(os.path.join(os.path.dirname(Target),os.path.basename
+ (Source))+".trim.deps", " \\\n".join([Source+":"] +AslIncludes),False)
 
     #
     # Undef MIN and MAX to avoid collision in ASL source code
--
2.29.2.windows.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.
       [not found] ` <1645C293A820CE46.8542@groups.io>
@ 2020-11-10  0:21   ` Bob Feng
  2020-11-10 20:56     ` Lendacky, Thomas
  0 siblings, 1 reply; 5+ messages in thread
From: Bob Feng @ 2020-11-10  0:21 UTC (permalink / raw)
  To: devel@edk2.groups.io, Feng, Bob C, Liang, MingyueX
  Cc: Liming Gao, Chen, Christine

Created the PR https://github.com/tianocore/edk2/pull/1102 

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Bob Feng
Sent: Monday, November 9, 2020 2:20 PM
To: Liang, MingyueX <mingyuex.liang@intel.com>; devel@edk2.groups.io
Cc: Liming Gao <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>
Subject: Re: [edk2-devel] [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.

Reviewed-by: Bob Feng <bob.c.feng@intel.com>

-----Original Message-----
From: Mingyue Liang <mingyuex.liang@intel.com> 
Sent: Monday, November 9, 2020 10:51 AM
To: devel@edk2.groups.io
Cc: Feng, Bob C <bob.c.feng@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>
Subject: [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2972

When using incremental build to delete an included xxx.asi file from the ASL file, the xxx.asl.trim.deps file generated by previous build process will not be deleted from the OUTPUT directory, which caused the dependency file still include the xxx.asl.trim.deps file.

If the include file is deleted and DEPs is empty.

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>
---
 V3:
  Modify Version Description location.

 BaseTools/Source/Python/Trim/Trim.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/BaseTools/Source/Python/Trim/Trim.py b/BaseTools/Source/Python/Trim/Trim.py
index 776474b20349..c479f7d2b2e7 100644
--- a/BaseTools/Source/Python/Trim/Trim.py
+++ b/BaseTools/Source/Python/Trim/Trim.py
@@ -372,8 +372,7 @@ def TrimAslFile(Source, Target, IncludePathFile,AslDeps = False):
     AslIncludes = []
     Lines = DoInclude(Source, '', IncludePathList,IncludeFileList=AslIncludes,filetype='ASL')
     AslIncludes = [item for item in AslIncludes if item !=Source]
-    if AslDeps and AslIncludes:
-        SaveFileOnChange(os.path.join(os.path.dirname(Target),os.path.basename(Source))+".trim.deps", " \\\n".join([Source+":"] +AslIncludes),False)
+
+ SaveFileOnChange(os.path.join(os.path.dirname(Target),os.path.basename
+ (Source))+".trim.deps", " \\\n".join([Source+":"] +AslIncludes),False)
 
     #
     # Undef MIN and MAX to avoid collision in ASL source code
--
2.29.2.windows.2







^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.
  2020-11-10  0:21   ` [edk2-devel] " Bob Feng
@ 2020-11-10 20:56     ` Lendacky, Thomas
  2020-11-11  6:05       ` Mingyue Liang
  0 siblings, 1 reply; 5+ messages in thread
From: Lendacky, Thomas @ 2020-11-10 20:56 UTC (permalink / raw)
  To: devel, bob.c.feng, Liang, MingyueX; +Cc: Liming Gao, Chen, Christine

After doing a git pull today, I started seeing these (error) messages in
my build:

build.py...                                                                                                                                                                                              
 : error 000E: /root/kernels/ovmf-build-X64/Build/OvmfX64/DEBUG_GCC5/X64/OvmfPkg/AcpiTables/AcpiTables/OUTPUT/Dsdt.asl.trim.deps doesn't exist                                                           
        list index out of range                                                                                                                                                                          
                                                                                                    
                                                                                                                                                                                                         
build.py...                                                                                                                                                                                              
 : error 000E: /root/kernels/ovmf-build-X64/Build/OvmfX64/DEBUG_GCC5/X64/OvmfPkg/AcpiTables/AcpiTables/OUTPUT/Ssdt.asl.trim.deps doesn't exist
        list index out of range                                                                     
Building ... /root/kernels/ovmf-build-X64/NetworkPkg/Library/DxeNetLib/DxeNetLib.inf [X64]                                                                                                               
                                                                                                    
                                                                                                    
build.py...                                                                                         
 : error 000E: /root/kernels/ovmf-build-X64/Build/OvmfX64/DEBUG_GCC5/X64/OvmfPkg/AcpiTables/AcpiTables/OUTPUT/Dsdt.asl.trim.deps doesn't exist
        list index out of range                                                                     
                                                                                                    
                                                                                                    
build.py...                                                                                                                                                                                              
 : error 000E: /root/kernels/ovmf-build-X64/Build/OvmfX64/DEBUG_GCC5/X64/OvmfPkg/AcpiTables/AcpiTables/OUTPUT/Ssdt.asl.trim.deps doesn't exist                                                           
        list index out of range                                                                     


I bisected it to this commit.

The build completes successfully, however. So is this just an "error" that
is really just informational?

Thanks,
Tom

On 11/9/20 6:21 PM, Bob Feng via groups.io wrote:
> Created the PR https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2Fpull%2F1102&amp;data=04%7C01%7Cthomas.lendacky%40amd.com%7C489d34e5582a4ca1766608d8850e9b7c%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637405645077123886%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=GcAFjR3rpnnRUzbDWhO5FaWki0kMCTBMpZLlzixkQ00%3D&amp;reserved=0 
> 
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Bob Feng
> Sent: Monday, November 9, 2020 2:20 PM
> To: Liang, MingyueX <mingyuex.liang@intel.com>; devel@edk2.groups.io
> Cc: Liming Gao <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>
> Subject: Re: [edk2-devel] [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.
> 
> Reviewed-by: Bob Feng <bob.c.feng@intel.com>
> 
> -----Original Message-----
> From: Mingyue Liang <mingyuex.liang@intel.com> 
> Sent: Monday, November 9, 2020 10:51 AM
> To: devel@edk2.groups.io
> Cc: Feng, Bob C <bob.c.feng@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>
> Subject: [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.
> 
> REF:https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugzilla.tianocore.org%2Fshow_bug.cgi%3Fid%3D2972&amp;data=04%7C01%7Cthomas.lendacky%40amd.com%7C489d34e5582a4ca1766608d8850e9b7c%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637405645077123886%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=dXdFf1EciwOH0PvZ%2BgmOeuAxOFULi3TXUFbYO1dR2O4%3D&amp;reserved=0
> 
> When using incremental build to delete an included xxx.asi file from the ASL file, the xxx.asl.trim.deps file generated by previous build process will not be deleted from the OUTPUT directory, which caused the dependency file still include the xxx.asl.trim.deps file.
> 
> If the include file is deleted and DEPs is empty.
> 
> 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>
> ---
>  V3:
>   Modify Version Description location.
> 
>  BaseTools/Source/Python/Trim/Trim.py | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/BaseTools/Source/Python/Trim/Trim.py b/BaseTools/Source/Python/Trim/Trim.py
> index 776474b20349..c479f7d2b2e7 100644
> --- a/BaseTools/Source/Python/Trim/Trim.py
> +++ b/BaseTools/Source/Python/Trim/Trim.py
> @@ -372,8 +372,7 @@ def TrimAslFile(Source, Target, IncludePathFile,AslDeps = False):
>      AslIncludes = []
>      Lines = DoInclude(Source, '', IncludePathList,IncludeFileList=AslIncludes,filetype='ASL')
>      AslIncludes = [item for item in AslIncludes if item !=Source]
> -    if AslDeps and AslIncludes:
> -        SaveFileOnChange(os.path.join(os.path.dirname(Target),os.path.basename(Source))+".trim.deps", " \\\n".join([Source+":"] +AslIncludes),False)
> +
> + SaveFileOnChange(os.path.join(os.path.dirname(Target),os.path.basename
> + (Source))+".trim.deps", " \\\n".join([Source+":"] +AslIncludes),False)
>  
>      #
>      # Undef MIN and MAX to avoid collision in ASL source code
> --
> 2.29.2.windows.2
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion.
  2020-11-10 20:56     ` Lendacky, Thomas
@ 2020-11-11  6:05       ` Mingyue Liang
  0 siblings, 0 replies; 5+ messages in thread
From: Mingyue Liang @ 2020-11-11  6:05 UTC (permalink / raw)
  To: Lendacky, Thomas, devel

[-- Attachment #1: Type: text/plain, Size: 102 bytes --]

Hi Tom

This is an information. I will make a patch to deal with this problem.

Thanks,
Mingyue

[-- Attachment #2: Type: text/html, Size: 141 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-11-11  6:05 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-09  2:51 [PATCH.V3] BaseTools: Incremental build issue for included ASI file's deletion Mingyue Liang
2020-11-09  6:19 ` Bob Feng
     [not found] ` <1645C293A820CE46.8542@groups.io>
2020-11-10  0:21   ` [edk2-devel] " Bob Feng
2020-11-10 20:56     ` Lendacky, Thomas
2020-11-11  6:05       ` Mingyue Liang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox