[PATCH v2 wayland] pkg-config: scanner isn't arch-independent

[Arnaud Vrac]

I'm wondering how you are supposed to use the .pc file when cross-compiling
weston for example. Typically you would have the following directory layout:

wayland build for host:

host/usr/bin/wayland-scanner
host/usr/lib/pkgconfig/wayland-scanner.pc
host/usr/share/aclocal/wayland-scanner.m4
host/usr/share/wayland/wayland.dtd
host/usr/share/wayland/wayland-scanner.mk
host/usr/share/wayland/wayland.xml

wayland build for target (configured with --disable-scanner):

target/usr/include/wayland-client.h
target/usr/include/wayland-client-protocol.h
target/usr/include/wayland-cursor.h
target/usr/include/wayland-egl.h
target/usr/include/wayland-server.h
target/usr/include/wayland-server-protocol.h
target/usr/include/wayland-util.h
target/usr/include/wayland-version.h
target/usr/lib/libwayland-client.a
target/usr/lib/libwayland-client.la
target/usr/lib/libwayland-client.so.0.1.0
target/usr/lib/libwayland-cursor.a
target/usr/lib/libwayland-cursor.la
target/usr/lib/libwayland-cursor.so.0.0.0
target/usr/lib/libwayland-server.a
target/usr/lib/libwayland-server.la
target/usr/lib/libwayland-server.so.0.1.0
target/usr/lib/pkgconfig/wayland-client.pc
target/usr/lib/pkgconfig/wayland-cursor.pc
target/usr/lib/pkgconfig/wayland-server.pc
target/usr/share/aclocal/wayland-scanner.m4
target/usr/share/wayland/wayland.dtd
target/usr/share/wayland/wayland-scanner.mk
target/usr/share/wayland/wayland.xml

Weston must be configured with PKG_CONFIG_LIBDIR=target/lib/pkgconfig, so
that the host pc files are not used. This prevents detecting
wayland-scanner.pc in the configure, making the build fail as
wayland-scanner.pc is needed to find the DTD.

What's the right way to make it work ?


On Tue, Nov 19, 2013 at 1:57 PM, Emilio Pozuelo Monfort
<pochu27 at gmail.com>wrote:

> Ping? wayland-scanner hasn't been dropped and the .pc location is still
> bogus.
>
> On 20/08/13 00:58, Kristian Høgsberg wrote:
> > On Fri, Aug 16, 2013 at 03:55:52PM +0100, Daniel Stone wrote:
> >> Hi,
> >>
> >> On 16 August 2013 15:50, David Herrmann <dh.herrmann at gmail.com> wrote:
> >>> On Fri, Aug 16, 2013 at 2:26 PM, Daniel Stone <daniel at fooishbar.org>
> wrote:
> >>>> $(datadir) rather than $(libdir), is for architecture-independent
> data.
> >>>> pkg-config files should only be installed there if they're shareable
> >>>> across architectures, e.g. headers/data only, which wayland-scanner is
> >>>> not.  Move it to $(libdir) with the other pkg-config files.
> >>>
> >>> Yepp, we should definitely use pkgconfig_DATA.
> >>> Reviewed-by: David Herrmann <dh.herrmann at gmail.com>
> >>
> >> Of course, this all becomes irrelevant if the patch to drop
> >> wayland-scanner.{m4,mk,pc} gets dropped.
> >
> > And I'm definitely in favor of just dropping it.
> >
> > Kristian
> >
> >> Cheers,
> >> Daniel
> >> _______________________________________________
> >> wayland-devel mailing list
> >> wayland-devel at lists.freedesktop.org
> >> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> > _______________________________________________
> > wayland-devel mailing list
> > wayland-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> >
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>



-- 
Arnaud Vrac
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20131119/444427b9/attachment-0001.html>