From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann E. MORIN Date: Fri, 15 Aug 2014 22:54:59 +0200 Subject: [Buildroot] [PATCH v4 2/4] pulseaudio: Add dependency on atomic intrinsics In-Reply-To: <1407510572-20603-2-git-send-email-Anton.Kolesov@synopsys.com> References: <1407510572-20603-1-git-send-email-Anton.Kolesov@synopsys.com> <1407510572-20603-2-git-send-email-Anton.Kolesov@synopsys.com> Message-ID: <20140815205459.GC25897@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net 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 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. | '------------------------------^-------^------------------^--------------------'