<!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.32.2">
</HEAD>
<BODY>
On Wed, 2011-09-14 at 20:17 -0500, Jeremy Huddleston wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
Every module building against xorg-server does not *Require* pixman nor
libpciaccess.  If such modules need pixman or pciaccess, they should be
depending on them directly rather than inheriting a dependency from
xorg-server.

This also moves pixman-1 to Requires.private, so CPPFLAGS is set right to
to satisfy include dependencies but avoid linking needlessly.

</PRE>
</BLOCKQUOTE>
Can you add in the commit text your recommendations for drivers which:<BR>
&nbsp;&nbsp;&nbsp; - are already released and need pciaccess; would they no longer build (or no longer work)?<BR>
&nbsp;&nbsp;&nbsp; - are actively maintained and need pciaccess:<BR>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; - should they bump up xserver version in PKG_CHECK_MODULES as they now would no longer build on previous versions of xserver?<BR>
<BR>
I don't have an opinion on what should or should not happen, just that it is helpful when it is written down.<BR>
A bonus would be a list of drivers that do use pciaccess (maybe not in the commit text!). A complete list of drivers can be found in build.sh.
<BLOCKQUOTE TYPE=CITE>
<PRE>
Signed-off-by: Jeremy Huddleston &lt;<A HREF="mailto:jeremyhu@apple.com">jeremyhu@apple.com</A>&gt;
---
 configure.ac      |   11 ++++++-----
 xorg-server.pc.in |    1 -
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/configure.ac b/configure.ac
index 415ead8..a97da47 100644
--- a/configure.ac
+++ b/configure.ac
@@ -785,11 +785,6 @@ VIDMODEPROTO=&quot;xf86vidmodeproto &gt;= 2.2.99.1&quot;
 WINDOWSWMPROTO=&quot;windowswmproto&quot;
 APPLEWMPROTO=&quot;applewmproto &gt;= 1.4&quot;
 
-dnl Core modules for most extensions, et al.
-SDK_REQUIRED_MODULES=&quot;[xproto &gt;= 7.0.22] [randrproto &gt;= 1.2.99.3] [renderproto &gt;= 0.11] [xextproto &gt;= 7.1.99] [inputproto &gt;= 1.9.99.902] [kbproto &gt;= 1.0.3] fontsproto&quot;
-# Make SDK_REQUIRED_MODULES available for inclusion in xorg-server.pc
-AC_SUBST(SDK_REQUIRED_MODULES)
-
 dnl List of libraries that require a specific version
 LIBAPPLEWM=&quot;applewm &gt;= 1.4&quot;
 LIBDMX=&quot;dmx &gt;= 1.0.99.1&quot;
@@ -811,6 +806,11 @@ dnl specific modules against it
 PKG_CHECK_MODULES(PIXMAN, $LIBPIXMAN)
 REQUIRED_LIBS=&quot;$REQUIRED_LIBS $LIBPIXMAN $LIBXFONT xau&quot;
 
+dnl Core modules for most extensions, et al.
+SDK_REQUIRED_MODULES=&quot;[xproto &gt;= 7.0.22] [randrproto &gt;= 1.2.99.3] [renderproto &gt;= 0.11] [xextproto &gt;= 7.1.99] [inputproto &gt;= 1.9.99.902] [kbproto &gt;= 1.0.3] fontsproto $LIBPIXMAN&quot;
+# Make SDK_REQUIRED_MODULES available for inclusion in xorg-server.pc
+AC_SUBST(SDK_REQUIRED_MODULES)
+
 REQUIRED_MODULES=&quot;[fixesproto &gt;= 5.0] [damageproto &gt;= 1.1] [xcmiscproto &gt;= 1.2.0] [xtrans &gt;= 1.2.2] [bigreqsproto &gt;= 1.1.0] $SDK_REQUIRED_MODULES&quot;
 
 if test &quot;x$CONFIG_UDEV&quot; = xyes &amp;&amp;
@@ -1561,6 +1561,7 @@ if test &quot;x$XORG&quot; = xyes; then
         if test &quot;x$PCI&quot; = xyes; then
 
         PKG_CHECK_MODULES([PCIACCESS], $LIBPCIACCESS)
+        SDK_REQUIRED_MODULES=&quot;$SDK_REQUIRED_MODULES $LIBPCIACCESS&quot;
         XORG_SYS_LIBS=&quot;$XORG_SYS_LIBS $PCIACCESS_LIBS $GLX_SYS_LIBS&quot;
         XORG_CFLAGS=&quot;$XORG_CFLAGS $PCIACCESS_CFLAGS&quot;
 
diff --git a/xorg-server.pc.in b/xorg-server.pc.in
index fb238b5..a98eca8 100644
--- a/xorg-server.pc.in
+++ b/xorg-server.pc.in
@@ -15,7 +15,6 @@ abi_extension=@abi_extension@
 Name: xorg-server
 Description: Modular X.Org X Server
 Version: @PACKAGE_VERSION@
-Requires: pixman-1 pciaccess
 Requires.private: @SDK_REQUIRED_MODULES@
 Cflags: -I${sdkdir} @symbol_visibility@
 Libs: -L${libdir}
</PRE>
</BLOCKQUOTE>
<BR>
Reviewed-by: Gaetan Nadon &lt;memsize@videotron.ca&gt;<BR>
<BR>
</BODY>
</HTML>