diff mbox

liquid-dsp: blacklist CS powerpc toolchains

Message ID 1447784444-5336-1-git-send-email-gustavo@zacarias.com.ar
State Accepted
Headers show

Commit Message

Gustavo Zacarias Nov. 17, 2015, 6:20 p.m. UTC
They're unable to build liquid-dsp successfully resulting in an internal
compiler error. Fixes:
http://autobuild.buildroot.net/results/2ac/2acd745f4e5ec6edfd09afcd57b9ece216724c1b/

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/liquid-dsp/Config.in | 3 +++
 1 file changed, 3 insertions(+)

Comments

Thomas Petazzoni Nov. 17, 2015, 8:19 p.m. UTC | #1
Dear Gustavo Zacarias,

On Tue, 17 Nov 2015 15:20:44 -0300, Gustavo Zacarias wrote:
> They're unable to build liquid-dsp successfully resulting in an internal
> compiler error. Fixes:
> http://autobuild.buildroot.net/results/2ac/2acd745f4e5ec6edfd09afcd57b9ece216724c1b/
> 
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
>  package/liquid-dsp/Config.in | 3 +++
>  1 file changed, 3 insertions(+)

You forgot to use the same dependencies in the comment, to avoid it
being displayed in cases where the package can anyway not be enabled.
I've applied to master after fixing this. Thanks!

Thomas
Gustavo Zacarias Nov. 17, 2015, 8:24 p.m. UTC | #2
On 17/11/15 17:19, Thomas Petazzoni wrote:

> You forgot to use the same dependencies in the comment, to avoid it
> being displayed in cases where the package can anyway not be enabled.
> I've applied to master after fixing this. Thanks!
>
> Thomas

Hi Thomas.
I believe that would be a no-op, since:


comment "liquid-dsp requires a (e)glibc/musl toolchain w/ dynamic library"
         depends on !(BR2_TOOLCHAIN_USES_GLIBC || 
BR2_TOOLCHAIN_USES_MUSL) || BR2_STATIC_LIBS

It is a glibc toolchain (well, eglibc i believe, but same deal), and 
glibc toolchains can't do static, so unreachable.

Regards.
diff mbox

Patch

diff --git a/package/liquid-dsp/Config.in b/package/liquid-dsp/Config.in
index 95233da..c388734 100644
--- a/package/liquid-dsp/Config.in
+++ b/package/liquid-dsp/Config.in
@@ -5,6 +5,9 @@  config BR2_PACKAGE_LIQUID_DSP
 	bool "liquid-dsp"
 	depends on BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL
 	depends on !BR2_STATIC_LIBS
+	# These result in internal compiler error
+	depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC201103
+	depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC201009
 	help
 	  Liquid-DSP is a free and open-source signal processing
 	  library for software-defined radios written in C.  Its