From: Miles Bader <miles@gnu.org>
Date: Sat, 22 Jan 2011 02:45:57 +0000 (+0900)
Subject: configure.in: Don't zero-out FONTCONFIG_CFLAGS and FONTCONFIG_LIBS
X-Git-Tag: emacs-pretest-24.0.90~104^2~618^2~1322^2~171
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=48dcd48260a2e6c3dc3b02ff71df9b62d7830600;p=emacs.git

configure.in: Don't zero-out FONTCONFIG_CFLAGS and FONTCONFIG_LIBS
when building with XFT (doing so is incorrect, as Emacs directly uses
fontconfig, and breaks building when using a strict linker).
---

diff --git a/ChangeLog b/ChangeLog
index 364fe879c5e..e00254b23ab 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2011-01-22  Miles Bader  <miles@gnu.org>
+
+	* configure.in: Don't zero-out FONTCONFIG_CFLAGS and
+	FONTCONFIG_LIBS when building with XFT (doing so is incorrect, as
+	Emacs directly uses fontconfig, and breaks building when using a
+	strict linker).
+
 2011-01-21  Paul Eggert  <eggert@cs.ucla.edu>
 
 	src/config.in: shrink slightly
diff --git a/configure.in b/configure.in
index bd1133319ba..161ba74f944 100644
--- a/configure.in
+++ b/configure.in
@@ -2252,17 +2252,9 @@ if test "${HAVE_X11}" = "yes"; then
        HAVE_XFT=no
     fi
 
-
-    HAVE_FREETYPE=no
     ## We used to allow building with FreeType and without Xft.
     ## However, the ftx font backend driver is not in good shape.
-    if test "${HAVE_XFT}" = "yes"; then
-      dnl As we use Xft, we anyway use freetype.
-      dnl There's no need for additional CFLAGS and LIBS.
-      HAVE_FREETYPE=yes
-      FONTCONFIG_CFLAGS=
-      FONTCONFIG_LIBS=
-    fi
+    HAVE_FREETYPE=$HAVE_XFT
 
     HAVE_LIBOTF=no
     if test "${HAVE_FREETYPE}" = "yes"; then