Use BoringSSL instead of OpenSSL#4235
Merged
murgatroid99 merged 5 commits intogrpc:release-0_12from Dec 3, 2015
Merged
Conversation
Member
|
The travis build failed. |
Contributor
Author
|
It's weird, it used XCode 6.1 for those builds, instead of 7... |
Contributor
Author
|
Restarting them made them use the correct XCode and pass. The change took 10min off each of the 3 tests. Half of what remains is compiling protoc, and the other half is probably compiling OpenSSL for the test server. |
murgatroid99
added a commit
that referenced
this pull request
Dec 3, 2015
Use BoringSSL instead of OpenSSL
Contributor
Author
|
For the record, this also fixes #4295, which happens every time OpenSSL is released and lasts O(day) until the OpenSSL podspec maintainer updates it. |
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a prerequisite for solving #4199 (which is essentially #4121), because a dynamic framework can't be created with precompiled libraries.
It fixes #4081 and any future issue of the sort, because the BoringSSL pod works from sources.
And removes steps 6, 7, 8 and 9 from #3492, because it eliminates the 15 - 20 min of time building OpenSSL the first time one does
pod install(which times outrun_tests.sh). By the same token, it should speed up Travis builds considerably.