-
Notifications
You must be signed in to change notification settings - Fork 725
fix: enable -Wcast-qual flag for libcrypto=awslc #4735
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
base: main
Are you sure you want to change the base?
Conversation
* Remove the condition which only add -Wcast-qual back for non AWS-lc Libcrypto.
This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
* add -Wcast-qual to the general compilation options
This PR should be unblocked. The original problem is caused by AWSLC having cast-qual issue, and it is fixed now. This was the original problem with compilation:
We have removed that |
@@ -135,6 +135,7 @@ target_compile_options(${PROJECT_NAME} PRIVATE | |||
-std=gnu99 | |||
-Wall | |||
-Wcast-align | |||
-Wcast-qual |
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 think this change conflicts with #5198?
If we want to continue to support older versions of AWS-LC, I'm not sure we can add this flag. Unless there's a way to check the AWS-LC version first.
Resolved issues:
resolves #4710
Description of changes:
Call-outs:
Testing:
S2N_LIBCRYPTO=openssl
.AWSLC
andAWSLC-FIPS
.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.