Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python VfrCompiler implementation

2023-12-07 Thread Yuwei Chen
Hi Liming, Is this feature been tested and reviewed these two weeks? 😊 Thanks, Christine From: devel@edk2.groups.io On Behalf Of Yuwei Chen Sent: Monday, November 13, 2023 8:32 AM To: Gao, Liming ; 'Rebecca Cran' ; Gao, Liming ; Zimmer, Vincent ; Kinney, Michael D ; 

[edk2-devel] [PATCH] [edk2-staging]BaseTools: Python VfrCompiler Tool Update

2023-12-07 Thread Yuwei Chen
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#112216): https://edk2.groups.io/g/devel/message/112216 Mute This Topic: https://groups.io/mt/103051162/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/dev

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-12-11 Thread Yuwei Chen
istine ; > > > Feng, Bob C > > > Cc: 'Rebecca Cran' ; Guo, Gua > > > > Subject: 回复: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT > > ShrinkFv > > > issue > > > > > > Yuwei: > > > Please update commit m

Re: [edk2-devel] [Patch V2] BaseTools: VfrCompiler Adds DefaultValueError Feature

2023-12-12 Thread Yuwei Chen
+Cc Yang, Yuting2 > -Original Message- > From: Yang, Yuting2 > Sent: Tuesday, December 12, 2023 5:01 PM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Gao, Liming > ; Feng, Bob C ; Chen, > Christine > Subject: [Patch V2] BaseTools: VfrCompiler Adds DefaultValueError Feature > > Add --c

[edk2-devel] [PATCH] BaseTools: FMMT GuidTool Auto Select Config file Enabling

2023-12-15 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4624 Currently, Python FMMT tool does not support automatically select FMMTConf.ini file which saves GuidTool settings. This patch supports this features. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen

Re: [edk2-devel] [PATCH] BaseTools: FMMT GuidTool Auto Select Config file Enabling

2023-12-15 Thread Yuwei Chen
Edk2-basetools repo PR link: https://github.com/tianocore/edk2-basetools/pull/100 Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Friday, December 15, 2023 5:12 PM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Ga

Re: [edk2-devel] [PATCH V2 1/1] Silicon/Intel/FitGen:FIT change for FBM entry.

2023-12-17 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Liu, Liqi > Sent: Tuesday, December 12, 2023 4:42 PM > To: devel@edk2.groups.io > Cc: Liu, Liqi ; Chen, Christine ; > Feng, Bob C ; Chaganty, Rangasai V > ; Gao, Liming > Subject: [PATCH V2 1/1] Silicon/Int

Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset

2024-01-07 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: S, Ashraf Ali > Sent: Monday, December 25, 2023 11:28 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali ; Chen, Christine > ; Rebecca Cran ; Gao, Liming > ; Feng, Bob C ; Chan, > Amy ; Chaganty, Rangasa

Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Fix raw strings containing valid escape characters

2024-01-07 Thread Yuwei Chen
Hi Joey, Thanks for fixing. Reviewed-by: Yuwei Chen > -Original Message- > From: Joey Vagedes > Sent: Friday, December 29, 2023 12:48 AM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Gao, Liming > ; Feng, Bob C ; Chen, > Christine > Subject: [PATCH v1

Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Fix raw strings containing valid escape characters

2024-01-07 Thread Yuwei Chen
Hi Liming and Cran, We should merge this fixing patch asap as it influences BaseTool's usage. Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Monday, January 8, 2024 9:16 AM > To: Joey Vagedes ; devel@edk2.gro

Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Fix raw strings containing valid escape characters

2024-01-09 Thread Yuwei Chen
Christine: Is this a regression issue? Or does it resolve new usage? I am curious why it is not reported before. Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io<mailto:devel@edk2.groups.io> > mailto:devel@edk2.groups.io>> 代表 Yuwei Chen > 发送时间: 2024年1月

Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python VfrCompiler implementation

2024-01-09 Thread Yuwei Chen
> > C ; Yang, Yuting2 ; > > Hartung, Stephen > > Subject: Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python > > VfrCompiler implementation > > > > On Thu, Dec 7, 2023 at 9:08 AM Yuwei Chen > wrote: > > > > > > Hi Liming, >

Re: [edk2-devel] [PATCH v3 2/7] BaseTools/Plugin/CodeQL: Add CodeQL build plugin

2023-10-24 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Wednesday, October 18, 2023 9:05 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Kinney, Michael D > ; Rebecca Cran ; Sean > Brogan ; Chen, Christine >

Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python VfrCompiler implementation

2023-11-12 Thread Yuwei Chen
Hi Liming, I know your point. Would you like to start reviewing it and give us more feedbacks? We really hope it can be merged as soon as possible. We can do the merging after the stable tag, but just like you said, maybe we need to review it firstly. Looking forward to your feedbacks, and I th

Re: [edk2-devel] [PATCH 1/1] BaseTools/Scripts/PatchCheck.py: Check for Change-id

2023-11-28 Thread Yuwei Chen
The patch is good for me. Reviewed-by: Yuwei Chen > -Original Message- > From: Pierre Gondois > Sent: Wednesday, November 22, 2023 9:15 PM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Gao, Liming > ; Feng, Bob C ; Chen, > Christine ; Sami Mujawar > ; yeore

Re: [edk2-devel] [PATCH 1/1] BaseTools/Scripts/PatchCheck.py: Check for Change-id

2023-11-28 Thread Yuwei Chen
i > Mujawar ; yeoreum@arm.com > Subject: RE: [edk2-devel] [PATCH 1/1] BaseTools/Scripts/PatchCheck.py: Check > for Change-id > > It's good. But I am curious why --ignore-change-id is needed? > > Thanks, > Ray > > -Original Message- > > Fr

Re: [edk2-devel] [PATCH v1 1/1] BaseTools/Conf/target.template: Use VS2019 as default tool chain

2023-11-28 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Wednesday, November 29, 2023 3:49 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Rebecca Cran ; Chen, > Christine > Subject: [PATCH v1 1/1] BaseTools/C

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-11-29 Thread Yuwei Chen
Hi Liming and Cran, I have created the Edk2 PR for this one: https://github.com/tianocore/edk2/pull/5009 As it has been reviewed, please help merge that. Thanks~ Thanks, Yuwei (Christine) > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen >

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-12-03 Thread Yuwei Chen
I will add push > label for this patch set. > > Thanks > Liming > > -邮件原件- > > 发件人: devel@edk2.groups.io 代表 Yuwei Chen > > 发送时间: 2023年11月29日 16:26 > > 收件人: Gao, Liming ; Feng, Bob C > > > > 抄送: 'Rebecca Cran' ; devel@edk2.groups.io; Ch

Re: [edk2-devel] Fixing edk2-basetools CI

2024-02-22 Thread Yuwei Chen
Thanks a lot Rebecca~~ Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of Michael > D Kinney > Sent: Saturday, February 17, 2024 1:24 AM > To: Rebecca Cran ; Joey Vagedes > ; Rebecca Cran > ; devel@edk2.groups.io; Sean > ; Michael Kubacki > > Cc: Kinney, Mi

Re: [edk2-devel] [PATCH] BaseTools: FMMT Skip empty Lines while parsing FMMTConfig.ini

2024-03-11 Thread Yuwei Chen
Ali, please also create PR in BaseTools repo. Reviewed-by: Yuwei Chen > -Original Message- > From: S, Ashraf Ali > Sent: Wednesday, February 28, 2024 5:57 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali ; Rebecca Cran > ; Liming Gao ; Feng, Bob > C ; Chen, C

Re: [edk2-devel] [Patch V2] [edk2-staging] BaseTools: Update ReadMe in Python VfrCompiler Tool

2024-03-27 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Yang, Yuting2 > Sent: Wednesday, March 27, 2024 4:43 PM > To: devel@edk2.groups.io > Cc: Rebecca Cran ; Liming Gao > ; Feng, Bob C ; Chen, > Christine > Subject: [Patch V2] [edk2-staging] BaseTools:

[edk2-devel] [PATCH] BaseTools: Add FMMT Return Code for Error checking

2024-04-11 Thread Yuwei Chen
Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/BaseTools/Source/Python/FMMT/FMMT.py b/BaseTools/Source/Python/FMMT/FMMT.py index 26fc4c5792..d9f1c265a7 100644 --- a

Re: [edk2-devel][PATCH][edk2-staging] BaseTools: Fix BaseTools compilation issues

2024-07-11 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Kuo, Ted > Sent: Friday, July 5, 2024 5:32 PM > To: devel@edk2.groups.io > Cc: Liming Gao ; Feng, Bob C > ; Chen, Christine > Subject: [edk2-devel][PATCH][edk2-staging] BaseTools: Fix BaseTools > compi

[edk2-devel] [PATCH 1/2] BaseTools/FMMT: Add Extract FV function

2022-09-15 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3938 With this patch "-e" parameter supports extract FV function. Usage: FMMT -e Inputfile TargetFv Outputfile Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 3 +-

[edk2-devel] [PATCH 2/2] BaseTools/FMMT: Add Shrink Fv function

2022-09-15 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3938 This function is used to remove the useless FV free space. Usage: FMMT -s Inputfile Outputfile Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 7 + .../Source/Python/FMMT

[edk2-devel] [PATCH] BaseTools: FMMT replace output file is not generated successfully

2022-10-18 Thread Yuwei Chen
For replace function, when target Ffs and new ffs are with same size, the output file can not be generated successfully. This patch fixes this issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/FMMT/core/BiosTree.py | 4 ++-- edk2basetools/FMMT/core/FvHandler.py

Re: [edk2-devel] [PATCH 1/1] pip-requirement: Upgrade the edk2-basetools version from 0.1.29 to 0.1.39

2022-10-18 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Sunday, October 16, 2022 8:11 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine > Subject: [PATCH 1/1] pip-requirement: Upgrade the edk2-basetools version &g

[edk2-devel] [PATCH] BaseTools: Fix build option overrides Pcd Feature Flag issue

2022-10-25 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4081 INF [Sources] section Feature Flag Expressions do not use override values from build --pcd option currently. This patch fix this issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/Workspace/InfBuildData.py

[edk2-devel] [Patch v4] BaseTools: Add Guid name support in GenFfs.

2020-07-13 Thread Yuwei Chen
: I29b52663613286c6210d99eaaa09338677a21b31 Signed-off-by: Yuwei Chen Cc: Bob Feng Cc: Liming Gao --- .../Source/Python/AutoGen/WorkspaceAutoGen.py | 24 +++ BaseTools/Source/Python/Common/GlobalData.py | 1 + BaseTools/Source/Python/GenFds/FdfParser.py | 14 ++- 3 files

[edk2-devel] [Patch v4] BaseTools: Add Guid name support in GenFfs.

2020-07-13 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2762 The Fv Section in the FDF files use hard coding Guid values which is inconvenient to manage. This patch adds Guid name support in GenFfs to solve this problem. Signed-off-by: Yuwei Chen --- .../Source/Python/AutoGen/WorkspaceAutoGen.py

[edk2-devel] [Patch v5 1/1] BaseTools: Add Guid name support in GenFfs.

2020-07-13 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2762 The Fv Section in the FDF files use hard coding Guid values which is inconvenient to manage. This patch adds Guid name support in GenFfs to solve this problem. v5 modified the format without fuctional change. Signed-off-by: Yuwei Chen

Re: [edk2-devel] [PATCH] Using LLVM compiler set to build BaseTools in Linux

2020-07-14 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of > Zhiguang Liu > Sent: Monday, July 6, 2020 2:38 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > Subject: [edk2-devel] [PATCH] Using LLVM compiler set to build

Re: [edk2-devel] [PATCH] BaseTools/PatchCheck.py: Skip length check for user name in xxx-by

2020-07-16 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Zhang, > Shenglei > Sent: Thursday, July 16, 2020 11:14 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > Subject: [edk2-devel] [PATCH] BaseTools/PatchCheck.py: Sk

Re: [edk2-devel] [PATCH V2 1/2] BaseTools: Add gcc flag to warn on void* pointer arithmetic

2020-07-16 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of > PierreGondois > Sent: Tuesday, July 7, 2020 4:35 PM > To: devel@edk2.groups.io > Cc: Pierre Gondois ; Feng, Bob C > ; Gao, Liming ; > tomas.pi...@arm.com; n...@arm.com > S

[edk2-devel] [PATCH 1/1] Maintainers.txt: Add 'Yuwei Chen' for BaseTools review

2020-07-24 Thread Yuwei Chen
Add 'Yuwei Chen' as a reviewer for Edk2\BaseTools. Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- Maintainers.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Maintainers.txt b/Maintainers.txt index 599f94d57a3d..da09b0be1b96 100644 --- a/Maintainers

[edk2-devel] [PATCH] Maintainers.txt: Add 'Yuwei Chen' for Tools review

2020-07-24 Thread Yuwei Chen
Add 'Yuwei Chen' as a reviewer for Edk2-platforms\Platform\Intel\Tools and Edk2-platforms\Silicon\Intel\Tools. Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- Maintainers.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Maintainers.txt b/Maintainers.txt index

Re: [edk2-devel] [PATCH 1/2] BaseTools: fix ucs-2 lookup on python 3.9

2020-08-12 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Cole > Sent: Wednesday, August 12, 2020 1:28 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Cole Robinson > Subject: [edk2-devel] [PATCH 1/2] BaseTools: fix u

Re: [edk2-devel] [PATCH 2/2] BaseTools: Work around array.array.tostring() removal in python 3.9

2020-08-12 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Cole > Sent: Wednesday, August 12, 2020 1:28 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Cole Robinson > Subject: [edk2-devel] [PATCH 2/2] B

[edk2-devel] [PATCH 1/1] BaseTools: Warn user the file not found issue instead of break build.

2020-06-01 Thread Yuwei Chen
Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/Trim/Trim.py | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/Trim/Trim.py b/BaseTools/Source/Python/Trim/Trim.py index c5638376e41a..776474b20349 100644 --- a/BaseTools

Re: [edk2-devel] [PATCH v2 1/5] BaseTools:ECC needs to update the contents of CParser4

2020-06-04 Thread Yuwei Chen
Hi, Shenglei Since the input arguments number of the getText() function changed, the build failed. May be you should check about the getText() function's input. While how to choose the CParser4 and CParser3, I am not familiar with it, would you like to explain for me? Thanks, Yuwei > -Origi

Re: [edk2-devel] [PATCH v4] BaseTools/PatchCheck.py: Add LicenseCheck

2020-06-07 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Zhang, > Shenglei > Sent: Friday, June 5, 2020 5:22 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > Subject: [edk2-devel] [PATCH v4] BaseTools/PatchCheck.p

Re: [edk2-devel] [PATCH] BaseTools/Fmmt: Fix found the next FFS issue

2020-06-07 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Feng, > YunhuaX > Sent: Saturday, May 9, 2020 3:01 PM > To: devel@edk2.groups.io > Cc: Ni, Ray ; Gao, Liming ; Feng, > Bob C > Subject: [edk2-devel] [PATCH] BaseTools/Fmmt

Re: [edk2-devel] [Patch 01/15] BaseTools/Python: Allow HOST_APPLICATION to use NULL libraries

2020-06-14 Thread Yuwei Chen
Hi, Michael This patch looks good. While I have a little format suggestion: the Bugzilla link "https://bugzilla.tianocore.org/show_bug.cgi?id=2797"; could be written as "REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2797";. Reviewed-by: Yuwei Chen Thanks, Yuwei >

Re: [edk2-devel] [BaseTools PATCH v1] BaseTools: Add OpenSBI external definitions for RISC-V assembly build

2020-06-17 Thread Yuwei Chen
Hi, abner I have a little suggestion for the patch format, it would be better if the length of each commit line is less than 70 characters. You can follow the Commit Message Format in the wiki of tianocore. Thanks, Yuwei > -Original Message- > From: devel@edk2.groups.io On Behalf Of Ab

Re: [edk2-devel] [BaseTools PATCH v1] BaseTools: Add OpenSBI external definitions for RISC-V assembly build

2020-06-18 Thread Yuwei Chen
> ; Leif Lindholm > Subject: RE: [edk2-devel] [BaseTools PATCH v1] BaseTools: Add OpenSBI > external definitions for RISC-V assembly build > > Sure YuWei, v2 patch is sent. > > Thanks > Abner > > > -Original Message- > > From: devel@edk2.groups.io [mailto:d

Re: [edk2-devel] [PATCH v2 1/4] BaseTools: Generate multiple rules when multiple output files

2020-06-18 Thread Yuwei Chen
Hi, Pierre The patch looks good. Thanks, Yuwei > -Original Message- > From: devel@edk2.groups.io On Behalf Of > PierreGondois > Sent: Thursday, June 18, 2020 10:27 PM > To: devel@edk2.groups.io > Cc: Pierre Gondois ; sami.muja...@arm.com; > tomas.pi...@arm.com; Feng, Bob C ; Gao, Liming

Re: [edk2-devel] [PATCH] BaseTools/build.py: Exit with 1 when AutoGen error occurred

2020-06-28 Thread Yuwei Chen
Reivewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Irene > Park > Sent: Wednesday, June 3, 2020 5:59 AM > To: devel@edk2.groups.io > Cc: Irene Park > Subject: [edk2-devel] [PATCH] BaseTools/build.py: Exit with 1 when AutoGen >

[edk2-devel] [PATCH 1/1] BaseTools: Fix StructurePcd offset error.

2021-10-27 Thread Yuwei Chen
Currently when parse VFR files to get the struct info, BOOLEAN type struct is ignored, which caused offset errors. This patch fixed these bugs. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Scripts/ConvertFceToStructurePcd.py | 4 ++-- 1 file changed, 2 insertions(+), 2

[edk2-devel] RFC: Replace FMMT Tool from C to Python Version

2021-10-28 Thread Yuwei Chen
Hi everyone, I would like to propose the new FMMT Python tool, which is used to replace the origin C version tool. Currently the FMMT C tool is saved in edk2-staging repo, but its quality and coding style can't meet the Edk2 quality, which is hard to maintain (Hard/Duplicate Code; Regression bu

Re: [edk2-devel] [PATCH 1/1] BaseTools: Fix StructurePcd offset error.

2021-10-28 Thread Yuwei Chen
oups.io > Cc: Feng, Bob C > Subject: 回复: [PATCH 1/1] BaseTools: Fix StructurePcd offset error. > > Yuwei: > > > -邮件原件- > > 发件人: Yuwei Chen > > 发送时间: 2021年10月27日 18:00 > > 收件人: devel@edk2.groups.io > > 抄送: Bob Feng ; Liming Gao > > > >

Re: [edk2-devel] [Patch 1/1 V2] BaseTools: fix the split output files root dir

2021-02-04 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Monday, February 1, 2021 6:29 PM > To: devel@edk2.groups.io > Cc: Liming Gao ; Chen, Christine > > Subject: [Patch 1/1 V2] BaseTools: fix the split output files root dir > > If the out

Re: [edk2-devel] [PATCH v5 1/1] BaseTools: Use pip module if available, CI uses it by default

2021-02-08 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: matthewfcarl...@gmail.com > Sent: Tuesday, February 9, 2021 8:51 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Liming Gao > ; Chen, Christine ; > Matthew Carlson > Subject: [PATCH v5 1/1] BaseTools: Use pip

Re: [edk2-devel] [PATCH v2 1/1] BaseTools: Align include guards policy

2021-02-17 Thread Yuwei Chen
Hi Pierre, There seems already have a Bugzilla link for this issue: https://bugzilla.tianocore.org/show_bug.cgi?id=3094 And a personally concern: Some of the current codes still use "_***_H_", such as " __PEI_APRIORI_FILE_NAME_H__ ". If the ECC check only support the coding standard you mention

[edk2-devel] [PATCH 1/1] BaseTools: Modify struct parser for StructPcd

2021-03-02 Thread Yuwei Chen
Currently the struct parser for StructPcd Generation does not fliter the types such as UINT8 which should be ignored successfully. This patch modifies this issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Scripts/ConvertFceToStructurePcd.py | 2 ++ 1 file changed, 2

[edk2-devel] [PATCH 1/1] BaseTools/Ecc: Update structpcd parsing method.

2021-03-09 Thread Yuwei Chen
From: mliang2x Update the pcdparser method in Dec and DSC files. Signed-off-by: Mingyue Liang Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- .../Ecc/MetaFileWorkspace/MetaFileParser.py | 464 ++ 1 file changed, 265 insertions(+), 199 deletions(-) diff --git a/BaseTools

Re: [edk2-devel] [Patch] BaseTools: Fix the bug of --cmd-len build option

2021-12-27 Thread Yuwei Chen
This patch looks good to me. Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Friday, December 24, 2021 10:50 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > ; Chen, Christine > Subject: [Patch] BaseTools: Fix the bug of -

Re: [edk2-devel] [PATCH v2 6/6] BaseTools: Upgrade the version of NASM tool

2022-01-16 Thread Yuwei Chen
This patch looks good to me. Reviewed-by: Yuwei Chen > -Original Message- > From: Lou, Yun > Sent: Monday, January 10, 2022 11:13 PM > To: devel@edk2.groups.io > Cc: Lou, Yun ; Feng, Bob C ; > Gao, Liming ; Chen, Christine > > Subject: [PATCH v2 6/6] BaseTools

[edk2-devel] [PATCH] BaseTools: Add BaseTools/Scripts to path.

2022-02-27 Thread Yuwei Chen
s and Linux systems. Signed-off-by: Mingyue Liang Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- BaseTools/BuildEnv | 24 +--- BaseTools/toolsetup.bat | 6 ++ 2 files changed, 27 insertions(+), 3 deletions(-) diff --git a/BaseTools/BuildEnv b/BaseTools/BuildEnv

[edk2-devel] [PATCH] BaseTools: Catch the exception in build.py.

2022-03-01 Thread Yuwei Chen
-defined exception class should be imported. Signed-off-by: Mingyue Liang Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- BaseTools/Source/Python/build/build.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Python/build/build.py

Re: [edk2-devel] [Patch] BaseTools: Remove the redundant __FLEXIBLE_SIZE from PcdValueInit.c

2022-03-20 Thread Yuwei Chen
This patch reduces compilation time , it looks good to me. Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Tuesday, March 15, 2022 8:17 PM > To: devel@edk2.groups.io > Cc: Gao, Liming ; Chen, Christine > > Subject: [Patch] BaseTools:

Re: [edk2-devel] [PATCH v2 1/1] Basetools: fix gcc workaround

2022-03-30 Thread Yuwei Chen
Hi Hoffmann, I received some reports that some platforms are using gcc5.4. With this patch, these platform builds are still broken. Would you like to revert the original commits? Thanks, Christine (Yuwei) > -Original Message- > From: Gerd Hoffmann > Sent: Tuesday, March 29, 2022 4:38 P

Re: [edk2-devel] [PATCH v2 1/1] Basetools: fix gcc workaround

2022-03-30 Thread Yuwei Chen
hristine (Yuwei) > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Thursday, March 31, 2022 8:51 AM > To: Gerd Hoffmann ; devel@edk2.groups.io > Cc: Feng, Bob C ; Rebecca Cran > ; Pawel Polawski ; Oliver > Steffen ; Gao, Liming &g

[edk2-devel] [PATCH 1/1] BaseTools/Ecc: Update structpcd parsing method.

2021-04-01 Thread Yuwei Chen
From: mliang2x Update the pcdparser method in Dec and DSC files. Signed-off-by: Mingyue Liang Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- .../Ecc/MetaFileWorkspace/MetaFileParser.py | 464 ++ 1 file changed, 265 insertions(+), 199 deletions(-) diff --git a/BaseTools

Re: [edk2-devel] [PATCH 1/1] BaseTools/Ecc: Update structpcd parsing method.

2021-04-01 Thread Yuwei Chen
Please ignore this patch, sorry for wrong sending. Thanks, Yuwei > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Thursday, April 1, 2021 3:04 PM > To: devel@edk2.groups.io > Cc: Liang, MingyueX ; Feng, Bob C > ; Liming Gao &

[edk2-devel] [PATCH 1/1] Basetools: Keep StdErr in screen when StdOut in file

2021-04-01 Thread Yuwei Chen
Currently, when using GCC toolchain and loging the StdOut into files, the StdErr will also be logged into files without shown in screen. This patch fixes this issue. REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3278 Cc: Bob Feng Cc: Liming Gao Cc: Wenyi Xie Signed-off-by: Yuwei Chen

Re: [EXTERNAL] [edk2-devel] [PATCH 1/1] BaseTools/Ecc: Update structpcd parsing method.

2021-04-07 Thread Yuwei Chen
; Feng, Bob C ; Liming Gao Subject: RE: [EXTERNAL] [edk2-devel] [PATCH 1/1] BaseTools/Ecc: Update structpcd parsing method. What does "update" mean in this context? What behavior is changing? - Bret From: Yuwei Chen via groups.io<mailto:yuwei.chen=intel@groups.io> Sent: T

Re: [edk2-devel] [Patch 1/1] BaseTools/Conf: Fix MAKE_FLAGS typos in tools_def.template

2021-04-11 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Kinney, Michael D > Sent: Saturday, April 10, 2021 6:00 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Liming Gao > ; Chen, Christine > Subject: [Patch 1/1] BaseTools/Conf: Fix MAKE_FLAGS typos in > too

Re: [edk2-devel] [Patch 1/1] BaseTools/PlatformAutoGen: MAKE_FLAGS and MAKE_PATH fixes

2021-04-11 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Kinney, Michael D > Sent: Saturday, April 10, 2021 5:49 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Liming Gao > ; Chen, Christine > Subject: [Patch 1/1] BaseTools/PlatformAutoGen: MAKE_FLAGS and >

Re: [edk2-devel] [Patch 1/1] BaseTools/Conf: Fix MAKE_FLAGS typos in tools_def.template

2021-04-13 Thread Yuwei Chen
l Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Monday, April 12, 2021 9:48 AM > To: Kinney, Michael D ; devel@edk2.groups.io > Cc: Feng, Bob C ; Liming Gao > > Subject: Re: [edk2-devel] [Patch 1/1] BaseTools/Conf: Fix MAKE_FLAGS typos > in t

Re: [edk2-devel] [Patch 1/1] BaseTools/GenMake: Sort generated makefile tool definitions

2021-04-25 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Kinney, Michael D > Sent: Monday, April 26, 2021 8:50 AM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Liming Gao > ; Chen, Christine > Subject: [Patch 1/1] BaseTools/GenMake: Sort generated makefile tool &

[edk2-devel] [PATCH] BaseTools: Change non-ascii character of StructurePcd comment

2021-04-28 Thread Yuwei Chen
Currently, the ConvertFceToStructurePcd.py tool generate StructurePcd dsc file with comments including non-ascii character circle R. This patch changes the non-ascii character circle R to (R) when adding the comment. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Scripts

Re: [edk2-devel] [PATCH] BaseTools: Change non-ascii character of StructurePcd comment

2021-04-28 Thread Yuwei Chen
ls. > > Mike > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of Yuwei > > Chen > > Sent: Wednesday, April 28, 2021 1:45 AM > > To: devel@edk2.groups.io > > Cc: Feng, Bob C ; Liming Gao > > > > Subject: [edk2-deve

Re: [edk2-devel] [PATCH] BaseTools: Change non-ascii character of StructurePcd comment

2021-04-28 Thread Yuwei Chen
river UNI file. It may > have non-ascii character. > > Thanks > Liming > > -----邮件原件- > > 发件人: devel@edk2.groups.io 代表 Yuwei Chen > > 发送时间: 2021年4月29日 8:25 > > 收件人: Kinney, Michael D ; > > devel@edk2.groups.io > > 抄送: Feng, Bob C ; Liming G

[edk2-devel] [PATCH 1/1] BaseTools/Brotli: Fix compressed data loss issue

2021-05-13 Thread Yuwei Chen
called again until both input stream is depleted and encoder has no more output after the method is called. This patch fixes this issue based on the Brotli document. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- .../Source/C/BrotliCompress/BrotliCompress.c | 61 --- 1

Re: [edk2-devel] [PATCH 3/3] BaseTools: Drop check for distutils.utils

2021-07-25 Thread Yuwei Chen
This patch looks good to me. Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Cole > Sent: Saturday, July 24, 2021 4:02 AM > To: devel@edk2.groups.io > Cc: Cole Robinson > Subject: [edk2-devel] [PATCH 3/3] BaseTools: Drop check f

Re: [edk2-devel] [Patch] BaseTools: use shutil.copyfile instead shutil.copy2

2021-08-01 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Wednesday, July 28, 2021 7:45 PM > To: devel@edk2.groups.io > Cc: Liming Gao ; Chen, Christine > > Subject: [Patch] BaseTools: use shutil.copyfile instead shutil.copy2 > > In Split tool,

Re: [edk2-devel] [PATCH 0/3] BaseTools: fix some python DeprecationWarnings

2021-08-10 Thread Yuwei Chen
Hi Robinson, When we doing the internal test, the issue is found that py27 is blocked by this patch. You can reproduce the error with below steps: For Linux: [Error Reproduce steps] $ export PYTHON3_ENABLE=FALSE $ export PYTHON_COMMAND=/usr/bin/python2.7 $ build -p OvmfPkg/OvmfPkgIa32X64.dsc -

[edk2-devel] [PATCH 1/1] BaseTools/GenFds: Change FV Header to FileSystemGuid3.

2021-08-10 Thread Yuwei Chen
EFI_FFS_FILE_HEADER2, the Fv FileSystemGuid does not changed consistent with Ffs type.That caused build issue. This patch fixes this issue. As FileSystemGuid3 is compatible with FileSystemGuid2, change all the Fv header to FileSystemGuid3. Signed-off-by: Yuwei Chen Cc: Bob Feng Cc: Liming Gao --- BaseTools

[edk2-devel] [PATCH 1/1] BaseTools/GenFds: Change FV Header to FileSystemGuid3, edk2-stable202108

2021-08-16 Thread Yuwei Chen
: Yuwei Chen Cc: Bob Feng Cc: Liming Gao Cc: Andrew Fish Cc: Leif Lindholm Cc: Michael D Kinney --- BaseTools/Source/C/GenFv/GenFv.c | 2 +- BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a

Re: [edk2-devel] [Patch] [edk2-staging] BaseTools: Add checkpoint for that there is no fv ext_header

2021-05-26 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Feng, Bob C > Sent: Tuesday, May 25, 2021 2:20 PM > To: devel@edk2.groups.io > Cc: Liming Gao ; Chen, Christine > > Subject: [Patch] [edk2-staging] BaseTools: Add checkpoint for that there is > no fv ext_header

[edk2-devel] [Edk2-platforms][PATCH] Tools/Fsp: Replace PYTHON_HOME with sys.executable

2021-05-30 Thread Yuwei Chen
From: Mingyue Liang Currently script Python interpreter is inconsistent with the interpreter specified by Python home. This patch is to change pythonhome to sys.executable. Signed-off-by: MingYue Liang Cc: Yuwei Chen Cc: Bob Feng Cc: Liming Gao --- .../Tools/Fsp

[edk2-devel] [PATCH 1/1] BaseTools: Remove non-ascii character of StructurePcd comment

2021-06-28 Thread Yuwei Chen
circle R to (R). Signed-off-by: Yuwei Chen Cc: Bob Feng Cc: Liming Gao --- BaseTools/Scripts/ConvertFceToStructurePcd.py | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/BaseTools/Scripts/ConvertFceToStructurePcd.py b/BaseTools/Scripts/ConvertFceToStructurePcd.py

Re: [edk2-devel] [PATCH] Platform/Intel: Enhance the BOARD_ID without PCH_TYPE support

2023-08-29 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Srinivasan, ManickamX > Sent: Wednesday, August 23, 2023 3:29 PM > To: devel@edk2.groups.io > Cc: Srinivasan, ManickamX ; Feng, Bob C > ; Gao, Liming ; Chen, > Christine ; T V, Krishnamoorthy > > Subj

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-09-10 Thread Yuwei Chen
From: Chen, Christine > > Sent: Thursday, June 29, 2023 11:34 AM > > To: devel@edk2.groups.io > > Cc: Rebecca Cran ; Gao, Liming > > ; Feng, Bob C > > Subject: [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue > > > > 1. FvLength not change issue; > >

Re: [edk2-devel] [PATCH] Platform/Intel: Enhance the BOARD_ID without PCH_TYPE support

2023-09-15 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: Srinivasan, ManickamX > Sent: Wednesday, September 13, 2023 4:56 PM > To: devel@edk2.groups.io > Cc: Srinivasan, ManickamX ; Feng, Bob C > ; Gao, Liming ; Chen, > Christine ; T V, Krishnamoorthy > > Subj

[edk2-devel] [PATCH 1/1] BaseTools: trim warning to error

2023-09-21 Thread Yuwei Chen
As the error is changed to warning, Trim.py will skip the build error when the source code have exactly issue. This patch change warning to error to opens the checking. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/Trim/Trim.py | 4 ++-- 1

Re: [edk2-devel] [PATCH] BaseTools: Fix build option overrides Pcd Feature Flag issue

2022-10-28 Thread Yuwei Chen
Cc Mike. Thanks, Christine > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Tuesday, October 25, 2022 4:51 PM > To: devel@edk2.groups.io > Cc: Feng, Bob C ; Gao, Liming > > Subject: [edk2-devel] [PATCH] BaseTools: Fix bu

[edk2-devel] [Patch V1 2/3] [edk2-staging]BaseTools: Add new build option for PyVfrCompiler yaml generation

2022-11-14 Thread Yuwei Chen
Add new build option "--vfr-yaml-enable" for Python VfrCompiler extended output yaml file generation. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source/Python/AutoGen/ModuleAutoGen.py | 8

[edk2-devel] [Patch V1 3/3] [edk2-staging]BaseTools: Add new build option for Variable default value generation

2022-11-14 Thread Yuwei Chen
Add new build option "--gen-default-variable-bin" for Variable default value generation from Python VFR tool's extended json output file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source

[edk2-devel] [Patch V2 2/3] [edk2-staging]BaseTools: Add new build option for PyVfrCompiler yaml generation

2022-11-15 Thread Yuwei Chen
Add new build option "--vfr-yaml-enable" for Python VfrCompiler extended output yaml file generation. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source/Python/AutoGen/ModuleAutoGen.py | 8

[edk2-devel] [Patch V2 3/3] [edk2-staging]BaseTools: Add new build option for Variable default value generation

2022-11-15 Thread Yuwei Chen
Add new build option "--gen-default-variable-bin" for Variable default value generation from Python VFR tool's extended json output file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/AutoGen/DataPipe.py | 2 ++ BaseTools/Source

[edk2-devel] [Patch V1 2/3] BaseTools: Add new build option for PyVfrCompiler yaml generation

2022-11-15 Thread Yuwei Chen
Add new build option "--vfr-yaml-enable" for Python VfrCompiler extended output yaml file generation. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/AutoGen/DataPipe.py | 2 ++ edk2basetools/AutoGen/ModuleAutoGen.py | 8 edk2baseto

[edk2-devel] [Patch V1 3/3] BaseTools: Add new build option for Variable default value generation

2022-11-15 Thread Yuwei Chen
Add new build option "--gen-default-variable-bin" for Variable default value generation from Python VFR tool's extended json output file. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- edk2basetools/AutoGen/DataPipe.py | 2 ++ edk2basetools/AutoGen/G

[edk2-devel] [Patch V1 2/3] BaseTools: FMMT replace output file is not generated successfully

2023-04-18 Thread Yuwei Chen
For replace function, when target Ffs and new ffs are with same size, the output file can not be generated successfully. This patch fixes this issue. Cc: Rebecca Cran Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/core/BiosTree.py | 4 ++-- BaseTools

[edk2-devel] [Patch V1 3/3] BaseTools: FMMT support ELF UPLD parser

2023-04-18 Thread Yuwei Chen
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103156): https://edk2.groups.io/g/devel/message/103156 Mute This Topic: https://groups.io/mt/98340352/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/deve

[edk2-devel] [Patch V1 3/3] BaseTools: FMMT support ELF UPLD parser

2023-04-18 Thread Yuwei Chen
- ImageId UPLD Buffer ''' Cc: Rebecca Cran Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/FMMT/FMMT.py | 2 ++ BaseTools/Source/Python/FMMT/core/BinaryFactoryProduct.py | 36 +++- BaseT

Re: [edk2-devel] [Patch V1 1/3] BaseTools: fixing FMMT ShrinkFv issue

2023-04-18 Thread Yuwei Chen
For edk2-basetools repo, have made a PR for these three patches: https://github.com/tianocore/edk2-basetools/pull/95 Thanks, Christine (Yuwei) > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yuwei > Chen > Sent: Tuesday, April 18, 2023 5:53 PM > To: devel

Re: [edk2-devel] [Patch V2] pip-requirements.txt: Update basetools version to 0.1.24

2022-07-03 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Bob > Feng > Sent: Thursday, June 30, 2022 12:11 PM > To: devel@edk2.groups.io > Cc: Kinney, Michael D ; Kubacki, Michael > > Subject: [edk2-devel] [Patch V2] pip-requiremen

Re: [edk2-devel] [PATCH v3 1/3] [edk2-platforms] Silicon/Intel/FitGen: Support multiple Startup ACM Type 2 entries in FitGen tool

2022-07-03 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of Lin, > Jason1 > Sent: Friday, July 1, 2022 11:10 PM > To: devel@edk2.groups.io > Cc: Lin, Jason1 ; Feng, Bob C ; > Gao, Liming ; Chen, Christine > ; Oram, Isaac W ; > Ch

  1   2   >