<!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&nbsp; XORG_DRIVER_CHECK_EXT() uses the hard-coded executable name &quot;pkg-config&quot; (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&nbsp; 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=&quot;$CFLAGS&quot;
        CFLAGS=&quot;$CFLAGS -I`<B><FONT COLOR="#ff0000">$PKG_CONFIG</FONT></B> --variable=sdkdir xorg-server`&quot;
        cat &gt;conftest.$ac_ext &lt;&lt;_ACEOF
/* confdefs.h.&nbsp; */
_ACEOF
cat confdefs.h &gt;&gt;conftest.$ac_ext
cat &gt;&gt;conftest.$ac_ext &lt;&lt;_ACEOF
/* end confdefs.h.&nbsp; */

#include &quot;xorg-server.h&quot;
#if !defined RANDR
#error RANDR not defined
#endif

int
main ()
{

&nbsp; ;
&nbsp; 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>