Re: FC_COLOR in src/ftfont.c

[Paul Eggert <eggert@cs.ucla.edu>]

[-- Attachment #1: Type: text/plain, Size: 452 bytes --]

On 04/03/2018 06:58 AM, Eli Zaretskii wrote:
> I guess the solution depends on whether systems with fontconfig that
> doesn't support FC_COLOR are still susceptible to the original bug.

I doubt whether they're susceptible. Even if they are, we're better off 
compiling Emacs on such systems rather than having the build fail; at 
least that way we can further debug the problem. So I installed the 
attached workaround to help move the ball forward.


[-- Attachment #2: 0001-Port-FC_COLOR-change-to-older-fontconfig.txt --]
[-- Type: text/plain, Size: 1099 bytes --]

From 0f923ac2629c819cb16860d154945485e48287cb Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Tue, 3 Apr 2018 08:12:41 -0700
Subject: [PATCH] Port FC_COLOR change to older fontconfig
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Problem reported by John ff in:
https://lists.gnu.org/r/emacs-devel/2018-04/msg00058.html
* src/ftfont.c (ftfont_spec_pattern) [!FC_COLOR]:
Don’t use FC_COLOR on older fontconfigs that don’t have it.
---
 src/ftfont.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/ftfont.c b/src/ftfont.c
index 24a92dd52e..84e4a30389 100644
--- a/src/ftfont.c
+++ b/src/ftfont.c
@@ -764,7 +764,7 @@ ftfont_spec_pattern (Lisp_Object spec, char *otlayout, struct OpenTypeSpec **ots
   if (scalable >= 0
       && ! FcPatternAddBool (pattern, FC_SCALABLE, scalable ? FcTrue : FcFalse))
     goto err;
-#ifdef HAVE_XFT
+#if defined HAVE_XFT && defined FC_COLOR
   /* We really don't like color fonts, they cause Xft crashes.  See
      Bug#30874.  */
   if (Vxft_ignore_color_fonts
-- 
2.14.3