[Buildroot] [PATCH v4 2/4] pulseaudio: Add dependency on atomic intrinsics

[Yann E. MORIN <yann.morin.1998@free.fr>]

Anton, All,

On 2014-08-08 19:09 +0400, Anton Kolesov spake thusly:
> Pulseaudio requires GCC built-in atomic functions which are architecture specific
> and may not be implemented.
> 
> This fixes:
> http://autobuild.buildroot.net/results/d463f3bf730a600a07ed6cd33695bf45e9fd3540/
> 
> Signed-off-by: Anton Kolesov <Anton.Kolesov@synopsys.com>

You forgot espeak's pulseaudio backen option:

    config BR2_PACKAGE_ESPEAK_AUDIO_BACKEND_PULSEAUDIO
        select BR2_PACKAGE_PULSEAUDIO

which should also depend on intrinsics.

Regards,
Yann E. MORIN.

> ---
>  package/gstreamer/gst-plugins-good/Config.in   | 5 +++++
>  package/gstreamer1/gst1-plugins-good/Config.in | 5 +++++
>  package/mpd/Config.in                          | 5 +++--
>  package/pulseaudio/Config.in                   | 6 ++++--
>  4 files changed, 17 insertions(+), 4 deletions(-)
> 
> diff --git a/package/gstreamer/gst-plugins-good/Config.in b/package/gstreamer/gst-plugins-good/Config.in
> index 2e81a6a..7ab9519 100644
> --- a/package/gstreamer/gst-plugins-good/Config.in
> +++ b/package/gstreamer/gst-plugins-good/Config.in
> @@ -212,9 +212,14 @@ config BR2_PACKAGE_GST_PLUGINS_GOOD_PLUGIN_PULSE
>  	depends on BR2_TOOLCHAIN_HAS_THREADS # pulseaudio
>  	depends on BR2_LARGEFILE # pulseaudio -> libsndfile
>  	depends on BR2_USE_MMU # pulseaudio
> +	depends on BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS # pulseaudio
>  	select BR2_PACKAGE_PULSEAUDIO
>  	bool "pulseaudio"
>  
> +comment "pulseaudio support needs a toolchain w/ largefile, threads, atomic intrinsics"
> +	depends on !BR2_LARGEFILE || !BR2_TOOLCHAIN_HAS_THREADS \
> +		|| !BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS
> +
>  config BR2_PACKAGE_GST_PLUGINS_GOOD_PLUGIN_SOUPHTTPSRC
>  	bool "souphttpsrc (http client)"
>  	depends on BR2_USE_WCHAR # libsoup -> glib2
> diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
> index aa45ba7..71b0a37 100644
> --- a/package/gstreamer1/gst1-plugins-good/Config.in
> +++ b/package/gstreamer1/gst1-plugins-good/Config.in
> @@ -308,11 +308,16 @@ config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_PULSE
>  	depends on BR2_TOOLCHAIN_HAS_THREADS # pulseaudio
>  	depends on BR2_LARGEFILE # pulseaudio -> libsndfile
>  	depends on BR2_USE_MMU # pulseaudio
> +	depends on BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS # pulseaudio
>  	select BR2_PACKAGE_PULSEAUDIO
>  	bool "pulseaudio"
>  	help
>  	  PulseAudio plugin library
>  
> +comment "pulseaudio support needs a toolchain w/ largefile, threads, atomic intrinsics"
> +	depends on !BR2_LARGEFILE || !BR2_TOOLCHAIN_HAS_THREADS \
> +		|| !BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS
> +
>  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SOUPHTTPSRC
>  	bool "souphttpsrc (http client)"
>  	depends on BR2_USE_WCHAR # libsoup -> glib2
> diff --git a/package/mpd/Config.in b/package/mpd/Config.in
> index f5f5b34..8382a5d 100644
> --- a/package/mpd/Config.in
> +++ b/package/mpd/Config.in
> @@ -45,12 +45,13 @@ config BR2_PACKAGE_MPD_AUDIOFILE
>  config BR2_PACKAGE_MPD_PULSEAUDIO
>  	bool "pulseaudio"
>  	depends on BR2_LARGEFILE # pulseaudio -> libsndfile
> +	depends on BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS # pulseaudio
>  	select BR2_PACKAGE_PULSEAUDIO
>  	help
>  	  Enable pulseaudio output support.
>  
> -comment "pulseaudio support needs a toolchain w/ largefile"
> -	depends on !BR2_LARGEFILE
> +comment "pulseaudio support needs a toolchain w/ largefile, atomic intrinsics"
> +	depends on !BR2_LARGEFILE || !BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS
>  
>  config BR2_PACKAGE_MPD_BZIP2
>  	bool "bzip2"
> diff --git a/package/pulseaudio/Config.in b/package/pulseaudio/Config.in
> index ac6222c..de99fc8 100644
> --- a/package/pulseaudio/Config.in
> +++ b/package/pulseaudio/Config.in
> @@ -8,6 +8,7 @@ config BR2_PACKAGE_PULSEAUDIO
>  	select BR2_PACKAGE_SPEEX
>  	depends on BR2_LARGEFILE
>  	depends on BR2_USE_MMU # fork()
> +	depends on BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS
>  	help
>  	  PulseAudio is a sound system for POSIX OSes, meaning that it
>  	  is a proxy for your sound applications. It allows you to do
> @@ -30,6 +31,7 @@ config BR2_PACKAGE_PULSEAUDIO_DAEMON
>  
>  endif
>  
> -comment "pulseaudio needs a toolchain w/ wchar, largefile, threads"
> +comment "pulseaudio needs a toolchain w/ wchar, largefile, threads, atomic intrinsics"
>  	depends on BR2_USE_MMU
> -	depends on !BR2_USE_WCHAR || !BR2_LARGEFILE || !BR2_TOOLCHAIN_HAS_THREADS
> +	depends on !BR2_USE_WCHAR || !BR2_LARGEFILE || !BR2_TOOLCHAIN_HAS_THREADS \
> +		|| !BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS
> -- 
> 1.8.4.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'