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

Clone ring from the head of the ring repository into third_party #2681

Merged
merged 4 commits into from
Apr 6, 2022

Conversation

jul-sh
Copy link
Contributor

@jul-sh jul-sh commented Apr 5, 2022

Ring has been cloned cloned into third_party, using commit 32b2c6c39ef459aad47f476fe9139b8d57532ab1 from https://github.com/briansmith/ring. This is done so that we will be able to later port our own patch onto it, specifically UEFI compatibility from: briansmith/ring#1406.

Note that this does not yet build or use the ring clone. In order to build it in our repo some patches are needed. The reason this is a separate PR is purely that we squash PR commits. For provenance it would be nice to have a commit in the main branch that contains the unaltered clone, allowing us to record all subsequent changes in version history.

@jul-sh jul-sh force-pushed the ring-clone branch 2 times, most recently from d4da1d6 to 135cc4b Compare April 6, 2022 10:40
@jul-sh jul-sh requested a review from a team as a code owner April 6, 2022 10:40
Ring has been cloned cloned into third_party, using commit 32b2c6c39ef459aad47f476fe9139b8d57532ab1 from https://github.com/briansmith/ring. This is done so that we will be able to later port our own patch onto it, specifically UEFI compatibility from: briansmith/ring#1406.
@jul-sh jul-sh merged commit 662cb9f into project-oak:main Apr 6, 2022
@jul-sh jul-sh deleted the ring-clone branch April 6, 2022 16:05
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.

4 participants