Date 1 - 3 of 3
feature flag change notifications
Thanks for the consideration. I think the release notes are a
sufficient starting point.
On Mon, Dec 5, 2022 at 6:22 PM Kinney, Michael D
Michael D Kinney
Hi Dann,toggle quoted message Show quoted text
These are reasonable requests.
Adding Liming Gao who is currently responsible for the edk2 releases.
Let's start with release notes and see if some of these can be added
to the release documentation and add to the edk2 release process
to clearly note these type of changes in future release notes.
As far as #error and #warn. Those are possible, but we would need
to do some analysis for the types of changes that would require
that behavior. Your examples are useful as a starting point.
Once concern is how many of those #error/#warn conditions will
build up over time and then a process to remove them after an
I maintain the edk2 packages in the Debian and Ubuntu
distributions. A few times over the years I've had users report
regressions that turned out to be due to upstream build flag
changes. I wonder if it would be possible for upstream to communicate
such changes - either via release notes or, where possible, as
build-time checks - to us downstream projects. Here are the examples I
commit 4de8d61bcec02a13ceed84f92b0cf3ea58adf9c5For this I needed to change -DTPM_ENABLE=TRUE to -DTPM2_ENABLE=TRUE. I
feel like a build-time check would've been a good way to communicate
this one, e.g.:
#error TPM_ENABLE has been renamed to TPM2_ENABLE
commit 1631bb26ae991e530d3c96fe3161ea15144b358eSimilar to the above, I needed to deal with this by setting
-DNETWORK_HTTP_BOOT_ENABLE=TRUE instead of -DHTTP_BOOT_ENABLE=TRUE. It
seems like maybe an #error on the removed #define could've helped
here as well.
commit 57783adfb579da32b1eeda77b2bec028a5e0b7b3With this types of change, it would be useful just to know this is
coming and why so I can decide to either notify users of the change
and/or override the default to avoid the regression. Would it be
possible to describe such changes in the release notes at
https://github.com/tianocore/edk2/releases - and, if not urgent,
perhaps #warn of deprecation in the source for one stable release
|1 - 3 of 3|