[hybrid] [PATCH] Fix autoconf script for SSL support

Michael Wobst wobst.michael at web.de
Sun Jul 1 12:07:14 EDT 2012


Prepending "-lcrypto -ssl" to LIBS as well as defining "HAVE_LIBCRYPTO" 
is already done by the AC_CHECK_LIB macro calls when the ssl and crypto 
libraries are tested for existence.
So adding these two lines just makes things redundant, because this is 
exactly what AC_CHECK_LIB's job is.

Anyway, I'll take a look on how the configure script behaves on OpenBSD.
You could also send me the configure.log of the original, unmodified 
configure script.

-Michael

Brad Smith schrieb:
> The following diff fixes the autoconf script to be able to
> build with the SSL support. I noticed these two lines were
> removed between 7.3.1 and 8.0.0beta1. Removing the first
> link breaks the OpenSSL checks on OpenBSD. Removing the
> second line means even if the OpenSSL checks were to
> succeed with your OS the SSL code within ircd-hybrid
> would not be included.
>
>
> --- configure.ac.orig	Sun Jul  1 08:36:32 2012
> +++ configure.ac	Sun Jul  1 08:37:52 2012
> @@ -105,8 +105,10 @@ if test "$cf_enable_openssl" != "no"; then
>     dnl If we have a basedir defined, then everything is okay.  Otherwise,
>     dnl we have a problem.
>     if test ! -z "$cf_openssl_basedir"; then
> +    LIBS="-lcrypto -lssl $LIBS"
>       AC_MSG_RESULT([$cf_openssl_basedir])
>       cf_enable_openssl="yes"
> +    AC_DEFINE(HAVE_LIBCRYPTO, 1, [If we support ssl])
>     else
>       AC_MSG_RESULT([not found. Please check your path.])
>       cf_enable_openssl="no"
>




More information about the hybrid mailing list