* [Buildroot] [PATCH/next v1] package/gst1-plugins-base: add tools option
@ 2020-03-03 16:47 Peter Seiderer
2020-03-05 22:34 ` Thomas Petazzoni
0 siblings, 1 reply; 3+ messages in thread
From: Peter Seiderer @ 2020-03-03 16:47 UTC (permalink / raw)
To: buildroot
Add tools option to disable building/installing of
gst-discoverer, gst-device-monitor and gst-play commandline
tools (analog BR2_PACKAGE_GSTREAMER1_INSTALL_TOOLS).
Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
package/gstreamer1/gst1-plugins-base/Config.in | 7 +++++++
package/gstreamer1/gst1-plugins-base/gst1-plugins-base.mk | 6 ++++++
2 files changed, 13 insertions(+)
diff --git a/package/gstreamer1/gst1-plugins-base/Config.in b/package/gstreamer1/gst1-plugins-base/Config.in
index f60d540337..3a3694b72e 100644
--- a/package/gstreamer1/gst1-plugins-base/Config.in
+++ b/package/gstreamer1/gst1-plugins-base/Config.in
@@ -10,6 +10,13 @@ menuconfig BR2_PACKAGE_GST1_PLUGINS_BASE
if BR2_PACKAGE_GST1_PLUGINS_BASE
+config BR2_PACKAGE_GST1_PLUGINS_BASE_INSTALL_TOOLS
+ bool "install gst-discoverer & gst-device-monitor & gst-play
+ default y
+ help
+ Install the gst-discoverer, gst-device-monitor and gst-play
+ tools.
+
comment "dependency-less plugins"
config BR2_PACKAGE_GST1_PLUGINS_BASE_PLUGIN_ADDER
diff --git a/package/gstreamer1/gst1-plugins-base/gst1-plugins-base.mk b/package/gstreamer1/gst1-plugins-base/gst1-plugins-base.mk
index 1781a5d461..ccecbb3c08 100644
--- a/package/gstreamer1/gst1-plugins-base/gst1-plugins-base.mk
+++ b/package/gstreamer1/gst1-plugins-base/gst1-plugins-base.mk
@@ -26,6 +26,12 @@ GST1_PLUGINS_BASE_CONF_OPTS += \
-Dlibvisual=disabled \
-Diso-codes=disabled
+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BASE_INSTALL_TOOLS),y)
+GST1_PLUGINS_BASE_CONF_OPTS += -Dtools=enabled
+else
+GST1_PLUGINS_BASE_CONF_OPTS += -Dtools=disabled
+endif
+
GST1_PLUGINS_BASE_DEPENDENCIES = gstreamer1 $(TARGET_NLS_DEPENDENCIES)
GST1_PLUGINS_BASE_LDFLAGS = $(TARGET_LDFLAGS) $(TARGET_NLS_LIBS)
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH/next v1] package/gst1-plugins-base: add tools option
2020-03-03 16:47 [Buildroot] [PATCH/next v1] package/gst1-plugins-base: add tools option Peter Seiderer
@ 2020-03-05 22:34 ` Thomas Petazzoni
2020-03-06 14:34 ` Peter Seiderer
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni @ 2020-03-05 22:34 UTC (permalink / raw)
To: buildroot
On Tue, 3 Mar 2020 17:47:02 +0100
Peter Seiderer <ps.report@gmx.net> wrote:
> Add tools option to disable building/installing of
> gst-discoverer, gst-device-monitor and gst-play commandline
> tools (analog BR2_PACKAGE_GSTREAMER1_INSTALL_TOOLS).
>
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
> package/gstreamer1/gst1-plugins-base/Config.in | 7 +++++++
> package/gstreamer1/gst1-plugins-base/gst1-plugins-base.mk | 6 ++++++
> 2 files changed, 13 insertions(+)
>
> diff --git a/package/gstreamer1/gst1-plugins-base/Config.in b/package/gstreamer1/gst1-plugins-base/Config.in
> index f60d540337..3a3694b72e 100644
> --- a/package/gstreamer1/gst1-plugins-base/Config.in
> +++ b/package/gstreamer1/gst1-plugins-base/Config.in
> @@ -10,6 +10,13 @@ menuconfig BR2_PACKAGE_GST1_PLUGINS_BASE
>
> if BR2_PACKAGE_GST1_PLUGINS_BASE
>
> +config BR2_PACKAGE_GST1_PLUGINS_BASE_INSTALL_TOOLS
> + bool "install gst-discoverer & gst-device-monitor & gst-play
Missing closing double quote. Also, I changed this to "install tools",
to keep the prompt shorter.
> + default y
I know gstreamer1 does this, but this "default y" is really not in line
with the typical Buildroot policy of minimalism, so I dropped this.
Applied to next with those changes.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH/next v1] package/gst1-plugins-base: add tools option
2020-03-05 22:34 ` Thomas Petazzoni
@ 2020-03-06 14:34 ` Peter Seiderer
0 siblings, 0 replies; 3+ messages in thread
From: Peter Seiderer @ 2020-03-06 14:34 UTC (permalink / raw)
To: buildroot
Hello Thomas,
On Thu, 5 Mar 2020 23:34:10 +0100, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:
> On Tue, 3 Mar 2020 17:47:02 +0100
> Peter Seiderer <ps.report@gmx.net> wrote:
>
> > Add tools option to disable building/installing of
> > gst-discoverer, gst-device-monitor and gst-play commandline
> > tools (analog BR2_PACKAGE_GSTREAMER1_INSTALL_TOOLS).
> >
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> > ---
> > package/gstreamer1/gst1-plugins-base/Config.in | 7 +++++++
> > package/gstreamer1/gst1-plugins-base/gst1-plugins-base.mk | 6 ++++++
> > 2 files changed, 13 insertions(+)
> >
> > diff --git a/package/gstreamer1/gst1-plugins-base/Config.in b/package/gstreamer1/gst1-plugins-base/Config.in
> > index f60d540337..3a3694b72e 100644
> > --- a/package/gstreamer1/gst1-plugins-base/Config.in
> > +++ b/package/gstreamer1/gst1-plugins-base/Config.ihttps://github.com/octo/liboping/pull/50/commits/664414123a3fe89984abfa0917946c513bf47163n
> > @@ -10,6 +10,13 @@ menuconfig BR2_PACKAGE_GST1_PLUGINS_BASE
> >
> > if BR2_PACKAGE_GST1_PLUGINS_BASE
> >
> > +config BR2_PACKAGE_GST1_PLUGINS_BASE_INSTALL_TOOLS
> > + bool "install gst-discoverer & gst-device-monitor & gst-play
>
> Missing closing double quote. Also, I changed this to "install tools",
> to keep the prompt shorter.
Thanks for fixing it...
>
> > + default y
>
> I know gstreamer1 does this, but this "default y" is really not in line
> with the typical Buildroot policy of minimalism, so I dropped this.
The reasoning was to do the same as for BR2_PACKAGE_GSTREAMER1_INSTALL_TOOLS
(which defaults to y) but I am although o.k. with this one...
Regards,
Peter
>
> Applied to next with those changes.
>
> Thanks!
>
> Thomas
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-03-06 14:34 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-03 16:47 [Buildroot] [PATCH/next v1] package/gst1-plugins-base: add tools option Peter Seiderer
2020-03-05 22:34 ` Thomas Petazzoni
2020-03-06 14:34 ` Peter Seiderer
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.