[hybrid] [PATCH] Fix autoconf script for SSL support
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
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_DEFINE(HAVE_LIBCRYPTO, 1, [If we support ssl])
> AC_MSG_RESULT([not found. Please check your path.])
More information about the hybrid