[PATCH] Replace libx11 by libxcb
Simon McVittie
simon.mcvittie at collabora.co.uk
Tue Jul 27 06:07:05 PDT 2010
Could you attach (a revised version of) this to a freedesktop bug if you
haven't already, please?
I don't speak X11, so I haven't reviewed the C, but here are some fixes for
the autofoo:
On Tue, 27 Jul 2010 at 13:48:20 +0200, Julien Danjou wrote:
> +if test x$enable_x11 = xyes -o x$enable_x11 = xauto ; then
More portable to deficient shells as:
test x$enable_x11 = xyes || test x$enable_x11 = xauto
> + if ! $have_xcb -a x$enable_x11 = xyes; then
The second condition isn't checked: it's passed as an argument to true(1) or
false(1). I think you meant:
if ! $have_xcb && test x$enable_x11 = xyes; then
> + if ! $have_xcb_aux -a x$enable_x11 = xyes; then
This needs the same change.
> + if $have_xcb -a $have_xcb_aux; then
Again, the second condition isn't checked. I think you meant:
if $have_xcb && $have_xcb_aux; then
Regards,
S
More information about the dbus
mailing list