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

MdeModulePkg HobPrintLib: Add Guid to Guids section. #6406

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

apop5
Copy link
Contributor

@apop5 apop5 commented Nov 5, 2024

Description

gEfiHobMemoryAllocModuleGuid is referenced in the HobPrintLib, but it is not defined in the INF file, causing an unresolved external error when the module is consumed by code.

  • Breaking change?
  • Impacts security?
  • Includes tests?

How This Was Tested

Modified module to consume this library, experienced build error.
Corrected INF, module compiled correctly.

Integration Instructions

N/A

@niruiyu niruiyu added the push Auto push patch series in PR if all checks pass label Nov 6, 2024
@lgao4 lgao4 force-pushed the personal/apop5/addguidtohobprint branch from e897bc8 to e34ced9 Compare November 11, 2024 01:13
@lgao4 lgao4 removed the push Auto push patch series in PR if all checks pass label Nov 11, 2024
@mdkinney
Copy link
Member

@lgao4 is this for edk2-stable202411?

@apop5
Copy link
Contributor Author

apop5 commented Nov 19, 2024

This is a bug fix for declaring the guids consumed by the library. If it is possible. Without it, the library cannot be consumed by any module, unless the module's INF declares gEfiHobMemoryAllocModuleGuid, or some other library the module consumes declares gEfiHobMemoryAllocModuleGuid.

@lgao4 lgao4 added the push Auto push patch series in PR if all checks pass label Nov 22, 2024
gEfiHobMemoryAllocModuleGuid is referenced in the HobPrintLib,
but it is not defined in the INF file, causing an unresolved
external error when the module is consumed by code.

Signed-off-by: Aaron Pop <[email protected]>
@lgao4 lgao4 force-pushed the personal/apop5/addguidtohobprint branch from e34ced9 to 18f8f8a Compare November 22, 2024 15:37
@mergify mergify bot merged commit f3bc601 into tianocore:master Nov 22, 2024
126 checks passed
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.

4 participants