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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
---
 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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]