On Thu, 2010-09-23 at 15:43 -0700, Keith Packard wrote:

> What's the state of this patch?  Seems like it should default to
> whatever the old default was in the absence of a dri.pc file? Or is
> everyone happy with the patch as-is?
> 


The only use of the driver path is within a formatted string.
Should the glx code attempt to load a driver when the dri package is
missing,
the load will fail as the filaname would be "/driverName_dri.so".


            snprintf(filename, sizeof filename,
                     "%s/%s_dri.so", dri_driver_path, driverName);



        ./glx/glxdriswrast.c:static const char dri_driver_path[] = 
DRI_DRIVER_PATH;
        ./glx/glxdriswrast.c:        "%s/%s_dri.so", dri_driver_path, 
driverName);
        ./glx/glxdri2.c:static const char dri_driver_path[] = DRI_DRIVER_PATH;
        ./glx/glxdri2.c:             "%s/%s_dri.so", dri_driver_path, 
driverName);
        ./glx/glxdri.c:static const char dri_driver_path[] = DRI_DRIVER_PATH;
        ./glx/glxdri.c:             dri_driver_path, driverName);


Thanks for the review,

Gaetan

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to