[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: ld-output-def

From: Simon Josefsson
Subject: Re: ld-output-def
Date: Wed, 01 Apr 2009 15:38:32 +0200
User-agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.0.90 (gnu/linux)

Eric Blake <address@hidden> writes:

> According to Simon Josefsson on 3/31/2009 10:56 PM:
>>     AC_MSG_RESULT($gl_output_def)
>>     LDFLAGS="$gl_ldflags_save"
>>   fi
>>   AM_CONDITIONAL(HAVE_LD_OUTPUT_DEF, test "$gl_output_def" = "yes")
> Still missing m4 quoting.  On the other hand, shell quotes around yes are
> redundant (although harmless).
>     AC_MSG_RESULT([$gl_output_def])
>     LDFLAGS="$gl_ldflags_save"
>   fi
>   AM_CONDITIONAL([HAVE_LD_OUTPUT_DEF], [test "$gl_output_def" = yes])
> Also, any reason why you are not using an AC_CACHE_VAL (or even
> AC_CACHE_CHECK, with the contents of the cache variable being used as your
> followup message)?  Without using the cache, you are paying the cost of a
> link test on every subsequent configure run?

Good points!  Updated script below.


# ld-output-def.m4 serial 2
dnl Copyright (C) 2008, 2009 Free Software Foundation, Inc.
dnl This file is free software; the Free Software Foundation
dnl gives unlimited permission to copy and/or distribute it,
dnl with or without modifications, as long as this notice is preserved.

dnl From Simon Josefsson

# -------------
# Check if linker supports -Wl,--output-def and define automake
# conditional HAVE_LD_OUTPUT_DEF if it is.
  AC_CACHE_CHECK([if gcc/ld supports -Wl,--output-def],
    [if test "$enable_shared" = no; then
       gl_cv_ld_output_def="not needed, shared libraries are disabled"
       rm -f conftest.def
  AM_CONDITIONAL(HAVE_LD_OUTPUT_DEF, test "$gl_cv_ld_output_def" = "yes")

reply via email to

[Prev in Thread] Current Thread [Next in Thread]