Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config.mk: wrong BROKER_LIBS for FreeBSD #11

Closed
ralight opened this issue Mar 15, 2016 · 0 comments
Closed

config.mk: wrong BROKER_LIBS for FreeBSD #11

ralight opened this issue Mar 15, 2016 · 0 comments

Comments

@ralight
Copy link
Contributor

ralight commented Mar 15, 2016

migrated from Bugzilla #485131
status RESOLVED severity trivial in component Mosquitto for 1.4
Reported in version 1.4 on platform PC
Assigned to: Roger Light

On 2016-01-04 07:13:05 -0500, Peter Morjan wrote:

On FreeBSD BROKER_LIBS should not include '-ldl'
The following patch solves the issue:

diff --git a/config.mk b/config.mk
index 01532c3..9eb5fe5 100644
--- a/config.mk
+++ b/config.mk
@@ -115,7 +115,7 @@ LIB_LDFLAGS:=${LDFLAGS}
BROKER_CFLAGS:=${LIB_CFLAGS} ${CPPFLAGS} -DVERSION=""${VERSION}"" -DTIMESTAMP=""${TIMESTAMP}"" -DWITH_BROKER
CLIENT_CFLAGS:=${CFLAGS} ${CPPFLAGS} -I../lib -DVERSION=""${VERSION}""

-ifneq ($(or $(find $(UNAME),FreeBSD), $(find $(UNAME),OpenBSD)),)
+ifneq ($(or $(findstring $(UNAME),FreeBSD), $(findstring $(UNAME),OpenBSD)),)
BROKER_LIBS:=-lm
else
BROKER_LIBS:=-ldl -lm

On 2016-02-11 16:53:00 -0500, Roger Light wrote:

Thanks, I've fixed this in the fixes branch.

@ralight ralight closed this as completed Mar 15, 2016
@lock lock bot locked as resolved and limited conversation to collaborators Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant