diff mbox

zeromq: disable for static linking

Message ID 20160826212821.GA21305@waldemar-brodkorb.de
State Rejected
Headers show

Commit Message

Waldemar Brodkorb Aug. 26, 2016, 9:28 p.m. UTC
Fixes:
  http://autobuild.buildroot.net/results/3aa73620d0f1afcc82de93069c9693e1ca755bb9

Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
---
 package/zeromq/Config.in | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

Comments

Thomas Petazzoni Aug. 27, 2016, 12:55 p.m. UTC | #1
Hello,

On Fri, 26 Aug 2016 23:28:21 +0200, Waldemar Brodkorb wrote:
> Fixes:
>   http://autobuild.buildroot.net/results/3aa73620d0f1afcc82de93069c9693e1ca755bb9
> 
> Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>

I disagree with this change because it seems more like a toolchain
issue than the package really needing dynamic library support.

And indeed, the following static only configuration that includes zeromq
builds perfectly fine:

BR2_arm=y
BR2_STATIC_LIBS=y
BR2_TOOLCHAIN_EXTERNAL=y
BR2_TOOLCHAIN_EXTERNAL_CUSTOM=y
BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD=y
BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/br-arm-full-static-2016.08-rc1-4-g07e8d1c.tar.bz2"
BR2_TOOLCHAIN_EXTERNAL_GCC_4_9=y
BR2_TOOLCHAIN_EXTERNAL_HEADERS_3_14=y
BR2_TOOLCHAIN_EXTERNAL_LOCALE=y
# BR2_TOOLCHAIN_EXTERNAL_HAS_THREADS_DEBUG is not set
BR2_TOOLCHAIN_EXTERNAL_INET_RPC=y
BR2_TOOLCHAIN_EXTERNAL_CXX=y
BR2_INIT_NONE=y
BR2_SYSTEM_BIN_SH_NONE=y
# BR2_PACKAGE_BUSYBOX is not set
BR2_PACKAGE_ZEROMQ=y
# BR2_TARGET_ROOTFS_TAR is not set

Best regards,

Thomas
diff mbox

Patch

diff --git a/package/zeromq/Config.in b/package/zeromq/Config.in
index d5c9dea..178b044 100644
--- a/package/zeromq/Config.in
+++ b/package/zeromq/Config.in
@@ -1,12 +1,13 @@ 
-comment "zeromq needs a toolchain w/ C++, wchar, threads"
+comment "zeromq needs a toolchain w/ C++, wchar, threads, dynamic library"
 	depends on !(BR2_INSTALL_LIBSTDCPP && BR2_USE_WCHAR && \
-		BR2_TOOLCHAIN_HAS_THREADS)
+		BR2_TOOLCHAIN_HAS_THREADS) || BR2_STATIC_LIBS
 
 config BR2_PACKAGE_ZEROMQ
 	bool "zeromq"
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_TOOLCHAIN_HAS_THREADS
+	depends on !BR2_STATIC_LIBS
 	select BR2_PACKAGE_UTIL_LINUX
 	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
 	help
@@ -27,12 +28,8 @@  config BR2_PACKAGE_ZEROMQ
 
 if BR2_PACKAGE_ZEROMQ
 
-comment "norm support needs a toolchain w/ dynamic library"
-	depends on BR2_STATIC_LIBS
-
 config BR2_PACKAGE_ZEROMQ_NORM
 	bool "NORM support"
-	depends on !BR2_STATIC_LIBS
 	select BR2_PACKAGE_NORM
 	help
 	  Add support for NACK-Oriented Reliable Multicast (RFC 5740)