Externals: Update mbedtls to 2.4.1 #4778

Merged
merged 1 commit into from Feb 15, 2017

Projects

None yet

4 participants

@leoetlino
Member

The latest version has tons of security fixes (which is expected for a library such as mbedtls).

Updating also allows getting rid of a few deprecation warnings.

@sepalani
Contributor

It doesn't seem to break SSL features on Windows so LGTM concerning that part.

@Sarkie
Sarkie commented Jan 28, 2017

Don't have access to desktop but is zlib ok to be updated to latest?

@leoetlino
Member

Is there a reason to update zlib? If not, I'm not sure if it's worth doing, as it will require yet another build dir cleanup.

@lioncash
Member

This needs a rebase

@leoetlino leoetlino Externals: Update mbedtls to 2.4.1
The latest version has tons of security fixes (which is expected for a
library such as mbedtls).

Updating also allows getting rid of a few deprecation warnings.
b8dd3e6
@lioncash lioncash merged commit 8fd16db into dolphin-emu:master Feb 15, 2017

10 checks passed

default Very basic checks passed, handed off to Buildbot.
Details
lint Build succeeded on builder lint
Details
pr-android Build succeeded on builder pr-android
Details
pr-deb-dbg-x64 Build succeeded on builder pr-deb-dbg-x64
Details
pr-deb-x64 Build succeeded on builder pr-deb-x64
Details
pr-freebsd-x64 Build succeeded on builder pr-freebsd-x64
Details
pr-osx-x64 Build succeeded on builder pr-osx-x64
Details
pr-ubu-x64 Build succeeded on builder pr-ubu-x64
Details
pr-win-dbg-x64 Build succeeded on builder pr-win-dbg-x64
Details
pr-win-x64 Build succeeded on builder pr-win-x64
Details
@leoetlino leoetlino deleted the leoetlino:update-mbedtls branch Feb 15, 2017
@Sarkie
Sarkie commented Feb 24, 2017

This needs fixing as haven't had an Android build since this was committed.

https://dolphin-emu.org/download/list/master/2/

@leoetlino
Member

It'd require someone to clear the build cache on the Ubuntu builder, something I don't have access to.

@Sarkie
Sarkie commented Feb 24, 2017

@degasus @Parlane @lioncash Do you guys have visibility of this?

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