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
[BoringSSL] Update third_party/boringssl-with-bazel #36435
Conversation
@HannahShiSFB Hi Hannah could you help take a look at what are some potential causes for those iOS failures? I know "Basic Tests ObjC iOS" is flaky ~5% of the time so it could just be that. But the "Basic Tests ObjC Examples" failures aren't known to be flaky. |
@@ -70,7 +70,7 @@ | |||
|
|||
Pod::Spec.new do |s| | |||
s.name = 'BoringSSL-GRPC' | |||
version = '0.0.33' | |||
version = '0.0.34' | |||
s.version = version | |||
s.summary = 'BoringSSL is a fork of OpenSSL that is designed to meet Google\'s needs.' | |||
# Adapted from the homepage: |
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.
Can you try add the include files pattern in line 152 to include sub directories?
Add something like 'src/include/openssl/**/*.h'
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.
That fixed that, but looks like there are other issues now :(
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.
Looks like path to err_data.c changed, can you update it accordingly?
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.
Thanks! That fixed it
Change was created by the release automation script. See go/grpc-release.