From 1a373194922bc73ab4113eb4d5550e5dc922782b Mon Sep 17 00:00:00 2001 From: Juergen Gross Date: Fri, 23 Mar 2018 08:42:53 +0100 Subject: [PATCH] tools/xenstore: fix linking libxenstore with ldl Commit 448c03b3cbe1487 ("tools/xenstore: try to get minimum thread stack size for watch thread") added a dependency to libdl to libxenstore. Unfortunately the way it was added requires now all users of libxenstore to specify "-ldl" when linking. This can be avoided by linking libxenstore.so specifying "-ldl" as a trailing option. So use APPEND_LDFLAGS instead of LDFLAGS for adding the "-ldl" option when linking libxenstore.so. Signed-off-by: Juergen Gross Reviewed-by: Doug Goldstein Tested-by: Doug Goldstein Acked-by: Wei Liu --- tools/xenstore/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/xenstore/Makefile b/tools/xenstore/Makefile index 69e55e73e5..445e9911b2 100644 --- a/tools/xenstore/Makefile +++ b/tools/xenstore/Makefile @@ -104,7 +104,7 @@ libxenstore.so.$(MAJOR): libxenstore.so.$(MAJOR).$(MINOR) xs.opic: CFLAGS += -DUSE_PTHREAD ifeq ($(CONFIG_Linux),y) xs.opic: CFLAGS += -DUSE_DLSYM -libxenstore.so.$(MAJOR).$(MINOR): LDFLAGS += -ldl +libxenstore.so.$(MAJOR).$(MINOR): APPEND_LDFLAGS += -ldl else PKG_CONFIG_REMOVE += -ldl endif -- 2.30.2