Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BaseTools: Add comments in StructurePcd DSC file. #1225

Closed
wants to merge 1 commit into from

Conversation

BobCF
Copy link
Member

@BobCF BobCF commented Dec 15, 2020

Currently VFR files have variables comments which will not be
added into StructurePcd.dsc file. Thus, it is not convenient for
developer to Modify Pcds. To solve this problem, The comments will
be modified to user friendly format and added after the corresponding
Pcd values in StructurePcd.dsc file.

Cc: Bob Feng bob.c.feng@intel.com
Cc: Liming Gao gaoliming@byosoft.com.cn
Signed-off-by: Yuwei Chen yuwei.chen@intel.com

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

Currently VFR files have variables comments which will not be
added into StructurePcd.dsc file. Thus, it is not convenient for
developer to Modify Pcds. To solve this problem, The comments will
be modified to user friendly format and added after the corresponding
Pcd values in StructurePcd.dsc file.

Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Yuwei Chen <yuwei.chen@intel.com>

Reviewed-by: Bob Feng <bob.c.feng@intel.com>
@BobCF BobCF added push Auto push patch series in PR if all checks pass and removed push Auto push patch series in PR if all checks pass labels Dec 15, 2020
@mergify
Copy link

mergify bot commented Dec 15, 2020

PR can not be merged due to an Ubuntu GCC5 failure. Please resolve and resubmit

@mergify
Copy link

mergify bot commented Dec 15, 2020

PR can not be merged due to a Windows VS2019 failure. Please resolve and resubmit

@BobCF BobCF closed this Dec 15, 2020
@BobCF BobCF deleted the pr1215 branch April 25, 2022 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants