[Buildroot] [PATCH v9 3/4] gobject-introspection: new package

[Adam Duskett]

All;

On Tue, Feb 5, 2019 at 9:29 PM Arnout Vandecappelle <arnout at mind.be> wrote:
>
>
>
> On 05/02/2019 19:22, Thomas Petazzoni wrote:
>
> >> +export GIR_EXTRA_LIBS_PATH=$$($$(PKG)_GIR_EXTRA_LIBS_PATH)
> >
> > This is horrible and just doesn't work. You're going to export
> > GIR_EXTRA_LIBS_PATH with the value of whatever is the last parsed
> > package. I don't see how this can work, and it's anyway not the
> > Buildroot way of doing things.
>
>  Of course it works - same way as exporting STAGING_DIR works in PPSH...
>
Yes, this is why I use this syntax.

>  Both $(PKG) and $(<PKG>_GIR_EXTRA_LIBS_PATH) are only expanded when the
> (configure, build, install) rule is executed. And those rules have a PKG=...
> modifier to make sure it is set to the actual package name.
>
>
> > Could you explain what you are trying to achieve ? Infrastructure
> > changes also probably need to be split into a separate patch.
>
>  It makes it possible for an individual package using GIR to add extra libs. I
> guess this will be required by some consumer.
>
Gstreamer specifically needs this.

>  I would only add this as a follow-up patch, to prepare for a package that is
> actually going to use it. That makes the use case much clearer.
>
>  Regards,
>  Arnout