On Wed, Dec 30, 2009 at 6:32 AM, Gaetan Nadon <[email protected]> wrote: > Resending. > > On Thu, 2009-12-17 at 19:48 -0500, Gaetan Nadon wrote: > > Hi, > > There are 41 driver modules with the line: > > sdkdir=$(pkg-config --variable=sdkdir xorg-server) > > Following up on review > http://lists.x.org/archives/xorg-devel/2009-November/003711.html > > From Dan Nicholson: > This should be > > sdkdir=`$PKG_CONFIG --variable=sdkdir xorg-server` > > 1. The user may need to use a specific pkg-config or pass custom > arguments to it. Think multiarch or cross-compiling scenarios. > > 2. Backticks (``) are supported by all shells while $() command > substitution is for strictly POSIX conforming shells. We don't need to > limit ourselves to that subset. > > There are 35 modules who are not using that line at all. Rather than making > dead code more portable, this patch removes this statement. I have done > extensive grepping and tested with make distcheck: > > acecad, aiptek, keyboard, mouse, vmmouse, ast, ati, chips, cirrus, dummy, > fbdev, geode, i128, i740 > neomagic, newport, nv, rendition, s3, s3virge, siliconmotion, sisusb > suncg3, suncg6, suncg14, sunffb, sunleo, suntcx, tga, trident, tseng, v4l > vmware, voodoo, wsfb > > I intend to apply this patch to all 35 of them. Other patches for the > remainder modules will follow. > > 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. http://cgit.freedesktop.org/xorg/driver/xf86-video-nv/commit/?id=6ea22f19210448605b9a18bf500c33168a77e77c 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 <[email protected]> _______________________________________________ xorg-devel mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-devel
