<!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 Thu, 2010-09-23 at 15:43 -0700, Keith Packard wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
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?
</PRE>
</BLOCKQUOTE>
<BR>
The only use of the driver path is within a formatted string.<BR>
Should the glx code attempt to load a driver when the dri package is missing,<BR>
the load will fail as the filaname would be "/driverName_dri.so".<BR>
<BR>
<BLOCKQUOTE>
<PRE>
snprintf(filename, sizeof filename,
"%s/%s_dri.so", dri_driver_path, driverName);
</PRE>
</BLOCKQUOTE>
<BR>
<BLOCKQUOTE>
<PRE>
./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);
</PRE>
</BLOCKQUOTE>
<BR>
Thanks for the review,<BR>
<BR>
Gaetan
</BODY>
</HTML>