Re: /srv/bzr/emacs/trunk r99850: Lucid menus can now use Xft for fonts.

[Óscar Fuentes]

Óscar Fuentes <address@hidden> writes:

> "Jan D." <address@hidden> writes:
>
>> ------------------------------------------------------------
>> revno: 99850 [merge]
>> committer: Jan D. <address@hidden>
>> branch nick: trunk
>> timestamp: Thu 2010-04-08 18:21:25 +0200
>> message:
>>   Lucid menus can now use Xft for fonts.
>>   
>>   * xsettings.c (current_font, SYSTEM_FONT, XSETTINGS_FONT_NAME): New.
>>   (parse_xft_settings): Also check for XSETTINGS_FONT_NAME and save that
>>   in current_font.
>>   (init_gconf): Read value of SYSTEM_FONT and save it in current_font.
>
> [snip]
>
> This change breaks the build for me.
>
> #define XSETTINGS_FONT_NAME is within a #ifdef HAVE_GCONF ... #endif but
> it is used without that condition. This is a KDE machine, so HAVE_GCONF
> is undefined here. (I'll love to see nice fonts on menus, though ;-)

BTW, moving the

#define XSETTINGS_FONT_NAME  "Gtk/FontName"

outside the #ifdef HAVE_GCONF ... #endif fixes the build. The menus now
look great. Thanks!