Re: [PATCH 7/8] gobject-introspection.bbclass: make additional DEPENDS conditional

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Thu, 2016-03-17 at 22:36 +0100, Martin Jansa wrote:
> On Thu, Mar 17, 2016 at 05:29:59PM +0000, Burton, Ross wrote:
> > On 17 March 2016 at 17:19, Andre McCurdy <armccurdy@gmail.com>
> > wrote:
> > 
> > >  # Generating introspection data depends on a combination of
> > > native and
> > > target
> > >  # introspection tools, and qemu to run the target tools.
> > > -DEPENDS_append = " gobject-introspection gobject-introspection
> > > -native
> > > qemu-native"
> > > +DEPENDS_append = " ${@bb.utils.contains('COMBINED_FEATURES',
> > > 'gobject-introspection-data', 'gobject-introspection
> > > gobject-introspection-native qemu-native', '', d)}"
> > > 
> > 
> > Have you verified this works with g-i disabled and a clean sysroot?
> >   I'm
> > thinking builds will still need at least some of this to provide
> > the m4
> > macros.
> 
> Even with gobject-introspection-data removed from DISTRO_FEATURES I'm
> seeing
> couple recipes (our version of gst-*) failing with:
> 
> > configure:22282: error: possibly undefined macro:
> > PKG_CONFIG_SYSROOT_DIR
> >       If this token and others are legitimate, please use
> > m4_pattern_allow.
> >       See the Autoconf documentation.
> 
> and PKG_CONFIG_SYSROOT_DIR is added by introspection.m4 from g-i
> patchset.

Sounds like a missing pkgconfig depends somewhere. Does the g-i class
inherit pkgconfig?

Cheers,

Richard