<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
<META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0">
</HEAD>
<BODY>
Hi,<BR>
<BR>
This is a preventive maintenance patch to fix a dormant bug. It is intended for the git master branch.<BR>
The xorg-server macro XORG_DRIVER_CHECK_EXT() uses the hard-coded executable name "pkg-config" (without a path) rather than the variable PKG_CONFIG which is defined during configuration:<BR>
<BR>
<BLOCKQUOTE>
<TT>checking for pkg-config... /usr/bin/pkg-config</TT><BR>
<TT>checking pkg-config is at least version 0.9.0... yes</TT><BR>
<BR>
</BLOCKQUOTE>
The user is invited to overwrite the value of PKG_CONFIG if need be. This feature will only work if all invocation of this program is done through PKG_CONFIG variable. All driver modules use this macro. Output of ./configure --help:<BR>
<BR>
<BLOCKQUOTE>
<TT>Some influential environment variables:</TT><BR>
<TT>...</TT><BR>
<TT>PKG_CONFIG path to pkg-config utility</TT><BR>
</BLOCKQUOTE>
<BR>
For reference:<BR>
========<BR>
<BR>
This is the code that gets executed when the macro is expanded (example using from driver/acecad):<BR>
<BR>
<BLOCKQUOTE>
<PRE>
# Checks for extensions
        SAVE_CFLAGS="$CFLAGS"
        CFLAGS="$CFLAGS -I`<B><FONT COLOR="#ff0000">$PKG_CONFIG</FONT></B> --variable=sdkdir xorg-server`"
        cat >conftest.$ac_ext <<_ACEOF
/* confdefs.h. */
_ACEOF
cat confdefs.h >>conftest.$ac_ext
cat >>conftest.$ac_ext <<_ACEOF
/* end confdefs.h. */
#include "xorg-server.h"
#if !defined RANDR
#error RANDR not defined
#endif
int
main ()
{
;
return 0;
}
_ACEOF
</PRE>
</BLOCKQUOTE>
<BR>
And the result of the test for RANDR is:<BR>
<BR>
<BLOCKQUOTE>
<PRE>
configure:12628: checking if RANDR is defined
configure:12630: result: yes
</PRE>
</BLOCKQUOTE>
<BR>
<BR>
<BR>
<BR>
</BODY>
</HTML>