Skip to content
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

Finalize split preparation-2 #99

Merged

Conversation

ronald-cron-arm
Copy link
Contributor

@ronald-cron-arm ronald-cron-arm commented Dec 9, 2024

Companion PR of #9828.

@ronald-cron-arm ronald-cron-arm changed the title Move test_keys.h to include/test Finalize split preparation-2 Dec 9, 2024
@ronald-cron-arm ronald-cron-arm force-pushed the finalize-split-preparation-2 branch from 13c9bdb to 3663221 Compare December 10, 2024 11:11
Move test_keys.h to tests/include/test
instead of tests/src as it is used
outside of tests/src namely by
test_suite_pk.

Signed-off-by: Ronald Cron <[email protected]>
Adapt default path of psa_test_wrappers.[hc]
in generate_psa_wrappers.py when called in
the context of Mbed TLS 4.x.

Signed-off-by: Ronald Cron <[email protected]>
@ronald-cron-arm ronald-cron-arm force-pushed the finalize-split-preparation-2 branch from 3663221 to 8e3b171 Compare December 10, 2024 15:27
@ronald-cron-arm ronald-cron-arm added enhancement New feature or request needs-review Every commit must be reviewed by at least two team members, priority-high High priority - will be reviewed soon labels Dec 11, 2024
@ronald-cron-arm
Copy link
Contributor Author

Validated by #9832 CI for 3.6 and #9828 CI for development.

Copy link
Contributor

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

@gabor-mezei-arm gabor-mezei-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

@ronald-cron-arm ronald-cron-arm merged commit ff4c336 into Mbed-TLS:main Dec 11, 2024
1 of 2 checks passed
eleuzi01 pushed a commit to eleuzi01/mbedtls-framework that referenced this pull request Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-review Every commit must be reviewed by at least two team members, priority-high High priority - will be reviewed soon
Development

Successfully merging this pull request may close these issues.

3 participants