-
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
Replace MBEDTLS_MD_CAN_MD5 with PSA_WANT_ALG_MD5 #9123
Replace MBEDTLS_MD_CAN_MD5 with PSA_WANT_ALG_MD5 #9123
Conversation
2a9a667
to
4cb859d
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
The CI is failing after all test have been passed.
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 - thanks!
Only remaining hits for MBEDTLS_MD_CAN_MD5
are in the include
directory and the description in #9110 says that it is not required to remove from there.
Question though: Why not just remove these hits from check_config.h
and config_adjust_legacy_crypto.h
? Are they being removed later? @eleuzi01 @gabor-mezei-arm
Given this comment, I am ignoring the CI failures. |
It will be removed later. |
Signed-off-by: Elena Uziunaite <[email protected]>
b66a991
4cb859d
to
b66a991
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
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 - thanks!
Description
Issue #9110.
Status: blocked by #9290 which fixes dependencies around a use of MD5 in test code.
PR checklist
Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")
Notes for the submitter
Please refer to the contributing guidelines, especially the
checklist for PR contributors.
Help make review efficient: