Reviewed-by: Jaben Carsey <[email protected]>
> -----Original Message----- > From: edk2-devel [mailto:[email protected]] On Behalf Of > Yonghong Zhu > Sent: Friday, October 19, 2018 12:27 AM > To: [email protected] > Subject: [edk2] [Patch] BaseTools: Fix the bug that PcdValueFromComm is > not set > > the bug is PcdValueFromComm is not set, but the Pcd have been override > by the command line option. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Yonghong Zhu <[email protected]> > --- > BaseTools/Source/Python/Workspace/DscBuildData.py | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/BaseTools/Source/Python/Workspace/DscBuildData.py > b/BaseTools/Source/Python/Workspace/DscBuildData.py > index e481ea4..17e6f62 100644 > --- a/BaseTools/Source/Python/Workspace/DscBuildData.py > +++ b/BaseTools/Source/Python/Workspace/DscBuildData.py > @@ -1065,10 +1065,11 @@ class DscBuildData(PlatformBuildClassObject): > continue > for key in BuildData.Pcds: > PcdItem = BuildData.Pcds[key] > if (TokenSpaceGuidCName, TokenCName) == > (PcdItem.TokenSpaceGuidCName, PcdItem.TokenCName) and FieldName > =="": > PcdItem.DefaultValue = pcdvalue > + PcdItem.PcdValueFromComm = pcdvalue > #In command line, the latter full assign value in commandLine should > override the former field assign value. > #For example, --pcd Token.pcd.field="" --pcd Token.pcd=H"{}" > delete_assign = [] > field_assign = {} > if GlobalData.BuildOptionPcd: > -- > 2.6.1.windows.1 > > _______________________________________________ > edk2-devel mailing list > [email protected] > https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
