package/rabbitmq-c: needs dynamic library
syslog-ng expects that rabbitmq-c is built with openssl support however currently we're disabling openssl on rabbitmq-c in static build. To fix this issue, add a dependency on dynamic library on rabbitmq-c and its reverse dependencies (only BR2_PACKAGE_JANUS_GATEWAY_RABBITMQ as php-amqp already depends on dynamic library) Fixes: - http://autobuild.buildroot.org/results/fce91b98fb199a26ad5f5f726c9bdec4f9d64486 Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
This commit is contained in:
parent
2fb86873db
commit
6bc9ded5dd
@ -67,11 +67,12 @@ comment "MQTT transport needs a toolchain w/ threads and dynamic library support
|
||||
|
||||
config BR2_PACKAGE_JANUS_GATEWAY_RABBITMQ
|
||||
bool "RabbitMQ"
|
||||
depends on !BR2_STATIC_LIBS
|
||||
depends on BR2_TOOLCHAIN_HAS_THREADS
|
||||
select BR2_PACKAGE_RABBITMQ_C
|
||||
|
||||
comment "RabbitMQ transport needs a toolchain w/ threads"
|
||||
depends on !BR2_TOOLCHAIN_HAS_THREADS
|
||||
comment "RabbitMQ transport needs a toolchain w/ dynamic library, threads"
|
||||
depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS
|
||||
|
||||
config BR2_PACKAGE_JANUS_GATEWAY_REST
|
||||
bool "REST (HTTP/HTTPS)"
|
||||
|
@ -1,5 +1,6 @@
|
||||
config BR2_PACKAGE_RABBITMQ_C
|
||||
bool "rabbitmq-c"
|
||||
depends on !BR2_STATIC_LIBS
|
||||
depends on BR2_TOOLCHAIN_HAS_THREADS
|
||||
help
|
||||
This is a C-language AMQP client library for use with v2.0+
|
||||
@ -7,5 +8,5 @@ config BR2_PACKAGE_RABBITMQ_C
|
||||
|
||||
https://github.com/alanxz/rabbitmq-c
|
||||
|
||||
comment "rabbitmq-c needs a toolchain w/ threads"
|
||||
depends on !BR2_TOOLCHAIN_HAS_THREADS
|
||||
comment "rabbitmq-c needs a toolchain w/ dynamic library, threads"
|
||||
depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS
|
||||
|
@ -19,28 +19,20 @@ RABBITMQ_C_CONF_OPTS = \
|
||||
RABBITMQ_C_CONF_OPTS += -DTHREADS_PTHREAD_ARG=OFF
|
||||
|
||||
# BUILD_SHARED_LIBS is handled in pkg-cmake.mk as it is a generic cmake variable
|
||||
ifeq ($(BR2_STATIC_LIBS),y)
|
||||
RABBITMQ_C_CONF_OPTS += -DBUILD_STATIC_LIBS=ON
|
||||
else ifeq ($(BR2_SHARED_STATIC_LIBS),y)
|
||||
ifeq ($(BR2_SHARED_STATIC_LIBS),y)
|
||||
RABBITMQ_C_CONF_OPTS += -DBUILD_STATIC_LIBS=ON
|
||||
else ifeq ($(BR2_SHARED_LIBS),y)
|
||||
RABBITMQ_C_CONF_OPTS += -DBUILD_STATIC_LIBS=OFF
|
||||
endif
|
||||
|
||||
# CMake OpenSSL detection is buggy, and doesn't properly use
|
||||
# pkg-config, so it fails when statically linking. See
|
||||
# https://gitlab.kitware.com/cmake/cmake/issues/16885.
|
||||
ifeq ($(BR2_PACKAGE_OPENSSL):$(BR2_STATIC_LIBS),y:)
|
||||
ifeq ($(BR2_PACKAGE_OPENSSL),y)
|
||||
RABBITMQ_C_CONF_OPTS += -DENABLE_SSL_SUPPORT=ON
|
||||
RABBITMQ_C_DEPENDENCIES += openssl
|
||||
else
|
||||
RABBITMQ_C_CONF_OPTS += -DENABLE_SSL_SUPPORT=OFF
|
||||
endif
|
||||
|
||||
# Popt is sometimes linked against libintl, but CMake doesn't know
|
||||
# about that, and there's no way to tell manually CMake to link
|
||||
# against an additional library.
|
||||
ifeq ($(BR2_PACKAGE_POPT):$(BR2_STATIC_LIBS),y:)
|
||||
ifeq ($(BR2_PACKAGE_POPT),y)
|
||||
RABBITMQ_C_CONF_OPTS += -DBUILD_TOOLS=ON
|
||||
RABBITMQ_C_DEPENDENCIES += popt
|
||||
else
|
||||
|
Loading…
Reference in New Issue
Block a user