Skip to content

BaseTools/Build: Output warning mesage for library class mismatch#12875

Open
apop5 wants to merge 1 commit into
tianocore:masterfrom
apop5:personal/apop5/librarymismatch
Open

BaseTools/Build: Output warning mesage for library class mismatch#12875
apop5 wants to merge 1 commit into
tianocore:masterfrom
apop5:personal/apop5/librarymismatch

Conversation

@apop5

@apop5 apop5 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Description

Performs a check that will verify that the library instance implements the library specified in the dsc by ensuring a LIBRARY_CLASS definition exists in the INF [Defines] section and the value matches the library it says it is implementing.

As an example, from a platform dsc file:
BaseBmpSupportLib|MdeModulePkg/Library/BaseBmpSupportLib/BaseBmpSupportLib.inf

BaseBmpSupportLib is supposed to be of library class BmpSupportLib, but the dsc defines it incorrectly, so the warning message will be displayed during the build:

MdeModulePkg\Library\BaseBmpSupportLib\BaseBmpSupportLib.inf does not support LIBRARY_CLASS BaseBmpSupportLib

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

How This Was Tested

Testing on platform code. Detection of incorrect library classes.
No build breaks, just warning messages.

Integration Instructions

No Integration necessary

Performs a check that will verify that the library instance implements
the library specified in the dsc by ensuring a LIBRARY_CLASS definition
exists in the INF [Defines] section and the value matches the library it
says it is implementing.

As an example, from a platform dsc file:
BaseBmpSupportLib|MdeModulePkg/Library/BaseBmpSupportLib/BaseBmpSupportLib.inf

BaseBmpSupportLib is supposed to be of library class BmpSupportLib, but the
dsc defines it incorrectly, the warning message will be displayed during
the build:

MdeModulePkg\Library\BaseBmpSupportLib\BaseBmpSupportLib.inf
does not support LIBRARY_CLASS BaseBmpSupportLib

Signed-off-by: Aaron Pop <aaronpop@microsoft.com>
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.

2 participants