<!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>
On Wed, 2009-12-30 at 08:19 -0800, Dan Nicholson wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Wed, Dec 30, 2009 at 6:32 AM, Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt; wrote:
&gt; Resending.
&gt;
&gt; On Thu, 2009-12-17 at 19:48 -0500, Gaetan Nadon wrote:
&gt;
&gt; Hi,
&gt;
&gt; There are 41 driver modules with the line:
&gt;
&gt; sdkdir=$(pkg-config --variable=sdkdir xorg-server)
&gt;
&gt; Following up on review
&gt; <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>
&gt;
&gt; From Dan Nicholson:
&gt; This should be
&gt;
&gt; sdkdir=`$PKG_CONFIG --variable=sdkdir xorg-server`
&gt;
&gt; 1. The user may need to use a specific pkg-config or pass custom
&gt; arguments to it. Think multiarch or cross-compiling scenarios.
&gt;
&gt; 2. Backticks (``) are supported by all shells while $() command
&gt; substitution is for strictly POSIX conforming shells. We don't need to
&gt; limit ourselves to that subset.
&gt;
&gt; There are 35 modules who are not using that line at all. Rather than making
&gt; dead code more portable, this patch removes this statement. I have done
&gt; extensive grepping and tested with make distcheck:
&gt;
&gt; acecad, aiptek, keyboard, mouse, vmmouse, ast, ati, chips, cirrus, dummy,
&gt; fbdev, geode, i128, i740
&gt; neomagic, newport, nv, rendition, s3, s3virge, siliconmotion, sisusb
&gt; suncg3, suncg6, suncg14, sunffb, sunleo, suntcx, tga, trident, tseng, v4l
&gt; vmware, voodoo, wsfb
&gt;
&gt; I intend to apply this patch to all 35 of them. Other patches for the
&gt; remainder modules will follow.
&gt;
&gt; The reference patch is from video-ati

That seems alright as long as you're definitely sure that sdkdir is
not used anywhere by the driver. Looking at nv, it seems that line
comes from the initial autotools addition.

<A HREF="http://cgit.freedesktop.org/xorg/driver/xf86-video-nv/commit/?id=6ea22f19210448605b9a18bf500c33168a77e77c">http://cgit.freedesktop.org/xorg/driver/xf86-video-nv/commit/?id=6ea22f19210448605b9a18bf500c33168a77e77c</A>

I'd guess the other drivers have that line for the same reason, and in
that case I'd feel more confident that it's not being used most of the
time.

Acked-by: Dan Nicholson &lt;<A HREF="mailto:dbn.lists@gmail.com">dbn.lists@gmail.com</A>&gt;
</PRE>
</BLOCKQUOTE>
<BR>
Thanks, I checked the history for each of them and located if/where it was used and stopped being used. It was the same pattern in each case.
</BODY>
</HTML>