[PATCH 13-35/41] acecad .. wsfb: configure.ac: remove unused sdkdir=$(pkg-config...) statement
Gaetan Nadon
gaetan.nadon at videotron.ca
Thu Dec 17 16:48:15 PST 2009
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.x.org/archives/xorg-devel/attachments/20091217/ead9c38d/attachment.htm
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-configure.ac-remove-unused-sdkdir-pkg-config.patch
Type: text/x-patch
Size: 0 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-devel/attachments/20091217/ead9c38d/attachment.bin
More information about the xorg-devel
mailing list