All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adam Duskett <aduskett@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 4/5] package/gupnp-dlna: add optional dependency for gobject-introspection
Date: Sun, 15 Mar 2020 11:46:26 -0700	[thread overview]
Message-ID: <CAFSsvmpO-ZYPbSmUt_zNOAY97FOwnrT+_987xc6NfYUe7-Ro7w@mail.gmail.com> (raw)
In-Reply-To: <CAPi7W83qicTYKKCZ0tZe0h-6pH8cnxTUoKKOwNjwfRzx6zwXsQ@mail.gmail.com>

Fabrice;

That's quite odd! I must have had gstreamer1 built when I tested this.
I will issue a fix shortly.

On Sun, Mar 15, 2020 at 10:38 AM Fabrice Fontaine
<fontaine.fabrice@gmail.com> wrote:
>
> Dear Adam,
>
> Le dim. 15 mars 2020 ? 18:25, <aduskett@gmail.com> a ?crit :
> >
> > From: Adam Duskett <Aduskett@gmail.com>
> >
> > If gobject-introspection is selected, explicitly set --enable-introspection in
> > the configure options and add a dependency for gobject-introspection.
> >
> > Signed-off-by: Adam Duskett <Aduskett@gmail.com>
> > ---
> >  package/gupnp-dlna/gupnp-dlna.mk | 8 +++++++-
> >  1 file changed, 7 insertions(+), 1 deletion(-)
> >
> > diff --git a/package/gupnp-dlna/gupnp-dlna.mk b/package/gupnp-dlna/gupnp-dlna.mk
> > index 85d30b7cd2..6ce20c15fd 100644
> > --- a/package/gupnp-dlna/gupnp-dlna.mk
> > +++ b/package/gupnp-dlna/gupnp-dlna.mk
> > @@ -16,9 +16,15 @@ GUPNP_DLNA_INSTALL_STAGING = YES
> >  GUPNP_DLNA_DEPENDENCIES = host-pkgconf libglib2 libxml2
> >
> >  GUPNP_DLNA_CONF_OPTS = \
> > -       --disable-introspection \
> >         --disable-legacy-gstreamer-metadata-backend
> >
> > +ifeq ($(BR2_PACKAGE_GOBJECT_INTROSPECTION),y)
> > +GUPNP_DLNA_CONF_OPTS += --enable-introspection
> > +GUPNP_DLNA_DEPENDENCIES += gobject-introspection
> > +else
> > +GUPNP_DLNA_CONF_OPTS += --disable-introspection
> > +endif
> I tried to enable introspection on gupnp-dlna and it fails due to
> missing Gst-1.0.gir which is provided by gstreamer1 under the
> following condition:
> build_gir = gir.found() and not meson.is_cross_build()
>
> So, before enabling introspection on gupnp-dlna, gstreamer1's
> meson.build will have to be patched .
> I tried to do that but I got a build failure.
> > +
> >  ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BASE),y)
> >  GUPNP_DLNA_CONF_OPTS += --enable-gstreamer-metadata-backend
> >  GUPNP_DLNA_DEPENDENCIES += gstreamer1 gst1-plugins-base
> > --
> > 2.24.1
> >
> Best Regards,
>
> Fabrice

  reply	other threads:[~2020-03-15 18:46 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-15 17:24 [Buildroot] [PATCH 1/5] package/libsoup: add optional dependency for gobject-introspection aduskett at gmail.com
2020-03-15 17:24 ` [Buildroot] [PATCH 2/5] package/gssdp: " aduskett at gmail.com
2020-05-11  5:19   ` Thomas Petazzoni
2020-05-11  7:02     ` Fabrice Fontaine
2020-05-11  7:20       ` Fabrice Fontaine
2020-05-11  7:54         ` James Hilliard
2020-03-15 17:24 ` [Buildroot] [PATCH 3/5] package/gupnp: " aduskett at gmail.com
2020-03-15 17:24 ` [Buildroot] [PATCH 4/5] package/gupnp-dlna: " aduskett at gmail.com
2020-03-15 17:39   ` Fabrice Fontaine
2020-03-15 18:46     ` Adam Duskett [this message]
2020-03-15 17:24 ` [Buildroot] [PATCH 5/5] package/gupnp-av: " aduskett at gmail.com
2020-03-22 10:45 ` [Buildroot] [PATCH 1/5] package/libsoup: " Yann E. MORIN

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAFSsvmpO-ZYPbSmUt_zNOAY97FOwnrT+_987xc6NfYUe7-Ro7w@mail.gmail.com \
    --to=aduskett@gmail.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.