]> xenbits.xensource.com Git - xen.git/commitdiff
tools/xenstore: fix linking libxenstore with ldl
authorJuergen Gross <jgross@suse.com>
Fri, 23 Mar 2018 07:42:53 +0000 (08:42 +0100)
committerWei Liu <wei.liu2@citrix.com>
Fri, 23 Mar 2018 16:27:42 +0000 (16:27 +0000)
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 <jgross@suse.com>
Reviewed-by: Doug Goldstein <cardoe@cardoe.com>
Tested-by: Doug Goldstein <cardoe@cardoe.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
tools/xenstore/Makefile

index 69e55e73e5ecea96c83b9f3624f14e27c8ec7474..445e9911b207b64ffcaabb9e7c1ae19bc699d150 100644 (file)
@@ -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