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: Correct report the line number in INF file issue #1249

Merged
merged 1 commit into from
Dec 22, 2020

Conversation

lgao4
Copy link
Contributor

@lgao4 lgao4 commented Dec 21, 2020

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

If one protocol is not found in package dec file, Build will report the
error message that module.inf(-1): error 4000: Value of Protocol
[gXxxProtocolGuid] is not found under [Protocols] section in MdePkg.dec.

Here, -1 should be the line number of this protocol in module INF file.

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

@lgao4 lgao4 added the push Auto push patch series in PR if all checks pass label Dec 21, 2020
@mergify
Copy link

mergify bot commented Dec 21, 2020

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

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

If one protocol is not found in package dec file, Build will report the
error message that module.inf(-1): error 4000: Value of Protocol
[gXxxProtocolGuid] is not found under [Protocols] section in MdePkg.dec.

Here, -1 should be the line number of this protocol in module INF file.

Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Yuwei Chen <yuwei.chen@intel.com>
Signed-off-by: Yunhua Feng <fengyunhua@byosoft.com.cn>
Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
Reviewed-by: Bob Feng <bob.c.feng@intel.com>
@mergify mergify bot merged commit b6e3608 into tianocore:master Dec 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
push Auto push patch series in PR if all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants