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

[BoringSSL] Update third_party/boringssl-with-bazel #36435

Closed
wants to merge 7 commits into from

Conversation

yashykt
Copy link
Member

@yashykt yashykt commented Apr 24, 2024

Change was created by the release automation script. See go/grpc-release.

@sampajano
Copy link
Contributor

@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:
Copy link
Collaborator

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'

Copy link
Member Author

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 :(

Copy link
Collaborator

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?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! That fixed it

@copybara-service copybara-service bot closed this in a0bee78 May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants