Skip to content

Fix noopt builds#248

Draft
Flickdm wants to merge 2 commits into
microsoft:archive/old_mainfrom
Flickdm:fix/archived_main/noopt-builds-broken-for-consumers
Draft

Fix noopt builds#248
Flickdm wants to merge 2 commits into
microsoft:archive/old_mainfrom
Flickdm:fix/archived_main/noopt-builds-broken-for-consumers

Conversation

@Flickdm
Copy link
Copy Markdown
Member

@Flickdm Flickdm commented May 4, 2026

Description

Maps TARGET to DEBUG when the TARGET is not set to RELEASE

For details on how to complete these options and their meaning refer to CONTRIBUTING.md.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

TODO

Integration Instructions

TODO

Flickdm added 2 commits May 4, 2026 16:20
Only DEBUG and RELEASE crypto binaries are published, but the
generated DSC and FDF include files used $(TARGET) directly. When
building with TARGET=NOOPT, the DSC paths resolved to nonexistent
.inf files and the FDF conditionals matched neither DEBUG nor
RELEASE, excluding all crypto binaries.

Introduce a CRYPTO_TARGET define that maps $(TARGET) to RELEASE
when TARGET is RELEASE, and to DEBUG otherwise. Replace all
$(TARGET) usage in generated .inf paths (DSC) and binary selection
conditionals (FDF) with $(CRYPTO_TARGET).

Signed-off-by: Doug Flick <dougflick@microsoft.com>
Regenerate all autogenerated DSC, FDF, and INF files using
generate_cryptodriver.py to pick up the CRYPTO_TARGET changes.

Signed-off-by: Doug Flick <dougflick@microsoft.com>
@Flickdm Flickdm linked an issue May 4, 2026 that may be closed by this pull request
1 task
@mu-automation mu-automation Bot added language:python Pull requests that update Python code impact:non-functional Does not have a functional impact labels May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:non-functional Does not have a functional impact language:python Pull requests that update Python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: NOOPT Builds Broken For Consumers

1 participant