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

Update Mbed TLS version to 2.13.1 #8022

Merged
merged 1 commit into from Sep 7, 2018

Conversation

@sbutcher-arm
Contributor

sbutcher-arm commented Sep 6, 2018

Description

This PR updates Mbed TLS to the patch release, 2.13.1.

Note, this PR is still pending formal approval for release, but I understand because that release is only a patch release, it gates the Mbed OS release and release assessment, not merging this PR.

PR should be reviewed by @RonEld and/or @Patater if or where available.

cc: @ChiefBureaucraticOfficer

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[X] Functionality change
[ ] Breaking change
@cmonr

This comment has been minimized.

Contributor

cmonr commented Sep 6, 2018

/morph build

@sbutcher-arm

This comment has been minimized.

Contributor

sbutcher-arm commented Sep 6, 2018

In case @RonEld and @Patater are unavailable, @hanno-arm could also be a reviewer, as I think this needs to be handled urgently.

@Patater

Patater approved these changes Sep 6, 2018

@mbed-ci

This comment has been minimized.

mbed-ci commented Sep 6, 2018

Build : SUCCESS

Build number : 3024
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8022/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@andresag01

The changes look good to me.

@cmonr cmonr added needs: CI and removed needs: review labels Sep 6, 2018

@mbed-ci

This comment has been minimized.

@mbed-ci

This comment has been minimized.

@cmonr cmonr added needs: review and removed needs: CI labels Sep 7, 2018

@cmonr

cmonr approved these changes Sep 7, 2018

@cmonr cmonr added ready for merge and removed needs: review labels Sep 7, 2018

@teetak01

Looks good. Passes our tests.

@0xc0170

0xc0170 approved these changes Sep 7, 2018

@0xc0170 0xc0170 merged commit 8a43879 into ARMmbed:master Sep 7, 2018

14 checks passed

ci-morph-build build completed
Details
ci-morph-exporter build completed
Details
ci-morph-mbed2-build build completed
Details
ci-morph-test test completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci/cloud_client_smoke_test Test job: successful
Details
travis-ci/astyle Passed, 598 files
Details
travis-ci/docs Local docs testing has passed
Details
travis-ci/events Passed, runtime is 9121 cycles (-749 cycles)
Details
travis-ci/gitattributestest Local gitattributestest testing has passed
Details
travis-ci/licence_check Local licence_check testing has passed
Details
travis-ci/littlefs Passed, code size is 8372B (+0.00%)
Details
travis-ci/tools-py2.7 Local tools-py2.7 testing has passed
Details

@0xc0170 0xc0170 removed the ready for merge label Sep 7, 2018

@cmonr

This comment has been minimized.

Contributor

cmonr commented Sep 7, 2018

Looks good. Passes our tests.

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment