[PATCH fonts-util 2/2] If cross-compiling, we don't have to run mkfontdir
Jeremy Huddleston
jeremyhu at apple.com
Thu Sep 15 12:05:18 PDT 2011
On Sep 15, 2011, at 11:06 AM, Jon TURNEY wrote:
> On 14/09/2011 00:09, Jeremy Huddleston wrote:
>> This is confusing to me. Why do you add the extra " ; echo '** mkfontdir'"?
>
>> On Sep 12, 2011, at 9:18 AM, Jon TURNEY wrote:
>>> +AC_DEFUN([XORG_FONT_MKFONTDIR],[
>>> + if test x"$cross_compiling" != x"no" ; then
>>> + AC_PATH_PROG(MKFONTDIR, mkfontdir, "")
>>> + MKFONTDIR_WARN='echo "** Warning: mkfontdir not run" ; echo "** Run mkfontdir manually on host system"'
>>> +
>>> + if test x"$MKFONTDIR" = x; then
>>> + MKFONTDIR="${MKFONTDIR_WARN} ; echo '** mkfontdir'"
>>> + fi
>>> + else
>>> + XORG_FONT_REQUIRED_PROG(MKFONTDIR, mkfontdir)
>>> + fi
>>> +
>>> + AC_SUBST([MKFONTDIR])
>>> +])
>
> MKFONTDIR_WARN is the command to emit the warning that mkfontdir could be run, MKFONTDIR is the actual command which will be used in the makefile, and is supplied with a directory path, so we need to consume that, and do so by outputting the command we would have run if mkfontdir was available
Ah. I see. Clever.
> Hope that clears things up for you :-)
>
> It looks like this suffers a bit from being patterned after the XORG_FONT_FCCACHE macro a few lines above. This is perhaps wrong, as unlike XORG_FONT_FCCACHE, the warning is only used in one place.
Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>
More information about the xorg-devel
mailing list