Skip to content

Update feature/mbedtls with mbedtls' development HEAD #2843

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

Merged
merged 1 commit into from
Sep 28, 2016

Conversation

andresag01
Copy link

@andresag01 andresag01 commented Sep 28, 2016

Description

This is a follow up PR from #2716. It updates feature/mbedtls with the head of https://github.com/ARMmbed/mbedtls to enable the correct behaviour of RNG HAL. It also updates the importer Makefile to sync mbedtls with mbedtls.git rather than the restricted repository in the future.

PLEASE NOTE - This is the DEVELOPMENT HEAD of mbed TLS and is cannot be released. It will be updated shortly by the next formal release of mbed TLS.

Status

READY

Migrations

YES
This PR adds the latest API changes from mbedtls, but none of these should be breaking changes.

Related PRs

There are four related PRs, one in mbedtls and three in mbed-os. All except #2603 have been merged.

Todos

Steps to test or reproduce

Simply merge this PR and use mbed-os as normal.

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 28, 2016

/morph test

Copy link
Contributor

@0xc0170 0xc0170 left a comment

Choose a reason for hiding this comment

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

Code changes LGTM

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 977

All builds and test passed!

@c1728p9
Copy link
Contributor

c1728p9 commented Oct 4, 2016

@andresag01 this PR prevents customization from a file since #include MBEDTLS_USER_CONFIG_FILE is in the else case and never gets included.
https://github.com/ARMmbed/mbed-os/pull/2843/files#diff-89ba9a5d810662c2b06d0293d7eddcbfR36

CC: @sbutcher-arm

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.

5 participants