Re: pango configure patch

[Owen Taylor <otaylor redhat com>]

Sure, go ahead and make the change. (Could you change the other
use of pango_save_cflags to pango_save_cppflags at the same time;
there isn't much difference, but we might as well be consistent,
and using CPPFLAGS is slightly more "correct")

Thanks,
                                        Owen

Alexander Larsson <alla lysator liu se> writes:

> The Xft configure check in pango doesn't seem to work for me. I needed
> this patch, is it ok to commit?
> 
> Index: configure.in
> ===================================================================
> RCS file: /cvs/gnome/pango/configure.in,v
> retrieving revision 1.63
> diff -u -p -r1.63 configure.in
> --- configure.in	2001/05/04 23:12:58	1.63
> +++ configure.in	2001/05/06 05:17:34
> @@ -108,6 +108,9 @@ XFT_LIBS=""
>  XFT_CFLAGS=""
>  if test $have_x = true ; then
>    if test $have_freetype = true ; then
> +    pango_save_cppflags="$CPPFLAGS"
> +    CPPFLAGS="$CPPFLAGS $FREETYPE_CFLAGS"
> +
>      AC_CHECK_LIB(Xrender, XRenderFindFormat, 
>        AC_CHECK_LIB(Xft, XftFontOpen,
>  	AC_CHECK_HEADER(X11/Xft/XftFreetype.h,
> @@ -115,6 +118,8 @@ if test $have_x = true ; then
>  	:,-lXrender -lXext $X_LIBS $FREETYPE_LIBS)
>      ,:,-lXext $X_LIBS)
>  
> +    CPPFLAGS="$pango_save_cppflags"
> +    
>      if $have_xft = 'true' ; then
>        XFT_LIBS="-lXft -lXrender -lXext $X_LIBS $FREETYPE_LIBS"