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

[Laszlo Ersek]

On 2/28/24 00:41, Rebecca Cran wrote:
> edk2-basetools v0.1.50 has just been published so we're back up and
> running.
> 
> I'll work my way through the backlog of BaseTools changes in the next
> few days.

Thanks! Once you merge
 and make a new
release that I can refer to from edk2's "pip-requirements.txt", I'm
ready to submit the reworked edk2 series to CI.

Laszlo



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#116291): https://edk2.groups.io/g/devel/message/116291
Mute This Topic: https://groups.io/mt/104553597/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

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

[Laszlo Ersek]

Hi Mike,

can you please comment on the Build and Inf spec changes proposed in
this thread?

Thanks,
Laszlo

On 2/24/24 21:59, Laszlo Ersek wrote:
> v1 posting:
> 
>   https://edk2.groups.io/g/devel/message/115193
>   msgid <36593e23-d3e8-b71a-808d-ef94260b5...@redhat.com>
> 
> Bugzilla:
> 
>   https://bugzilla.tianocore.org/show_bug.cgi?id=991
> 
> In version 2, the feature is structured differently. Following Mike's
> advice, for compatibility, the ProcessLibraryConstructorList()
> declaration in AutoGen.h is now gated on the SEC module having
> INF_VERSION >= 1.30.
> 
> Accordingly,
> 
> - I now update the Build specification and the Inf specification (see
>   patch sets posted in response to this email),
> 
> - edk2 only receives a single patch (for AutoGen), for the time being,
> 
> - the same edk2 patch is being ported to edk2-basetools:
>   https://github.com/tianocore/edk2-basetools/pull/120.
> 
> Next steps: once all of the above is merged, *and* an edk2-basetools
> release has been tagged and published, I'll rework the C code patches
> for edk2 and edk2-platforms, from the v1 patch sets, as follows:
> 
> - all those SEC modules will have to see their INF_VERSIONs bumped to
>   1.30, for triggering the new code generation,
> 
> - pip-requirements.txt/edk2-basetools will need to reference the new
>   edk2-basetools release, for exposing the feature in the first place.
> 
> Laszlo
> 
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#116234): https://edk2.groups.io/g/devel/message/116234
Mute This Topic: https://groups.io/mt/104553597/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

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

[Rebecca Cran]

edk2-basetools v0.1.50 has just been published so we're back up and running.

I'll work my way through the backlog of BaseTools changes in the next 
few days.



--
Rebecca Cran


On 2/24/24 13:59, Laszlo Ersek wrote:

v1 posting:

   https://edk2.groups.io/g/devel/message/115193
   msgid <36593e23-d3e8-b71a-808d-ef94260b5...@redhat.com>

Bugzilla:

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

In version 2, the feature is structured differently. Following Mike's
advice, for compatibility, the ProcessLibraryConstructorList()
declaration in AutoGen.h is now gated on the SEC module having
INF_VERSION >= 1.30.

Accordingly,

- I now update the Build specification and the Inf specification (see
   patch sets posted in response to this email),

- edk2 only receives a single patch (for AutoGen), for the time being,

- the same edk2 patch is being ported to edk2-basetools:
   https://github.com/tianocore/edk2-basetools/pull/120.

Next steps: once all of the above is merged, *and* an edk2-basetools
release has been tagged and published, I'll rework the C code patches
for edk2 and edk2-platforms, from the v1 patch sets, as follows:

- all those SEC modules will have to see their INF_VERSIONs bumped to
   1.30, for triggering the new code generation,

- pip-requirements.txt/edk2-basetools will need to reference the new
   edk2-basetools release, for exposing the feature in the first place.

Laszlo




-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#116063): https://edk2.groups.io/g/devel/message/116063
Mute This Topic: https://groups.io/mt/104553597/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

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

[Laszlo Ersek]

v1 posting:

  https://edk2.groups.io/g/devel/message/115193
  msgid <36593e23-d3e8-b71a-808d-ef94260b5...@redhat.com>

Bugzilla:

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

In version 2, the feature is structured differently. Following Mike's
advice, for compatibility, the ProcessLibraryConstructorList()
declaration in AutoGen.h is now gated on the SEC module having
INF_VERSION >= 1.30.

Accordingly,

- I now update the Build specification and the Inf specification (see
  patch sets posted in response to this email),

- edk2 only receives a single patch (for AutoGen), for the time being,

- the same edk2 patch is being ported to edk2-basetools:
  https://github.com/tianocore/edk2-basetools/pull/120.

Next steps: once all of the above is merged, *and* an edk2-basetools
release has been tagged and published, I'll rework the C code patches
for edk2 and edk2-platforms, from the v1 patch sets, as follows:

- all those SEC modules will have to see their INF_VERSIONs bumped to
  1.30, for triggering the new code generation,

- pip-requirements.txt/edk2-basetools will need to reference the new
  edk2-basetools release, for exposing the feature in the first place.

Laszlo



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#115909): https://edk2.groups.io/g/devel/message/115909
Mute This Topic: https://groups.io/mt/104553597/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-