<div class="gmail_quote">2011/2/4 Gaetan Nadon <span dir="ltr">&lt;<a href="mailto:memsize@videotron.ca">memsize@videotron.ca</a>&gt;</span><br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div>You will need to rebase, file has changed since.
</div></blockquote><div><br>Sure.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div><div class="im"><blockquote type="CITE"><pre>
-SUBDIRS = util xcms xlibi18n $(XKB_SUBDIRS)
+SUBDIRS = util xcms xlibi18n $(XKB_SUBDIRS) . xcb
</pre>
</blockquote></div>
Can you explain why libX11 must be built before xcb subdir?<br>
The commit text says it does not use any symbols from libX11.</div></blockquote><div><br>The dependency removal was the last thing that I did. Of course I then forgot to fix this line.<br><br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div><div class="h5"><blockquote type="CITE"><pre> BUILT_SOURCES=ks_tables.h
 CLEANFILES=ks_tables.h ks_tables_h
@@ -13,6 +13,7 @@ AM_CPPFLAGS= \
         -I$(top_srcdir)/include/X11 \
         -I$(top_builddir)/include \
         -I$(top_builddir)/include/X11 \
+        -I$(top_srcdir)/src/xcb \
         -I$(top_srcdir)/src/xcms \
         -I$(top_srcdir)/src/xkb \
         -I$(top_srcdir)/src/xlibi18n \
</pre></blockquote></div></div></div></blockquote><div><br>I noticed that I can get rid of this change by just not moving the Xxcbint.h header file under xcb. The makefile under xcb already has a line to find the header from parent directory into xcb/x11_xcb.c. It seems that leaving Xxcbint.h where it was makes more sense.<br>
<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div><div><div class="h5"><blockquote type="CITE"><pre>+AM_CPPFLAGS= \
+        -I$(top_srcdir)/include/X11 \
+        -I$(top_builddir)/include/X11 \
+        -I$(top_srcdir)/src \
+        -D_BSD_SOURCE
</pre>
</blockquote></div></div>
_BSD_SOURCE is not required due to AC_USE_SYSTEM_EXTENSIONS.<br>
Notice the duplication in the gcc command</div></blockquote><div><br>Ok.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>
<div class="im"><blockquote type="CITE">
<pre>+libX11_xcb_la_SOURCES = x11_xcb.c Xxcbint.h
+libX11_xcb_la_LDFLAGS = -version-number 1:0:0 -no-undefined
</pre>
</blockquote></div>
AM_LDFLAGS would do just fine. Now that each target is separated no need<br>
to use per-target flags</div></blockquote><div><br>Ack.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>
<div><div></div><div class="h5"><br></div></div>
Thanks!<br>
<br>
Reviewed-by: <tt><font color="#1a1a1a">Gaetan Nadon &lt;<a href="mailto:memsize@videotron.ca" target="_blank">memsize@videotron.ca</a>&gt;</font></tt>
</div>

</blockquote></div><br>Thanks, I&#39;ll include your corrections and tag to a pull request.<br><br>-- Rami<br><br>