<!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>
There are 41 driver modules with the line:<BR>
<BLOCKQUOTE>
    <TT>sdkdir=$(pkg-config --variable=sdkdir xorg-server)</TT><BR>
</BLOCKQUOTE>
<BR>
Following up on review <A HREF="http://lists.x.org/archives/xorg-devel/2009-November/003711.html">http://lists.x.org/archives/xorg-devel/2009-November/003711.html</A><BR>
<BR>
<BLOCKQUOTE>
    <TT>From Dan Nicholson:</TT><BR>
    <TT>This should be</TT><BR>
    <BR>
    <TT>sdkdir=`$PKG_CONFIG --variable=sdkdir xorg-server`</TT><BR>
    <BR>
    <TT>1. The user may need to use a specific pkg-config or pass custom</TT><BR>
    <TT>arguments to it. Think multiarch or cross-compiling scenarios.</TT><BR>
    <BR>
    <TT>2. Backticks (``) are supported by all shells while $() command</TT><BR>
    <TT>substitution is for strictly POSIX conforming shells. We don't need to</TT><BR>
    <TT>limit ourselves to that subset.</TT><BR>
</BLOCKQUOTE>
<BR>
There are 12 drivers who are effectively using this line:<BR>
<BR>
<BLOCKQUOTE>
    <TT>joystick, synaptics, glint, </TT><TT><B>intel,</B></TT><TT> mach64, mga, r128, savage, sis, tdfx, xgi, xgixp</TT><BR>
</BLOCKQUOTE>
<BR>
I intend to apply this patch to all 12 of them. Other patches for the remainder modules will follow.<BR>
<BR>
The reference patch is from <B>input-synaptics</B>
</BODY>
</HTML>