Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

Bug#443032: Patch to enable the pixman package

28 views
Skip to first unread message

Synx

unread,
Oct 12, 2007, 5:20:05 PM10/12/07
to
I was shocked to see the wacom input driver hadn't been compiled for the
latest ABI, so I went and compiled it myself, fixed the problems that
were stopping it from compiling. pixman (whatever that is) is a
pkg-config package in the new xorg system, so anything that uses it (aka
wacom-tools) has to do the pkg-config niceties in the configuration and
makefiles. Also if we're using it we should depend on libpixman. So
that's what this patch does. I successfully built and installed the
packages after compiling this patch, though someone who knows more about
pixman might be able to minimize those compiler warnings.

After applying the attached patch (-p0) you have to run aclocal,
automake and autoconf again. I didn't want to include auto-generated
output in my patch.

wacomPixman.patch

Julien Cristau

unread,
Oct 12, 2007, 5:30:09 PM10/12/07
to

That patch is (mostly) wrong. wacom should indeed use pkg-config to get
the necessary cflags, but it doesn't have to care about pixman.

> diff -Naur linuxwacom/configure.in linuxwacom/configure.in
> --- linuxwacom/configure.in 2007-10-12 12:37:40.000000000 -0700
> +++ linuxwacom/configure.in 2007-10-12 12:18:47.000000000 -0700
> @@ -11,6 +11,8 @@
> WCM_LIBWACOMCFG_VER="0:1:0"
> AC_SUBST(WCM_LIBWACOMCFG_VER)
>
> +PKG_PROG_PKG_CONFIG
> +
> dnl Targets
> WCM_PROGS=""
> WCM_LIBS=""
> @@ -60,6 +62,9 @@
> AC_WCM_CHECK_TCL
> AC_WCM_CHECK_TK
>
> +dnl Check for pixman
> +PKG_CHECK_MODULES(PIXMAN,[pixman-1])
> +

should be PKG_CHECK_MODULES(XSERVER, [xorg-server]).

> dnl Check for ncurses
> AC_WCM_CHECK_NCURSES
>
> diff -Naur linuxwacom/src/xdrv/Makefile.am linuxwacom/src/xdrv/Makefile.am
> --- linuxwacom/src/xdrv/Makefile.am 2007-06-05 09:53:31.000000000 -0700
> +++ linuxwacom/src/xdrv/Makefile.am 2007-10-12 12:23:36.000000000 -0700
> @@ -79,6 +79,7 @@
> -DSMART_SCHEDULE -DBUILDDEBUG -DXResExtension \
> -DX_BYTE_ORDER=X_LITTLE_ENDIAN $(XSERVER64) \
> -DNDEBUG -DFUNCPROTO=15 \
> + $(PIXMAN_CFLAGS) \

XSERVER_CFLAGS

> -DNARROWPROTO -DIN_MODULE -DXFree86Module $(LINUX_INPUT) \
> -o $@ -c $(srcdir)/$(subst .o,.c,$@)
>
> diff -Naur debian/control debian/control
> --- debian/control 2007-10-12 12:37:40.000000000 -0700
> +++ debian/control 2007-10-12 12:46:04.000000000 -0700
> @@ -2,7 +2,7 @@
> Section: graphics
> Priority: optional
> Maintainer: Ron Lee <r...@debian.org>
> -Build-Depends: debhelper (>= 5.0.0), po-debconf, xserver-xorg-dev, libxi-dev, libxt-dev, libncurses5-dev
> +Build-Depends: debhelper (>= 5.0.0), po-debconf, xserver-xorg-dev, libxi-dev, libxt-dev, libncurses5-dev, libpixman-1-dev

no.

> Standards-Version: 3.7.2.2
>
> Package: wacom-kernel-source
> @@ -28,7 +28,7 @@
> Package: xserver-xorg-input-wacom
> Section: x11
> Architecture: any
> -Depends: ${shlibs:Depends}
> +Depends: ${shlibs:Depends}, libpixman-1

no.

> Description: X.Org X server -- wacom input driver
> This package provides the driver for wacom tablet devices.
> .
> diff -Naur debian/wacom.control debian/wacom.control
> --- debian/wacom.control 2007-10-12 12:37:40.000000000 -0700
> +++ debian/wacom.control 2007-10-12 12:44:34.000000000 -0700
> @@ -2,7 +2,7 @@
> Section: graphics
> Priority: optional
> Maintainer: Ron Lee <r...@debian.org>
> -Build-Depends: debhelper (>= 5.0.0)
> +Build-Depends: libpixman-1-dev (>= 0.9.5-2) debhelper (>= 5.0.0)
> Standards-Version: 3.7.2.0
>

I'm not sure what this file is for, but this change looks wrong too.

> Package: wacom-kernel-modules-${kpkg:Kernel-Version}

Cheers,
Julien

--
To UNSUBSCRIBE, email to debian-bugs-...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listm...@lists.debian.org

Synx

unread,
Oct 12, 2007, 6:10:09 PM10/12/07
to
Julien Cristau wrote:
> That patch is (mostly) wrong. wacom should indeed use pkg-config to get
> the necessary cflags, but it doesn't have to care about pixman.

The file linuxwacom/src/xdrv/xf86Wacom.c includes the file xf86.h, which
is part of the xorg package. However down the chain of includes:

In file included from /usr/include/xorg/regionstr.h:53,
from /usr/include/xorg/region.h:51,
from /usr/include/xorg/window.h:52,
from /usr/include/xorg/input.h:55,
from /usr/include/xorg/xf86str.h:38,
from /usr/include/xorg/xf86.h:45,
from xf86Wacom.h:66,
from xf86Wacom.c:71:
/usr/include/xorg/miscstruct.h:54:20: error: pixman.h: No such file or
directory

So wacom does use pixman on one hand. On the other hand,


> should be PKG_CHECK_MODULES(XSERVER, [xorg-server]).

You're right on there. I realize now that it's a dependancy of xorg,
which is a dependancy of the wacom tools, but pixman is not a direct
dependancy of wacom. Sorry for the trouble. I'll rewrite the patch then,
no problem.

>> diff -Naur debian/wacom.control debian/wacom.control


> I'm not sure what this file is for, but this change looks wrong too.

The change adds pixman as a build dependancy. But yeah I'm not sure what
that file is for either. I was just being careful.

And... yeah. This new patch successfully compiles too. Without worrying
about pixman.

wacomPixman.patch
0 new messages