Re: [edk2-devel] BaseTools/AutoGen: declare ProcessLibraryConstructorList() for SEC modules

["Laszlo Ersek" <lersek@redhat.com>]

On 2/8/24 17:40, Michael D Kinney wrote:
> Hi Laszlo,
> 
> I need to review the proposed BaseTools/AutoGen change to see what options
> are available for compatibility.
> 
> My main concern is downstream consumers that may break immediately with
> a change like this and we need a way for them to be informed and have
> time to update their components just like you outline a sequence to update
> the edk2 repo components.

Should AutoGen declare ProcessLibraryConstructorList() for a SEC module
if INF_VERSION >= 1.30?

Or should we introduce a new macro in [Defines]?

https://tianocore-docs.github.io/edk2-InfSpecification/draft/2_inf_overview/24_[defines]_section.html

"EDK II parsing utilities will use some of this section's information
for generating AutoGen.c and AutoGen.h files."

I'd prefer (INF_VERSION >= 1.30) over a dedicated macro. We should
ensure, over time, that ProcessLibraryConstructorList() is declared by
default, for SEC modules. If that declaration depended on an explicit
new macro in [Defines], it would much less likely become the default.

Laszlo



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#115491): https://edk2.groups.io/g/devel/message/115491
Mute This Topic: https://groups.io/mt/104210524/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-