Add mbedTLS support to nmake build system #606

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@BurningEnlightenment
Contributor

I modified the nmake scripts to accept an additional WITH_MBEDTLS option which works almost exactly like the WITH_SSL option.
I used the autoconf script as a reference for this implementation and it works well with Visual Studio 2015. If I missed something important I'll fix it ASAP.

Cheers,
Henrik

@mention-bot

By analyzing the blame information on this pull request, we identified @captain-caveman2k, @bagder and @yangtse to be potential reviewers

@bagder
Member
bagder commented Apr 9, 2016

Do you still think this is a good idea? It seems this patch now needs to be rebased to work again...

@bagder bagder added the build label Apr 9, 2016
@BurningEnlightenment
Contributor

I'll take a look at it tomorrow after uni.

@BurningEnlightenment BurningEnlightenment ADD mbedTLS SUPPORT TO NMAKE BUILD SYSTEM
Add WITH_MBEDTLS nmake option. Make WITH_SSL, WITH_MBEDTLS and
ENABLE_WINSSL options mutual exclusive.
9029380
@BurningEnlightenment
Contributor

I've rebased it on top of the master branch.

@bagder bagder added a commit that closed this pull request Apr 21, 2016
@BurningEnlightenment @bagder BurningEnlightenment + bagder winbuild: add mbedtls support
Add WITH_MBEDTLS option. Make WITH_SSL, WITH_MBEDTLS and ENABLE_WINSSL
options mutual exclusive.

Closes #606
5d8093e
@bagder bagder closed this in 5d8093e Apr 21, 2016
@bagder
Member
bagder commented Apr 21, 2016

Thanks, merged now!

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