[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