tianocore / edk2-basetools

EDK II BaseTools Python tools as a PIP module
Other
25 stars 35 forks source link

Remove the BUILD_CFLAGS compatibility code in Workspace/DscBuildData.py #97

Closed ghost closed 1 year ago

ghost commented 1 year ago

Since everyone should now be running a version of edk2 that expects CFLAGS instead of BUILD_CFLAGS, remove the compatibility code.

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (59c95d7) 4.28% compared to head (1a7953f) 4.28%.

:exclamation: Current head 1a7953f differs from pull request most recent head 8ee9a54. Consider uploading reports for the commit 8ee9a54 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #97 +/- ## ====================================== Coverage 4.28% 4.28% ====================================== Files 181 181 Lines 81538 81533 -5 ====================================== Hits 3490 3490 + Misses 78048 78043 -5 ``` | Flag | Coverage Δ | | |---|---|---| | Linux | `4.28% <ø> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/tianocore/edk2-basetools/pull/97?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore) | Coverage Δ | | |---|---|---| | [edk2basetools/Workspace/DscBuildData.py](https://codecov.io/gh/tianocore/edk2-basetools/pull/97?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMmJhc2V0b29scy9Xb3Jrc3BhY2UvRHNjQnVpbGREYXRhLnB5) | `0.00% <ø> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.