Skip to content

Resolution of #10022 #10239

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

ronald-cron-arm
Copy link
Contributor

@ronald-cron-arm ronald-cron-arm commented Jun 20, 2025

Description

Update TF-PSA-Crypto submodule to fix #10022

PR checklist

Fail the cmake install demonstration if the
installed mbedtls/build_info.h is the
alias used by TF-PSA-Crypto for its tests.

Signed-off-by: Ronald Cron <[email protected]>
@ronald-cron-arm ronald-cron-arm force-pushed the mbedtls-build-info-last branch from f715ee8 to d47ace1 Compare June 20, 2025 07:23
@ronald-cron-arm ronald-cron-arm added needs-ci Needs to pass CI tests needs-review Every commit must be reviewed by at least two team members, and removed needs-ci Needs to pass CI tests labels Jun 20, 2025
Copy link
Contributor

@davidhorstmann-arm davidhorstmann-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 but CI is unhappy

[Edit: Looks infra-related except for the API-ABI check, which is picking up the updated crypto submodule with a change to how PAKE struct types are used from #10200]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs-review Every commit must be reviewed by at least two team members,
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CMake install picks up the build_info.h from TF-PSA-Crypto
2 participants