[edk2-devel] [PATCH edk2-platforms 14/15] Platform/SocioNext: add VariablePolicy resolutions

[Ard Biesheuvel]

On 11/25/20 2:16 PM, Leif Lindholm wrote:
> On Wed, Nov 25, 2020 at 14:11:00 +0100, Ard Biesheuvel wrote:
>> On 11/25/20 1:55 PM, Leif Lindholm wrote:
>>> Add resolutions for
>>> - VariablePolicyLib
>>> - VariablePolicyHelperLib
>>> - VariablePolicyLib (DXE_RUNTIME_DRIVER)
>>> to DeveloperBox platform.
>>>
>>> Cc: Ard Biesheuvel <ard.biesheuvel at arm.com>
>>> Signed-off-by: Leif Lindholm <leif at nuviainc.com>
>>
>> Does this unbreak the build of DeveloperBoxMm.dsc as well?
> 
> No. I was unsure of that being pretty much our reference for secure
> variable handling, just cargo culting a fix in place was the correct
> course of action.
> 
>> If not, we should
>> probably drop some of these into the .dsc.in file instead.
> 
> Unless the above is an issue, sure.
> 

I double checked, and Platform/Socionext/DeveloperBox/DeveloperBoxMm.dsc 
is indeed broken as well, and it needs resolutions of both 
VariablePolicyLib and VariablePolicyHelperLib.

So adding those to [LibraryClasses] in 
Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc instead of 
elsewhere is probably the best approach here.

With that,

Acked-by: Ard Biesheuvel <ard.biesheuvel at arm.com>




-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#67972): https://edk2.groups.io/g/devel/message/67972
Mute This Topic: https://groups.io/mt/78498673/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-