-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[Backport 3.6] Move mbedtls_dev
to framework submodule
#9131
[Backport 3.6] Move mbedtls_dev
to framework submodule
#9131
Conversation
mbedtls_dev
to framework submodulembedtls_dev
to framework submodule
Signed-off-by: David Horstmann <[email protected]>
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]>
07b991d
to
7f6c81a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Ronald Cron <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Submodule update LGTM
There was a problem hiding this 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.
There was a problem hiding this 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.
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")
mbedtls_dev
to framework submodule #9094