]> granicus.if.org Git - php/commitdiff
ext/gettext: respect passed in library directory for all checks
authorMichael Heimpold <mhei@heimpold.de>
Thu, 13 Apr 2017 20:43:26 +0000 (22:43 +0200)
committerJoe Watkins <krakjoe@php.net>
Mon, 17 Apr 2017 07:11:40 +0000 (08:11 +0100)
A directory given to configure by --with-gettext=dir is only
used within the very first AC_CHECK_LIB. This is because the
temporary modified LDFLAGS variable is reset too early.

This results in functions not detected properly.

The original issue and patch was reported for OpenWrt/LEDE
distribution by @Dimazhan at:
https://github.com/openwrt/packages/issues/4250

Signed-off-by: Michael Heimpold <mhei@heimpold.de>
ext/gettext/config.m4

index 24dddd3a43fc4c64e1fa0d59569d0c20541e29d2..c82be9cc057b66c0fcddd2bf07150c2332ada827 100644 (file)
@@ -30,7 +30,6 @@ if test "$PHP_GETTEXT" != "no"; then
                AC_MSG_ERROR(Unable to find required gettext library)
        ])
   )
-  LDFLAGS=$O_LDFLAGS
 
   AC_DEFINE(HAVE_LIBINTL,1,[ ])
   PHP_NEW_EXTENSION(gettext, gettext.c, $ext_shared)
@@ -46,5 +45,6 @@ if test "$PHP_GETTEXT" != "no"; then
   AC_CHECK_LIB($GETTEXT_CHECK_IN_LIB, dngettext,  [AC_DEFINE(HAVE_DNGETTEXT, 1, [ ])])
   AC_CHECK_LIB($GETTEXT_CHECK_IN_LIB, dcngettext,  [AC_DEFINE(HAVE_DCNGETTEXT, 1, [ ])])
   AC_CHECK_LIB($GETTEXT_CHECK_IN_LIB, bind_textdomain_codeset,  [AC_DEFINE(HAVE_BIND_TEXTDOMAIN_CODESET, 1, [ ])])
+  LDFLAGS=$O_LDFLAGS
   
 fi