Re: [PATCH] cairo_1.10.0: add `glib-2.0-native` to `DEPENDS`

[Tom Rini <tom_rini@mentor.com>]

Paul Menzel wrote:
> Date: Thu, 7 Oct 2010 13:38:44 +0200
> 
> `do_compile()` of `cairo_1.10.0.bb` fails with the following error.
> 
> […]
>         make[4]: Entering directory `/oe/build/angstrom-dev/work/armv7a-angstrom-linux-gnueabi/cairo-1.10.0-r0/cairo-1.10.0/util/cairo-gobject'
>           CC     libcairo_gobject_la-cairo-gobject-enums.lo
>         In file included from cairo-gobject-enums.c:8:
>         cairo-gobject.h:44:25: error: glib-object.h: No such file or directory
>         In file included from cairo-gobject-enums.c:8:
>         cairo-gobject.h:52: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'cairo_gobject_context_get_type'
>         cairo-gobject.h:56: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'cairo_gobject_device_get_type'
>         cairo-gobject.h:60: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'cairo_gobject_pattern_get_type'
>         […]
> 
> `glib-object.h` is only in sysroot of the build host.
> 
>     $ find angstrom-dev/ -name glib-object.h
>     angstrom-dev/sysroots/i686-linux/usr/include/glib-2.0/glib-object.h
> 
> `cairo_1.8.8.bb` builds fine.
> 
> Adding `glib-2.0-native` to `DEPENDS` puts `glib-object.h` also into the sysroot of the target machine and fixes the build.
> 
>     $ find angstrom-dev/ -name glib-object.h
>     angstrom-dev/sysroots/i686-linux/usr/include/glib-2.0/glib-object.h
>     angstrom-dev/sysroots/armv7a-angstrom-linux-gnueabi/usr/include/glib-2.0/glib-object.h
> 
> Build tested with `angstrom-2008.1` for `MACHINE = "beagleboard"`.
> 
> Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
> ---
>  recipes/cairo/cairo_1.10.0.bb |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/recipes/cairo/cairo_1.10.0.bb b/recipes/cairo/cairo_1.10.0.bb
> index 7530cd1..9859b9c 100644
> --- a/recipes/cairo/cairo_1.10.0.bb
> +++ b/recipes/cairo/cairo_1.10.0.bb
> @@ -1,5 +1,7 @@
>  require cairo.inc
>  
> +DEPENDS += "glib-2.0-native"

This should probably go into cairo.inc.  And would you please try and 
experiment?  Diff the bitbake -g files before and after this change and 
make sure we aren't otherwise leaking unwanted changes around into the 
dep graph.  I suspect this will end up being fine...

-- 
Tom Rini
Mentor Graphics Corporation