Re: [PATCH] BUG/MEDIUM: build without openssl broken

2017-06-09 Thread Emmanuel Hocdet
> Le 9 juin 2017 à 17:07, Willy Tarreau a écrit : > > On Fri, Jun 09, 2017 at 05:02:56PM +0200, Emmanuel Hocdet wrote: >> Why ssl_free_dh (and #include ) suddenly (re)appear in >> haproxy.c >> with this commit? This looks like a remnant of old dev. > > It doesn't re-appear with

Re: [PATCH] BUG/MEDIUM: build without openssl broken

2017-06-09 Thread Willy Tarreau
On Fri, Jun 09, 2017 at 05:02:56PM +0200, Emmanuel Hocdet wrote: > Why ssl_free_dh (and #include ) suddenly (re)appear in > haproxy.c > with this commit? This looks like a remnant of old dev. It doesn't re-appear with this commit, look, it was already there : > > -#ifndef OPENSSL_NO_DH > > +#if

Re: [PATCH] BUG/MEDIUM: build without openssl broken

2017-06-09 Thread Emmanuel Hocdet
Why ssl_free_dh (and #include ) suddenly (re)appear in haproxy.c with this commit? This looks like a remnant of old dev. > Le 8 juin 2017 à 19:05, William Lallemand a écrit : > > The commit 872f9c213 ("MEDIUM: ssl: add basic support for OpenSSL crypto > engine") broke

Re: [PATCH] BUG/MEDIUM: build without openssl broken

2017-06-08 Thread Willy Tarreau
On Thu, Jun 08, 2017 at 07:05:48PM +0200, William Lallemand wrote: > The commit 872f9c213 ("MEDIUM: ssl: add basic support for OpenSSL crypto > engine") broke the build without openssl support. Applied, thanks William. Willy

[PATCH] BUG/MEDIUM: build without openssl broken

2017-06-08 Thread William Lallemand
The commit 872f9c213 ("MEDIUM: ssl: add basic support for OpenSSL crypto engine") broke the build without openssl support. The ssl_free_dh() function is not defined when USE_OPENSSL is not defined and leads to a compilation failure. --- src/haproxy.c | 2 +- 1 file changed, 1 insertion(+), 1