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

[third-party] Update SGX SDK to resolve DCAP issue #200

Merged
merged 1 commit into from Jan 2, 2020

Conversation

@dingelish
Copy link
Contributor

dingelish commented Jan 2, 2020

Description

teaclave-sgx-sdk PR #190

Type of change (select applied and DELETE the others)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • API change with a documentation update
  • Additional test coverage
  • Code cleanup or just sync with upstream third-party crates

How Has This Been Tested?

http://ci.mesalock-linux.org/dingelish/incubator-mesatee/44

Checklist (check ALL before submitting PR, even not applicable)

  • Fork the repo and create your branch from master.
  • If you've added code that should be tested, add tests.
  • If you've changed APIs, update the documentation.
  • Ensure the tests pass (see CI results).
  • Make sure your code lints/format.
@dingelish dingelish requested a review from mssun Jan 2, 2020
@dingelish dingelish self-assigned this Jan 2, 2020
@dingelish

This comment has been minimized.

Copy link
Contributor Author

dingelish commented Jan 2, 2020

We may need to vender sgx_types/sgx_urts from github in crates-io submodule, instead of keep using the one from crates.io

@dingelish

This comment has been minimized.

Copy link
Contributor Author

dingelish commented Jan 2, 2020

CI passed

@mssun
mssun approved these changes Jan 2, 2020
@mssun mssun merged commit 1a671e6 into apache:master Jan 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.