diff mbox

libvpx: needs thread support

Message ID 1383408297-17438-1-git-send-email-thomas.petazzoni@free-electrons.com
State Accepted
Commit 5702d2f3b7f12882d4e33d26606732290e442324
Headers show

Commit Message

Thomas Petazzoni Nov. 2, 2013, 4:04 p.m. UTC
Fixes:

  http://autobuild.buildroot.org/results/8de/8def7e54e4fecb7df350443824c00b53d1c61c01/build-end.log

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/gstreamer1/gst1-plugins-good/Config.in | 4 ++++
 package/libvpx/Config.in                       | 4 ++++
 2 files changed, 8 insertions(+)

Comments

Yann E. MORIN Nov. 2, 2013, 6 p.m. UTC | #1
Thomas, All,

On 2013-11-02 17:04 +0100, Thomas Petazzoni spake thusly:
> Fixes:
> 
>   http://autobuild.buildroot.org/results/8de/8def7e54e4fecb7df350443824c00b53d1c61c01/build-end.log
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

> ---
>  package/gstreamer1/gst1-plugins-good/Config.in | 4 ++++
>  package/libvpx/Config.in                       | 4 ++++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
> index 3342208..a9308c7 100644
> --- a/package/gstreamer1/gst1-plugins-good/Config.in
> +++ b/package/gstreamer1/gst1-plugins-good/Config.in
> @@ -335,9 +335,13 @@ comment "taglib needs a toolchain w/ C++"
>  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_VPX
>  	bool "vpx (webm)"
>  	select BR2_PACKAGE_LIBVPX
> +	depends on BR2_TOOLCHAIN_HAS_THREADS # libvpx
>  	help
>  	  VP8 plugin
>  
> +comment "libvpx needs a toolchain w/ threads"
> +	depends on !BR2_TOOLCHAIN_HAS_THREADS
> +
>  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_WAVPACK
>  	bool "wavpack (*.wv audio)"
>  	select BR2_PACKAGE_WAVPACK
> diff --git a/package/libvpx/Config.in b/package/libvpx/Config.in
> index 1332c56..0ed3e1a 100644
> --- a/package/libvpx/Config.in
> +++ b/package/libvpx/Config.in
> @@ -1,7 +1,11 @@
>  config BR2_PACKAGE_LIBVPX
>  	bool "libvpx"
> +	depends on BR2_TOOLCHAIN_HAS_THREADS
>  	help
>  	  A high-quality, open video format that's freely available to
>  	  everyone.
>  
>  	  http://webmproject.org
> +
> +comment "libvpx needs a toolchain w/ threads"
> +	depends on !BR2_TOOLCHAIN_HAS_THREADS
> -- 
> 1.8.1.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Samuel Martin Nov. 2, 2013, 6 p.m. UTC | #2
2013/11/2 Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

> Fixes:
>
>
> http://autobuild.buildroot.org/results/8de/8def7e54e4fecb7df350443824c00b53d1c61c01/build-end.log
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
>
Acked-by: Samuel Martin <s.martin49@gmail.com>

Regards,
Thomas Petazzoni Nov. 2, 2013, 6:37 p.m. UTC | #3
Dear Thomas Petazzoni,

On Sat,  2 Nov 2013 17:04:57 +0100, Thomas Petazzoni wrote:
> Fixes:
> 
>   http://autobuild.buildroot.org/results/8de/8def7e54e4fecb7df350443824c00b53d1c61c01/build-end.log
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  package/gstreamer1/gst1-plugins-good/Config.in | 4 ++++
>  package/libvpx/Config.in                       | 4 ++++
>  2 files changed, 8 insertions(+)

Committed to for-peter-2013.11, thanks!

Thomas
Peter Korsgaard Nov. 4, 2013, 9:05 p.m. UTC | #4
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 > Fixes:
 >   http://autobuild.buildroot.org/results/8de/8def7e54e4fecb7df350443824c00b53d1c61c01/build-end.log

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
 > ---
 >  package/gstreamer1/gst1-plugins-good/Config.in | 4 ++++
 >  package/libvpx/Config.in                       | 4 ++++
 >  2 files changed, 8 insertions(+)

 > diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
 > index 3342208..a9308c7 100644
 > --- a/package/gstreamer1/gst1-plugins-good/Config.in
 > +++ b/package/gstreamer1/gst1-plugins-good/Config.in
 > @@ -335,9 +335,13 @@ comment "taglib needs a toolchain w/ C++"
 >  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_VPX
 >  	bool "vpx (webm)"
 >  	select BR2_PACKAGE_LIBVPX
 > +	depends on BR2_TOOLCHAIN_HAS_THREADS # libvpx
 >  	help
 >  	  VP8 plugin
 
 > +comment "libvpx needs a toolchain w/ threads"
 > +	depends on !BR2_TOOLCHAIN_HAS_THREADS
 > +

Gstreamer1 itself is a heavy user of threads (and therefor depends on
BR2_TOOLCHAIN_HAS_THREADS), so this comment is never going to be
visible, but ok.
diff mbox

Patch

diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
index 3342208..a9308c7 100644
--- a/package/gstreamer1/gst1-plugins-good/Config.in
+++ b/package/gstreamer1/gst1-plugins-good/Config.in
@@ -335,9 +335,13 @@  comment "taglib needs a toolchain w/ C++"
 config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_VPX
 	bool "vpx (webm)"
 	select BR2_PACKAGE_LIBVPX
+	depends on BR2_TOOLCHAIN_HAS_THREADS # libvpx
 	help
 	  VP8 plugin
 
+comment "libvpx needs a toolchain w/ threads"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
+
 config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_WAVPACK
 	bool "wavpack (*.wv audio)"
 	select BR2_PACKAGE_WAVPACK
diff --git a/package/libvpx/Config.in b/package/libvpx/Config.in
index 1332c56..0ed3e1a 100644
--- a/package/libvpx/Config.in
+++ b/package/libvpx/Config.in
@@ -1,7 +1,11 @@ 
 config BR2_PACKAGE_LIBVPX
 	bool "libvpx"
+	depends on BR2_TOOLCHAIN_HAS_THREADS
 	help
 	  A high-quality, open video format that's freely available to
 	  everyone.
 
 	  http://webmproject.org
+
+comment "libvpx needs a toolchain w/ threads"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS