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
Move mbedtls_dev
to framework submodule
#9094
base: development
Are you sure you want to change the base?
Move mbedtls_dev
to framework submodule
#9094
Conversation
1e91186
to
eb11292
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.
This looks quite good to me. I can see that the "Move some files to framework repository" commits in mbedtls and mbedtls-framework are both based on the same commit: 61734ec. The one in mbedtls removes the mbedtls_dev
files while the one in mbedtls-framework removes all files but the mbedtls_dev
ones that are renamed (path change only) (checked with git show 75541028a9639a8c8cb7805fdec114dc50844931 --name-status in the framework submodule).
Otherwise, have you checked that you can play the same move game subsequently on another set of mbedtls
files, tests/data_files/*
for example without merge trouble in framework
?
I have now added to the script so that it can do this. The result is in Mbed-TLS/mbedtls-framework#18, which is based on the corresponding PR in the framework submodule. |
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.
This looks good to me, thanks. There is a conflict with development now thus you will need to rebase though.
Signed-off-by: David Horstmann <david.horstmann@arm.com>
This allows test scripts to find the new mbedtls_framework module. Signed-off-by: David Horstmann <david.horstmann@arm.com>
2df41f2
to
47fff1e
Compare
Change these to point to the new mbedtls_framework module in the framework submodule. Signed-off-by: David Horstmann <david.horstmann@arm.com>
This allows scripts in the scripts/ directory to get the path to the mbedtls_framework module in framework/scripts/ Signed-off-by: David Horstmann <david.horstmann@arm.com>
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 <david.horstmann@arm.com>
This allows us to use the mbedtls_framework python files moved there. Signed-off-by: David Horstmann <david.horstmann@arm.com>
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 <david.horstmann@arm.com>
47fff1e
to
ecd6d01
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.
I've checked the rebase. LGTM, thanks.
Move the
mbedtls_dev
python module into thembedtls-framework
repository. This PR pairs with Mbed-TLS/mbedtls-framework#15.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 #9131