Re: [PATCH] tests: Link virusbmock with probes

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

 



On 02/10/2017 02:04 PM, John Ferlan wrote:
Seeing similar error to commit id '997be5c27' with the inability
to find the libvirt_event_poll_purge_timeout_semaphore symbol
causing a virusbtest failure.

Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
---

 This is a really strange one... I applied Michal's patches, the error
occurred. I rebuilt cleanly and the error went away. This morning after
apply the top of tree, the error returns. Just changing the Makefile.am
doesn't always "clean" up the error, but after a make clean and rebuild
the error goes away.  Perhaps has to do with some build dependency on
the virusbmock that virusbtest has that somehow isn't being recognized
in the Makefile.am... But I'm not sure what magic incantation needs to
be levied at the Makefile.am in order to tell it to do the right thing!

 tests/Makefile.am | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 7149a86..32a7282 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -1204,6 +1204,7 @@ virusbmock_la_SOURCES = virusbmock.c
 virusbmock_la_CFLAGS = $(AM_CFLAGS)
 virusbmock_la_LDFLAGS = $(MOCKLIBS_LDFLAGS)
 virusbmock_la_LIBADD = $(MOCKLIBS_LIBS) \
+	$(PROBES_O) \
 	../src/libvirt_util.la

 virnetdevbandwidthmock_la_SOURCES = \

Hi John,
your patch worked for me.

Tested-by: Boris Fiuczynski <fiuczy@xxxxxxxxxxxxxxxxxx>


--
Mit freundlichen Grüßen/Kind regards
   Boris Fiuczynski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martina Köderitz
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294

--
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]
  Powered by Linux