diff mbox

czmq: needs MMU

Message ID 97ed4f06546e84c2b326a01447f77519460dfeff.1399262283.git.baruch@tkos.co.il
State Accepted
Headers show

Commit Message

Baruch Siach May 5, 2014, 3:58 a.m. UTC
Fixes:
http://autobuild.buildroot.net/results/6dd/6dd6954e5ab8c422070e53b0e616e1b1a821c626/

Also, remove the Blackfin toolchain dependencies as these are redundant now
since Blackfin is a noMMU arch.

Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 package/czmq/Config.in | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

Comments

Peter Korsgaard May 5, 2014, 6:45 a.m. UTC | #1
>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:

 > Fixes:
 > http://autobuild.buildroot.net/results/6dd/6dd6954e5ab8c422070e53b0e616e1b1a821c626/

 > Also, remove the Blackfin toolchain dependencies as these are redundant now
 > since Blackfin is a noMMU arch.

You forgot its reverse dependencies (filemq). Committed with that fixed,
thanks.
Mike Zick May 5, 2014, 8:33 a.m. UTC | #2
On Mon,  5 May 2014 06:58:03 +0300
Baruch Siach <baruch@tkos.co.il> wrote:

>  config BR2_PACKAGE_CZMQ
>  	bool "czmq"
>  	select BR2_PACKAGE_ZEROMQ
> +	depends on BR2_USE_MMU # fork()

  ^ ^ ^ ^
>  	depends on BR2_INSTALL_LIBSTDCPP # zeromq
>  	depends on BR2_INET_IPV6 # zeromq
>  	depends on BR2_LARGEFILE # util-linux
>  	depends on BR2_USE_WCHAR # util-linux
>  	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
> -	# uClibc toolchains provided by ADI don't have AI_ADDRCONFIG
> -	# support
> -	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
> && \
> -		!BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R1
>  	help
>  	  High-level C Binding for 0MQ
>  
>  	  http://czmq.zeromq.org/
>  
>  comment "czmq needs a toolchain w/ C++, IPv6, largefile, wchar,
> threads"
> -	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
> && \
> -		!BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R1
> +	depends on BR2_USE_MMU

  ^ ^ ^ ^
>  	depends on !(BR2_INSTALL_LIBSTDCPP && BR2_INET_IPV6 &&
> BR2_LARGEFILE \ && BR2_USE_WCHAR && BR2_TOOLCHAIN_HAS_THREADS)
>

Shouldn't the option and the comment have opposite conditions?
I.E: Either display the option or the comment?

Mike
Baruch Siach May 5, 2014, 8:41 a.m. UTC | #3
Hi Mike,

On Mon, May 05, 2014 at 03:33:09AM -0500, Mike Zick wrote:
> On Mon,  5 May 2014 06:58:03 +0300
> Baruch Siach <baruch@tkos.co.il> wrote:
> 
> >  config BR2_PACKAGE_CZMQ
> >  	bool "czmq"
> >  	select BR2_PACKAGE_ZEROMQ
> > +	depends on BR2_USE_MMU # fork()
> 
>   ^ ^ ^ ^
> >  	depends on BR2_INSTALL_LIBSTDCPP # zeromq
> >  	depends on BR2_INET_IPV6 # zeromq
> >  	depends on BR2_LARGEFILE # util-linux
> >  	depends on BR2_USE_WCHAR # util-linux
> >  	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
> > -	# uClibc toolchains provided by ADI don't have AI_ADDRCONFIG
> > -	# support
> > -	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
> > && \
> > -		!BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R1
> >  	help
> >  	  High-level C Binding for 0MQ
> >  
> >  	  http://czmq.zeromq.org/
> >  
> >  comment "czmq needs a toolchain w/ C++, IPv6, largefile, wchar,
> > threads"
> > -	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
> > && \
> > -		!BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R1
> > +	depends on BR2_USE_MMU
> 
>   ^ ^ ^ ^
> >  	depends on !(BR2_INSTALL_LIBSTDCPP && BR2_INET_IPV6 &&
> > BR2_LARGEFILE \ && BR2_USE_WCHAR && BR2_TOOLCHAIN_HAS_THREADS)
> >
> 
> Shouldn't the option and the comment have opposite conditions?
> I.E: Either display the option or the comment?

MMU is usually an attribute of the CPU architecture, and not a software 
configuration issue. In this case we don't show the software dependencies 
comment since the user can't do much about the CPU architecture of choice in 
the context of Buildroot. See 
http://buildroot.net/downloads/manual/manual.html#_literal_config_in_literal_file, 
under "Dependencies on target and toolchain options".

baruch
diff mbox

Patch

diff --git a/package/czmq/Config.in b/package/czmq/Config.in
index 29333680fd61..fbdedd1bec2f 100644
--- a/package/czmq/Config.in
+++ b/package/czmq/Config.in
@@ -1,22 +1,18 @@ 
 config BR2_PACKAGE_CZMQ
 	bool "czmq"
 	select BR2_PACKAGE_ZEROMQ
+	depends on BR2_USE_MMU # fork()
 	depends on BR2_INSTALL_LIBSTDCPP # zeromq
 	depends on BR2_INET_IPV6 # zeromq
 	depends on BR2_LARGEFILE # util-linux
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
-	# uClibc toolchains provided by ADI don't have AI_ADDRCONFIG
-	# support
-	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2 && \
-		!BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R1
 	help
 	  High-level C Binding for 0MQ
 
 	  http://czmq.zeromq.org/
 
 comment "czmq needs a toolchain w/ C++, IPv6, largefile, wchar, threads"
-	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2 && \
-		!BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R1
+	depends on BR2_USE_MMU
 	depends on !(BR2_INSTALL_LIBSTDCPP && BR2_INET_IPV6 && BR2_LARGEFILE \
 		&& BR2_USE_WCHAR && BR2_TOOLCHAIN_HAS_THREADS)