Re: svn commit: r189771 - /httpd/httpd/branches/fips-dev/acinclude.m4

[Joe Orton <jo...@redhat.com>]

On Thu, Jun 09, 2005 at 04:10:54PM -0000, ben@apache.org wrote:
> Author: ben
> Date: Thu Jun  9 09:10:53 2005
> New Revision: 189771
> 
> URL: http://svn.apache.org/viewcvs?rev=189771&view=rev
> Log:
> Make path check more portable, make specifying a local OpenSSL build
> actually work.

> @@ -441,7 +441,7 @@
>      else
>        ap_ssltk_lib="$ap_ssltk_base"
>      fi
> -    LDFLAGS="$LDFLAGS -L$ap_ssltk_lib"
> +    LDFLAGS="-L$ap_ssltk_lib $LDFLAGS"
>    fi
>    dnl make sure "other" flags are available so libcrypto and libssl can link
>    LIBS="$LIBS `$apr_config --libs`"
> @@ -477,7 +477,7 @@
>    fi
>    dnl (c) hook up linker paths
>    if test "x$ap_ssltk_lib" != "x"; then
> -    APR_ADDTO(LDFLAGS, ["-L$ap_ssltk_lib"])
> +    APR_SETVAR(LDFLAGS, [-L$ap_ssltk_lib $LDFLAGS])
>      if test "x$ap_platform_runtime_link_flag" != "x"; then
>        APR_ADDTO(LDFLAGS, ["$ap_platform_runtime_link_flag$ap_ssltk_lib"])
>      fi

These changes only make a difference if you've managed to get a
-L/usr/lib or similar in your LDFLAGS in which case all bets are off -
is that the case?  Mucking around with the ordering is otherwise likely
to fix something for you and break something for some other guy.

joe