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

OSSM-4338 Upgraded BoringSSL version from b95124305 to ca1690e22 #110

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

tedjpoole
Copy link
Contributor

Many of the patch files broke as a result of the upgrade, even though the changes in BoringSSL on the whole weren't that big.

Therefore, as part of this upgrade, the mechanism of copying and modifying BoringSSL files into the bssl-compat build has also been modified in the hope that subsequent upgrades will be easier.

Instead of using patch files to comment/uncomment/edit the BoringSSL source files, the new tools/uncomment.sh script operates at a higher level, operating on functions, macros, typedefs, structs, google tests etc. The hope is, that with more semantic info about what is being [un]commented, it will be less brittle.

Also upgraded envoy submodule to v1.26.3

Many of the patch files broke as a result of the upgrade, even
though the changes in BoringSSL on the whole weren't that big.

Therefore, as part of this upgrade, the mechanism of copying
and modifying BoringSSL files into the bssl-compat build has been
changed in the hope that subsequent upgrades will be easier.

Also upgraded envoy submodule to v1.26.3

Signed-off-by: Ted Poole <tpoole@redhat.com>
@tedjpoole tedjpoole requested a review from dcillera July 24, 2023 14:07
@tedjpoole tedjpoole merged commit cd6ab00 into envoyproxy:main Jul 25, 2023
2 checks passed
@tedjpoole tedjpoole deleted the OSSM-4338-upgrade-boringssl branch July 25, 2023 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants