Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-plugins-good: add option to enable/disable shout2 plugin
@ 2019-07-05  6:50 Arnold Bloemert
  2019-07-05 15:18 ` Peter Seiderer
  2019-07-05 21:50 ` [Buildroot] [PATCH v2] package/gst1-plugins-good: add shout2 plugin option Arnold Bloemert
  0 siblings, 2 replies; 7+ messages in thread
From: Arnold Bloemert @ 2019-07-05  6:50 UTC (permalink / raw)
  To: buildroot

Currently there is no way to turn the shout2 plugin on.
Create a menu entry allowing users to disable or enable the plugin,
like we have for other features (for example the Pulseaudio plugin).

Signed-off-by: Arnold Bloemert <abloemert@gmail.com>
---
 package/gstreamer1/gst1-plugins-good/Config.in           | 6 ++++++
 .../gstreamer1/gst1-plugins-good/gst1-plugins-good.mk    | 9 +++++++--
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
index 433084fd6b..ce85388eb0 100644
--- a/package/gstreamer1/gst1-plugins-good/Config.in
+++ b/package/gstreamer1/gst1-plugins-good/Config.in
@@ -300,6 +300,12 @@ comment "qmlgl (qt5) plugin needs an OpenGL-capable Qt5 backend"
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
 	depends on !BR2_PACKAGE_QT5_GL_AVAILABLE && !BR2_PACKAGE_QT5_VERSION_LATEST
 
+config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
+        bool "shout2"
+        select BR2_PACKAGE_LIBSHOUT
+        help
+          Elements to send data to an icecast server using libshout2
+
 config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
 	bool "v4l2"
 	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
index 829d27a2da..0a6cc77208 100644
--- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
+++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
@@ -25,8 +25,7 @@ GST1_PLUGINS_GOOD_CONF_OPTS = \
 # Options which require currently unpackaged libraries
 GST1_PLUGINS_GOOD_CONF_OPTS += \
 	--disable-libdv \
-	--disable-dv1394 \
-	--disable-shout2
+	--disable-dv1394
 
 GST1_PLUGINS_GOOD_DEPENDENCIES = gstreamer1 gst1-plugins-base
 
@@ -343,6 +342,12 @@ else
 GST1_PLUGINS_GOOD_CONF_OPTS += --disable-qt
 endif
 
+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2),y)
+GST1_PLUGINS_GOOD_CONF_OPTS += --enable-shout2
+else
+GST1_PLUGINS_GOOD_CONF_OPTS += --disable-shout2
+endif
+
 ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
 GST1_PLUGINS_GOOD_CONF_OPTS += --enable-gst_v4l2
 else
-- 
2.22.0

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-plugins-good: add option to enable/disable shout2 plugin
  2019-07-05  6:50 [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-plugins-good: add option to enable/disable shout2 plugin Arnold Bloemert
@ 2019-07-05 15:18 ` Peter Seiderer
  2019-07-05 21:48   ` Arnold Bloemert
  2019-07-05 21:50 ` [Buildroot] [PATCH v2] package/gst1-plugins-good: add shout2 plugin option Arnold Bloemert
  1 sibling, 1 reply; 7+ messages in thread
From: Peter Seiderer @ 2019-07-05 15:18 UTC (permalink / raw)
  To: buildroot

Hello Arnould,

a little shorter subject line as the following should be enough:

   package/gst1-plugins-good: add shout2 plugin option


On Fri,  5 Jul 2019 08:50:15 +0200, Arnold Bloemert <abloemert@gmail.com> wrote:

> Currently there is no way to turn the shout2 plugin on.
> Create a menu entry allowing users to disable or enable the plugin,
> like we have for other features (for example the Pulseaudio plugin).

Given text is more suitable for a cover letter ;-), no special justification
for adding a missing option needed....

>
> Signed-off-by: Arnold Bloemert <abloemert@gmail.com>
> ---
>  package/gstreamer1/gst1-plugins-good/Config.in           | 6 ++++++
>  .../gstreamer1/gst1-plugins-good/gst1-plugins-good.mk    | 9 +++++++--
>  2 files changed, 13 insertions(+), 2 deletions(-)
>
> diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
> index 433084fd6b..ce85388eb0 100644
> --- a/package/gstreamer1/gst1-plugins-good/Config.in
> +++ b/package/gstreamer1/gst1-plugins-good/Config.in
> @@ -300,6 +300,12 @@ comment "qmlgl (qt5) plugin needs an OpenGL-capable Qt5 backend"
>  	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
>  	depends on !BR2_PACKAGE_QT5_GL_AVAILABLE && !BR2_PACKAGE_QT5_VERSION_LATEST
>
> +config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
> +        bool "shout2"
> +        select BR2_PACKAGE_LIBSHOUT
> +        help
> +          Elements to send data to an icecast server using libshout2
> +
>  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
>  	bool "v4l2"
>  	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
> diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> index 829d27a2da..0a6cc77208 100644
> --- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> +++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> @@ -25,8 +25,7 @@ GST1_PLUGINS_GOOD_CONF_OPTS = \
>  # Options which require currently unpackaged libraries
>  GST1_PLUGINS_GOOD_CONF_OPTS += \
>  	--disable-libdv \
> -	--disable-dv1394 \
> -	--disable-shout2
> +	--disable-dv1394
>
>  GST1_PLUGINS_GOOD_DEPENDENCIES = gstreamer1 gst1-plugins-base
>
> @@ -343,6 +342,12 @@ else
>  GST1_PLUGINS_GOOD_CONF_OPTS += --disable-qt
>  endif
>
> +ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2),y)
> +GST1_PLUGINS_GOOD_CONF_OPTS += --enable-shout2

...missing:

    GST1_PLUGINS_GOOD_DEPENDENCIES += libshout

Otherwise looks good!

Regards,
Peter

> +else
> +GST1_PLUGINS_GOOD_CONF_OPTS += --disable-shout2
> +endif
> +
>  ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
>  GST1_PLUGINS_GOOD_CONF_OPTS += --enable-gst_v4l2
>  else

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-plugins-good: add option to enable/disable shout2 plugin
  2019-07-05 15:18 ` Peter Seiderer
@ 2019-07-05 21:48   ` Arnold Bloemert
  2019-07-07 18:47     ` Peter Seiderer
  0 siblings, 1 reply; 7+ messages in thread
From: Arnold Bloemert @ 2019-07-05 21:48 UTC (permalink / raw)
  To: buildroot

Hi Peter,

I tried to create a patch in the same style as the patch for jack by Adam,
but I like your proposal to simplify.

The missing dependency in the .mk file was a stupid mistake, thanks for
noticing it!

I'll create a new revision of the patch, contributing to Buildroot is new
for me, so hopefully I'm doing it right.

Kind regards,

Arnold

Op vr 5 jul. 2019 om 17:18 schreef Peter Seiderer <ps.report@gmx.net>:

> Hello Arnould,
>
> a little shorter subject line as the following should be enough:
>
>    package/gst1-plugins-good: add shout2 plugin option
>
>
> On Fri,  5 Jul 2019 08:50:15 +0200, Arnold Bloemert <abloemert@gmail.com>
> wrote:
>
> > Currently there is no way to turn the shout2 plugin on.
> > Create a menu entry allowing users to disable or enable the plugin,
> > like we have for other features (for example the Pulseaudio plugin).
>
> Given text is more suitable for a cover letter ;-), no special
> justification
> for adding a missing option needed....
>
> >
> > Signed-off-by: Arnold Bloemert <abloemert@gmail.com>
> > ---
> >  package/gstreamer1/gst1-plugins-good/Config.in           | 6 ++++++
> >  .../gstreamer1/gst1-plugins-good/gst1-plugins-good.mk    | 9 +++++++--
> >  2 files changed, 13 insertions(+), 2 deletions(-)
> >
> > diff --git a/package/gstreamer1/gst1-plugins-good/Config.in
> b/package/gstreamer1/gst1-plugins-good/Config.in
> > index 433084fd6b..ce85388eb0 100644
> > --- a/package/gstreamer1/gst1-plugins-good/Config.in
> > +++ b/package/gstreamer1/gst1-plugins-good/Config.in
> > @@ -300,6 +300,12 @@ comment "qmlgl (qt5) plugin needs an OpenGL-capable
> Qt5 backend"
> >       depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
> >       depends on !BR2_PACKAGE_QT5_GL_AVAILABLE &&
> !BR2_PACKAGE_QT5_VERSION_LATEST
> >
> > +config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
> > +        bool "shout2"
> > +        select BR2_PACKAGE_LIBSHOUT
> > +        help
> > +          Elements to send data to an icecast server using libshout2
> > +
> >  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
> >       bool "v4l2"
> >       depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
> > diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> > index 829d27a2da..0a6cc77208 100644
> > --- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> > +++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> > @@ -25,8 +25,7 @@ GST1_PLUGINS_GOOD_CONF_OPTS = \
> >  # Options which require currently unpackaged libraries
> >  GST1_PLUGINS_GOOD_CONF_OPTS += \
> >       --disable-libdv \
> > -     --disable-dv1394 \
> > -     --disable-shout2
> > +     --disable-dv1394
> >
> >  GST1_PLUGINS_GOOD_DEPENDENCIES = gstreamer1 gst1-plugins-base
> >
> > @@ -343,6 +342,12 @@ else
> >  GST1_PLUGINS_GOOD_CONF_OPTS += --disable-qt
> >  endif
> >
> > +ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2),y)
> > +GST1_PLUGINS_GOOD_CONF_OPTS += --enable-shout2
>
> ...missing:
>
>     GST1_PLUGINS_GOOD_DEPENDENCIES += libshout
>
> Otherwise looks good!
>
> Regards,
> Peter
>
> > +else
> > +GST1_PLUGINS_GOOD_CONF_OPTS += --disable-shout2
> > +endif
> > +
> >  ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
> >  GST1_PLUGINS_GOOD_CONF_OPTS += --enable-gst_v4l2
> >  else
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20190705/6961eb0c/attachment.html>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH v2] package/gst1-plugins-good: add shout2 plugin option
  2019-07-05  6:50 [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-plugins-good: add option to enable/disable shout2 plugin Arnold Bloemert
  2019-07-05 15:18 ` Peter Seiderer
@ 2019-07-05 21:50 ` Arnold Bloemert
  2019-07-07 19:03   ` Peter Seiderer
  2019-08-01 19:13   ` Arnout Vandecappelle
  1 sibling, 2 replies; 7+ messages in thread
From: Arnold Bloemert @ 2019-07-05 21:50 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Arnold Bloemert <abloemert@gmail.com>

---
Changes v1 -> v2:
  - add dependency to .mk file and simplify commit message (suggested by Peter Seiderer)

Signed-off-by: Arnold Bloemert <abloemert@gmail.com>
---
 package/gstreamer1/gst1-plugins-good/Config.in         |  6 ++++++
 .../gstreamer1/gst1-plugins-good/gst1-plugins-good.mk  | 10 ++++++++--
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
index 433084fd6b..ce85388eb0 100644
--- a/package/gstreamer1/gst1-plugins-good/Config.in
+++ b/package/gstreamer1/gst1-plugins-good/Config.in
@@ -300,6 +300,12 @@ comment "qmlgl (qt5) plugin needs an OpenGL-capable Qt5 backend"
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
 	depends on !BR2_PACKAGE_QT5_GL_AVAILABLE && !BR2_PACKAGE_QT5_VERSION_LATEST
 
+config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
+        bool "shout2"
+        select BR2_PACKAGE_LIBSHOUT
+        help
+          Elements to send data to an icecast server using libshout2
+
 config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
 	bool "v4l2"
 	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
index 829d27a2da..3c9f0d5605 100644
--- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
+++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
@@ -25,8 +25,7 @@ GST1_PLUGINS_GOOD_CONF_OPTS = \
 # Options which require currently unpackaged libraries
 GST1_PLUGINS_GOOD_CONF_OPTS += \
 	--disable-libdv \
-	--disable-dv1394 \
-	--disable-shout2
+	--disable-dv1394
 
 GST1_PLUGINS_GOOD_DEPENDENCIES = gstreamer1 gst1-plugins-base
 
@@ -343,6 +342,13 @@ else
 GST1_PLUGINS_GOOD_CONF_OPTS += --disable-qt
 endif
 
+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2),y)
+GST1_PLUGINS_GOOD_CONF_OPTS += --enable-shout2
+GST1_PLUGINS_GOOD_DEPENDENCIES += libshout
+else
+GST1_PLUGINS_GOOD_CONF_OPTS += --disable-shout2
+endif
+
 ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
 GST1_PLUGINS_GOOD_CONF_OPTS += --enable-gst_v4l2
 else
-- 
2.22.0

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-plugins-good: add option to enable/disable shout2 plugin
  2019-07-05 21:48   ` Arnold Bloemert
@ 2019-07-07 18:47     ` Peter Seiderer
  0 siblings, 0 replies; 7+ messages in thread
From: Peter Seiderer @ 2019-07-07 18:47 UTC (permalink / raw)
  To: buildroot

Hello Arnold,

On Fri, 5 Jul 2019 23:48:33 +0200, Arnold Bloemert <abloemert@gmail.com> wrote:

> Hi Peter,
>
> I tried to create a patch in the same style as the patch for jack by Adam,
> but I like your proposal to simplify.

Ohhh, I see (the comments are mainly a matter of taste)...

>
> The missing dependency in the .mk file was a stupid mistake, thanks for
> noticing it!

No problem (catching it is the main purpose of the review process ;-) )...

>
> I'll create a new revision of the patch, contributing to Buildroot is new
> for me, so hopefully I'm doing it right.

Many thanks for contributing, every patch is welcome! Yours is nearly perfect,
only one nitpick (see comment on v2 of the patch)...

Regards,
Peter

>
> Kind regards,
>
> Arnold
>
> Op vr 5 jul. 2019 om 17:18 schreef Peter Seiderer <ps.report@gmx.net>:
>
> > Hello Arnould,
> >
> > a little shorter subject line as the following should be enough:
> >
> >    package/gst1-plugins-good: add shout2 plugin option
> >
> >
> > On Fri,  5 Jul 2019 08:50:15 +0200, Arnold Bloemert <abloemert@gmail.com>
> > wrote:
> >
> > > Currently there is no way to turn the shout2 plugin on.
> > > Create a menu entry allowing users to disable or enable the plugin,
> > > like we have for other features (for example the Pulseaudio plugin).
> >
> > Given text is more suitable for a cover letter ;-), no special
> > justification
> > for adding a missing option needed....
> >
> > >
> > > Signed-off-by: Arnold Bloemert <abloemert@gmail.com>
> > > ---
> > >  package/gstreamer1/gst1-plugins-good/Config.in           | 6 ++++++
> > >  .../gstreamer1/gst1-plugins-good/gst1-plugins-good.mk    | 9 +++++++--
> > >  2 files changed, 13 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/package/gstreamer1/gst1-plugins-good/Config.in
> > b/package/gstreamer1/gst1-plugins-good/Config.in
> > > index 433084fd6b..ce85388eb0 100644
> > > --- a/package/gstreamer1/gst1-plugins-good/Config.in
> > > +++ b/package/gstreamer1/gst1-plugins-good/Config.in
> > > @@ -300,6 +300,12 @@ comment "qmlgl (qt5) plugin needs an OpenGL-capable
> > Qt5 backend"
> > >       depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
> > >       depends on !BR2_PACKAGE_QT5_GL_AVAILABLE &&
> > !BR2_PACKAGE_QT5_VERSION_LATEST
> > >
> > > +config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
> > > +        bool "shout2"
> > > +        select BR2_PACKAGE_LIBSHOUT
> > > +        help
> > > +          Elements to send data to an icecast server using libshout2
> > > +
> > >  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
> > >       bool "v4l2"
> > >       depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
> > > diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> > b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> > > index 829d27a2da..0a6cc77208 100644
> > > --- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> > > +++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> > > @@ -25,8 +25,7 @@ GST1_PLUGINS_GOOD_CONF_OPTS = \
> > >  # Options which require currently unpackaged libraries
> > >  GST1_PLUGINS_GOOD_CONF_OPTS += \
> > >       --disable-libdv \
> > > -     --disable-dv1394 \
> > > -     --disable-shout2
> > > +     --disable-dv1394
> > >
> > >  GST1_PLUGINS_GOOD_DEPENDENCIES = gstreamer1 gst1-plugins-base
> > >
> > > @@ -343,6 +342,12 @@ else
> > >  GST1_PLUGINS_GOOD_CONF_OPTS += --disable-qt
> > >  endif
> > >
> > > +ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2),y)
> > > +GST1_PLUGINS_GOOD_CONF_OPTS += --enable-shout2
> >
> > ...missing:
> >
> >     GST1_PLUGINS_GOOD_DEPENDENCIES += libshout
> >
> > Otherwise looks good!
> >
> > Regards,
> > Peter
> >
> > > +else
> > > +GST1_PLUGINS_GOOD_CONF_OPTS += --disable-shout2
> > > +endif
> > > +
> > >  ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
> > >  GST1_PLUGINS_GOOD_CONF_OPTS += --enable-gst_v4l2
> > >  else
> >
> >

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH v2] package/gst1-plugins-good: add shout2 plugin option
  2019-07-05 21:50 ` [Buildroot] [PATCH v2] package/gst1-plugins-good: add shout2 plugin option Arnold Bloemert
@ 2019-07-07 19:03   ` Peter Seiderer
  2019-08-01 19:13   ` Arnout Vandecappelle
  1 sibling, 0 replies; 7+ messages in thread
From: Peter Seiderer @ 2019-07-07 19:03 UTC (permalink / raw)
  To: buildroot

Hello Arnold,

On Fri,  5 Jul 2019 23:50:38 +0200, Arnold Bloemert <abloemert@gmail.com> wrote:

> Signed-off-by: Arnold Bloemert <abloemert@gmail.com>
>
> ---
> Changes v1 -> v2:
>   - add dependency to .mk file and simplify commit message (suggested by Peter Seiderer)
>
> Signed-off-by: Arnold Bloemert <abloemert@gmail.com>
> ---
>  package/gstreamer1/gst1-plugins-good/Config.in         |  6 ++++++
>  .../gstreamer1/gst1-plugins-good/gst1-plugins-good.mk  | 10 ++++++++--
>  2 files changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
> index 433084fd6b..ce85388eb0 100644d3b0edbba14c343b299b96dfdeeebc465a71ba78
> --- a/package/gstreamer1/gst1-plugins-good/Config.in
> +++ b/package/gstreamer1/gst1-plugins-good/Config.in
> @@ -300,6 +300,12 @@ comment "qmlgl (qt5) plugin needs an OpenGL-capable Qt5 backend"
>  	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
>  	depends on !BR2_PACKAGE_QT5_GL_AVAILABLE && !BR2_PACKAGE_QT5_VERSION_LATEST
>
> +config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
> +        bool "shout2"
> +        select BR2_PACKAGE_LIBSHOUT
> +        help
> +          Elements to send data to an icecast server using libshout2
> +
>  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
>  	bool "v4l2"
>  	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
> diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> index 829d27a2da..3c9f0d5605 100644
> --- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> +++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> @@ -25,8 +25,7 @@ GST1_PLUGINS_GOOD_CONF_OPTS = \
>  # Options which require currently unpackaged libraries
>  GST1_PLUGINS_GOOD_CONF_OPTS += \
>  	--disable-libdv \
> -	--disable-dv1394 \
> -	--disable-shout2
> +	--disable-dv1394
>
>  GST1_PLUGINS_GOOD_DEPENDENCIES = gstreamer1 gst1-plugins-base
>
> @@ -343,6 +342,13 @@ else
>  GST1_PLUGINS_GOOD_CONF_OPTS += --disable-qt
>  endif
>
> +ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2),y)
> +GST1_PLUGINS_GOOD_CONF_OPTS += --enable-shout2
> +GST1_PLUGINS_GOOD_DEPENDENCIES += libshout
> +else
> +GST1_PLUGINS_GOOD_CONF_OPTS += --disable-shout2
> +endif
> +
>  ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
>  GST1_PLUGINS_GOOD_CONF_OPTS += --enable-gst_v4l2
>  else


The other gstreamer1 files state for the options (see e.g. [1]):

   # These plugins are listed in the order from ./configure --help

For shout2 this should be after qt (right) and before soup (false), but
taking a look at Config.in/gst1-plugins-good.mk the option order is not
very strict....so I am o.k. with your patch ;-), you can add my

Reviewed-by: Peter Seiderer <ps.report@gmx.net>

Regards,
Peter

[1] https://git.buildroot.net/buildroot/tree/package/gstreamer1/gst1-plugins-base/gst1-plugins-base.mk#n27

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH v2] package/gst1-plugins-good: add shout2 plugin option
  2019-07-05 21:50 ` [Buildroot] [PATCH v2] package/gst1-plugins-good: add shout2 plugin option Arnold Bloemert
  2019-07-07 19:03   ` Peter Seiderer
@ 2019-08-01 19:13   ` Arnout Vandecappelle
  1 sibling, 0 replies; 7+ messages in thread
From: Arnout Vandecappelle @ 2019-08-01 19:13 UTC (permalink / raw)
  To: buildroot



On 05/07/2019 23:50, Arnold Bloemert wrote:
[snip]
> +config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
> +        bool "shout2"
> +        select BR2_PACKAGE_LIBSHOUT
> +        help
> +          Elements to send data to an icecast server using libshout2

 The indentation was wrong here. Please use utils/check-package to check things
before sending.

 I've fixed it and applied to master, so no need to resend, thanks.

 Regards,
 Arnout

[snip]

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2019-08-01 19:13 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-05  6:50 [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-plugins-good: add option to enable/disable shout2 plugin Arnold Bloemert
2019-07-05 15:18 ` Peter Seiderer
2019-07-05 21:48   ` Arnold Bloemert
2019-07-07 18:47     ` Peter Seiderer
2019-07-05 21:50 ` [Buildroot] [PATCH v2] package/gst1-plugins-good: add shout2 plugin option Arnold Bloemert
2019-07-07 19:03   ` Peter Seiderer
2019-08-01 19:13   ` Arnout Vandecappelle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox