<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 14-03-03 10:08 AM, Jon TURNEY wrote:<br>
</div>
<blockquote
cite="mid:1393859302-5616-1-git-send-email-jon.turney@dronecode.org.uk"
type="cite">
<pre wrap="">Build fbcmap_mi.c once, rather than for each DDX, and make it part of libfb or
libwfb convenience library.
Since 84e8de1271bb11b5b4b9747ae4647f47333a8ab7 we don't have fbcmap.c
This is a sort of revert of 7ecc2d526c4ea5db2589644a2fec0daf71df36da
</pre>
</blockquote>
<br>
kdrive fbdev fails because <tt>libkdrivestubs is added in
configure.ac. Consider this change:</tt><br>
<br>
<blockquote><tt>diff --git a/configure.ac b/configure.ac</tt><tt><br>
</tt><tt>index 162c0cf..d7a55a4 100644</tt><tt><br>
</tt><tt>--- a/configure.ac</tt><tt><br>
</tt><tt>+++ b/configure.ac</tt><tt><br>
</tt><tt>@@ -2400,8 +2400,7 @@ if test "$KDRIVE" = yes; then</tt><tt><br>
</tt><tt> fi</tt><tt><br>
</tt><tt> ;;</tt><tt><br>
</tt><tt> esac</tt><tt><br>
</tt><tt>-
KDRIVE_STUB_LIB='$(top_builddir)/hw/kdrive/src/libkdrivestubs.la'</tt><tt><br>
</tt><tt>- KDRIVE_LOCAL_LIBS="$MAIN_LIB $DIX_LIB $KDRIVE_LIB
$KDRIVE_STUB_LIB"</tt><tt><br>
</tt><tt>+ KDRIVE_LOCAL_LIBS="$MAIN_LIB $DIX_LIB $KDRIVE_LIB"</tt><tt><br>
</tt><tt> KDRIVE_LOCAL_LIBS="$KDRIVE_LOCAL_LIBS $FB_LIB
$MI_LIB $KDRIVE_PURE_LIBS"</tt><tt><br>
</tt><tt> KDRIVE_LOCAL_LIBS="$KDRIVE_LOCAL_LIBS
$KDRIVE_OS_LIB"</tt><tt><br>
</tt><tt> KDRIVE_LIBS="$KDRIVE_LOCAL_LIBS $XSERVER_SYS_LIBS
$GLX_SYS_LIBS $DLOPEN_LIB</tt><tt><br>
</tt></blockquote>
I used these configure options:<br>
<blockquote><tt>--enable-dmx --enable-kdrive --enable-xfake
--enable-xfbdev --enable-xnest --enable-glamor</tt><br>
</blockquote>
Note that kdrive ephyr fails to compile without glamor enabled.<br>
<blockquote><tt> XNEST_LIBS="$FB_LIB ...</tt><tt><br>
</tt></blockquote>
<br>
It looks like Xnest does not need libfb at all which is set in
configure.ac. It does not cause any problem, however.<br>
<br>
Thanks for the patch!<br>
</body>
</html>