* [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