Skip to content

Add fix for -DENABLE_PROGRAMS=OFF build error#10654

Open
bjwtaylor wants to merge 2 commits intoMbed-TLS:developmentfrom
bjwtaylor:no-program-failure
Open

Add fix for -DENABLE_PROGRAMS=OFF build error#10654
bjwtaylor wants to merge 2 commits intoMbed-TLS:developmentfrom
bjwtaylor:no-program-failure

Conversation

@bjwtaylor
Copy link
Copy Markdown

@bjwtaylor bjwtaylor commented Mar 25, 2026

Description

Add fix for -DENABLE_PROGRAMS=OFF build error, resolves #10587

PR checklist

Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
@bjwtaylor bjwtaylor added the needs-backports Backports are missing or are pending review and approval. label Mar 25, 2026
@bjwtaylor bjwtaylor marked this pull request as ready for review March 25, 2026 11:38
@bjwtaylor bjwtaylor added needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review size-s Estimated task size: small (~2d) priority-medium Medium priority - this can be reviewed as time permits labels Mar 25, 2026
@davidhorstmann-arm davidhorstmann-arm moved this to Scoped in Community Mar 26, 2026
@gilles-peskine-arm gilles-peskine-arm self-requested a review April 10, 2026 16:20
Copy link
Copy Markdown
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except for the missing changelog entry.

This is minor, so we don't need a test.

Comment thread tests/CMakeLists.txt
add_custom_target(handshake-generated.sh
DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/opt-testcases/handshake-generated.sh)
set_target_properties(handshake-generated.sh PROPERTIES EXCLUDE_FROM_ALL NO)
if(ENABLE_PROGRAMS)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this fixes a reported bug, please add a changelog entry with Fixes #10587.

@gilles-peskine-arm gilles-peskine-arm added needs-work size-xs Estimated task size: extra small (a few hours at most) and removed needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review size-s Estimated task size: small (~2d) labels Apr 10, 2026
Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-backports Backports are missing or are pending review and approval. needs-work priority-medium Medium priority - this can be reviewed as time permits size-xs Estimated task size: extra small (a few hours at most)

Projects

Status: Scoped

Development

Successfully merging this pull request may close these issues.

Making the library without programs fails

3 participants