Skip to content

Conversation

@davidhorstmann-arm
Copy link
Contributor

Trivial backport of #9094.

PR checklist

Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")

@davidhorstmann-arm davidhorstmann-arm added needs-review Every commit must be reviewed by at least two team members, needs-ci Needs to pass CI tests needs-reviewer This PR needs someone to pick it up for review priority-high High priority - will be reviewed soon labels May 13, 2024
@davidhorstmann-arm davidhorstmann-arm changed the base branch from development to mbedtls-3.6 May 13, 2024 12:53
@davidhorstmann-arm davidhorstmann-arm changed the title [backport 3.6] Move mbedtls_dev to framework submodule [Backport 3.6] Move mbedtls_dev to framework submodule May 13, 2024
This allows test scripts to find the new mbedtls_framework module.

Signed-off-by: David Horstmann <[email protected]>
Change these to point to the new mbedtls_framework module in the
framework submodule.

Signed-off-by: David Horstmann <[email protected]>
This allows scripts in the scripts/ directory to get the path to the
mbedtls_framework module in framework/scripts/

Signed-off-by: David Horstmann <[email protected]>
Since this python module resides in the framework submodule we must
add an extra explicit path to it for mypy.

Signed-off-by: David Horstmann <[email protected]>
This allows us to use the mbedtls_framework python files moved there.

Signed-off-by: David Horstmann <[email protected]>
This name is more descriptive of its purpose, since it actually adds
framework/scripts to the path rather than just framework/

Signed-off-by: David Horstmann <[email protected]>
@davidhorstmann-arm davidhorstmann-arm force-pushed the move-mbedtls-dev-to-framework-3.6 branch from 07b991d to 7f6c81a Compare May 13, 2024 13:44
@davidhorstmann-arm davidhorstmann-arm removed the needs-ci Needs to pass CI tests label May 13, 2024
@ronald-cron-arm ronald-cron-arm requested a review from bensze01 May 17, 2024 06:13
@ronald-cron-arm ronald-cron-arm removed the needs-reviewer This PR needs someone to pick it up for review label May 17, 2024
Copy link
Contributor

@ronald-cron-arm ronald-cron-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

Copy link
Contributor Author

@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.

Submodule update LGTM

Copy link
Contributor

@ronald-cron-arm ronald-cron-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 up to the last commit and @davidhorstmann-arm checked the last commit.

Copy link
Contributor

@bensze01 bensze01 left a comment

Choose a reason for hiding this comment

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

Faithful backport of #9094, LGTM.

@bensze01 bensze01 added approved Design and code approved - may be waiting for CI or backports and removed needs-review Every commit must be reviewed by at least two team members, labels May 28, 2024
@bensze01 bensze01 added this pull request to the merge queue May 28, 2024
Merged via the queue into Mbed-TLS:mbedtls-3.6 with commit 29c4017 May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Design and code approved - may be waiting for CI or backports priority-high High priority - will be reviewed soon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants