[PATCH] xorg-server: pixan-include
Arkadiusz Miskiewicz
arekm at maven.pl
Thu Sep 20 23:18:08 PDT 2007
On Friday 21 of September 2007, Dan Nicholson wrote:
> On 9/20/07, Hanno Böck <ml at hboeck.de> wrote:
> > miscstruct.h contains an include of pixman.h without a path.
> >
> > This will e.g. cause synaptics build to fail. Attached patch for
> > xorg-xserver fixes it. Please apply.
>
> Not if they're using pkg-config:
>
> $ pkg-config --cflags pixman-1
> -I/usr/include/pixman-1
The problem is that now every external app that uses these headers needs to
use pkgconfig to get pixman -I flags.
Why the include wasn't written as #include <pixman-1/whatever.h>?
pixman-1 name was choosen afaik to be able to easily change api/abi in future
and still have posibility go have both (old and new) pixman packages
installed (without conflicts). Since now xserver uses pixman-1 API only and
it simply could directly include <pixman-1/xyz> avoiding all the problems.
(That's what we did in PLD)
> Dan
--
Arkadiusz Miśkiewicz PLD/Linux Team
arekm / maven.pl http://ftp.pld-linux.org/
More information about the xorg
mailing list