On Fri, Nov 27, 2015 at 12:19:11PM +0100, Martin Kletzander wrote: > Signed-off-by: Martin Kletzander <mklet...@redhat.com>
Could do with a commit message that describes the problem seen, particularly since the virotatingfile code doesn't use DBus at all (unless by accident) > --- > tests/Makefile.am | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/tests/Makefile.am b/tests/Makefile.am > index b449286f1fb3..e2fb3b7f3b67 100644 > --- a/tests/Makefile.am > +++ b/tests/Makefile.am > @@ -172,7 +172,6 @@ test_programs = virshtest sockettest \ > virkeycodetest \ > virlockspacetest \ > virlogtest \ > - virrotatingfiletest \ > virstringtest \ > virportallocatortest \ > sysinfotest \ > @@ -210,6 +209,7 @@ endif WITH_LIBVIRTD > if WITH_DBUS > test_programs += virdbustest \ > virsystemdtest \ > + virrotatingfiletest \ > $(NULL) > if WITH_POLKIT1 > test_programs += virpolkittest > @@ -1112,7 +1112,9 @@ virsystemdtest_CFLAGS = $(AM_CFLAGS) $(DBUS_CFLAGS) > virsystemdtest_LDADD = $(LDADDS) $(DBUS_LIBS) > > else ! WITH_DBUS > -EXTRA_DIST += virdbustest.c virmockdbus.c virsystemdtest.c > +EXTRA_DIST += \ > + virdbustest.c virmockdbus.c \ > + virsystemdtest.c virrotatingfiletest.c > endif ! WITH_DBUS > > viruritest_SOURCES = \ NACK to this, it must be masking a different problem, since the file code doesn't need dbus at all. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
