Update framework pointer to fix ci.requirements issue #721

Merged
bjwtaylor merged 1 commit from ci.requirements into development 2026-03-20 17:17:34 -05:00
bjwtaylor commented 2026-03-20 09:18:02 -05:00 (Migrated from github.com)

Description

There is currently an issue that the version of the framework in tf-psa-crypto does not include ci.requirements.txt. This is causing issue with the CI as when you create a pull request for tf-psa-crypto the mbedtls framework pointer is updated to the one contained in tf-psa-crypto and this breaks the build when it tries to find the missing file. This update resolved this issue by updating to the commit that includes this file.

PR checklist

  • changelog not required because: No public changes
  • framework PR not required
  • mbedtls development PR not required because: No changes
  • mbedtls 3.6 PR not required because: No changes
  • tests not required because: No tests
## Description There is currently an issue that the version of the framework in tf-psa-crypto does not include ci.requirements.txt. This is causing issue with the CI as when you create a pull request for tf-psa-crypto the mbedtls framework pointer is updated to the one contained in tf-psa-crypto and this breaks the build when it tries to find the missing file. This update resolved this issue by updating to the commit that includes this file. ## PR checklist - [x] **changelog** not required because: No public changes - [x] **framework PR** not required - [x] **mbedtls development PR** not required because: No changes - [x] **mbedtls 3.6 PR** not required because: No changes - **tests** not required because: No tests
bensze01 (Migrated from github.com) requested changes 2026-03-20 09:24:14 -05:00
bensze01 (Migrated from github.com) left a comment

Please update the framework pointer to the head of main instead of a non-merge commit from a pull request.

Please update the framework pointer to the head of `main` instead of a non-merge commit from a pull request.
bjwtaylor commented 2026-03-20 09:25:56 -05:00 (Migrated from github.com)

Yeah make sense, I was going with the minimum changes. Though it makes sense to update to latest.

Yeah make sense, I was going with the minimum changes. Though it makes sense to update to latest.
bensze01 (Migrated from github.com) approved these changes 2026-03-20 09:46:08 -05:00
gilles-peskine-arm (Migrated from github.com) approved these changes 2026-03-20 15:15:38 -05:00
gilles-peskine-arm (Migrated from github.com) left a comment

LGTM

LGTM
Sign in to join this conversation.
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-Mbed-TLS/TF-PSA-Crypto!721
No description provided.