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