chriskohlhoff / asio

Asio C++ Library
http://think-async.com/Asio
4.74k stars 1.2k forks source link

--with-wolfssl configurations is broken #574

Open tmael opened 3 years ago

tmael commented 3 years ago

Hi @chriskohlhoff,

A build --with-wolfssl configuration option is broken.

wolfSSL previously submitted PR #383 which has been merged into the asio mainline. Thank you!

Unfortunately, the --with-wolfssl build option hook in configure.ac was missed when cherry-picking the changes.

Can you please merge the pieces that were missed?

The workaround is to build with the following options. ./configure CPPFLAGS="-DASIO_USE_WOLFSSL" LDFLAGS="-lwolfssl"

Please let me know if you need more info or if you would like me to submit a new pull request.

Thanks, Tesfa

tomoveu commented 3 years ago

I can confirm that wolfssl option switch is broken.

asio-wolfssl-switch-broken

tomoveu commented 3 years ago

Hi @chriskohlhoff could you please comment.

The original upstreaming effort was good, but the partial merge(cherry pick) caused a broken configuration.

Thanks, Dimi

tmael commented 3 years ago

Hi Team ASIO,

This issue is blocking users of ASIO with wolfSSL building against the main-line for production releases. Any update on this issue? Please let me know.

Thanks, Tesfa

ppavlik commented 2 years ago

@tmael , can I suggest you create a PR for this issue to help move things along? This has been outstanding for too long.

Thanks, Paul

tmael commented 2 years ago

@ppavlik, good idea! haven't had luck getting an email response from Chris neither. Anyway, @dgarske put up a PR to address the issue. Hoping it gets merged quickly. Thanks