[libvirt] [PATCH] tools: Link libvirt.la to libvirt_shell.a

Michal Privoznik mprivozn at redhat.com
Fri Sep 25 11:38:18 UTC 2015


So, our mingw build is broken. It's because while libvirt_shell
library is using some of our internal APIs, e.g. virStrndup, it's
not being linked with libvirt.la. Only subsequent users of the
library, like virsh, do link to libvirt.la. In fact, I'm
surprised Linux linker doesn't care, because how can it make a
static library with missing symbols is mystery to me.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 tools/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/Makefile.am b/tools/Makefile.am
index 0a00cee..3cf70db 100644
--- a/tools/Makefile.am
+++ b/tools/Makefile.am
@@ -143,6 +143,7 @@ libvirt_shell_la_LDFLAGS = \
 		$(NULL)
 libvirt_shell_la_LIBADD = \
 		$(LIBXML_LIBS)						\
+		../src/libvirt.la					\
 		../gnulib/lib/libgnu.la				\
 		$(NULL)
 libvirt_shell_la_SOURCES = vsh.c vsh.h
@@ -220,7 +221,6 @@ virsh_LDFLAGS = \
 virsh_LDADD =							\
 		$(STATIC_BINARIES)				\
 		$(PIE_LDFLAGS)					\
-		../src/libvirt.la				\
 		../src/libvirt-lxc.la				\
 		../src/libvirt-qemu.la				\
 		libvirt_shell.la				\
-- 
2.4.9




More information about the libvir-list mailing list