]> git.ipfire.org Git - thirdparty/libvirt.git/commit
systemd: fix build without dbus
authorDaniel P. Berrange <berrange@redhat.com>
Mon, 19 Jan 2015 12:30:24 +0000 (12:30 +0000)
committerDaniel P. Berrange <berrange@redhat.com>
Mon, 26 Jan 2015 09:14:04 +0000 (09:14 +0000)
commitd13b586a9113c2c3f3f541a82865458bbebdf2a0
tree0862b899ddcd6d20c716ce6c459c428e786ced0d
parentee4c13ce1d1f1c4de2aca613176f30026c6a0905
systemd: fix build without dbus

The virDBusMethodCall method has a DBusError as one of its
parameters. If the caller wants to pass a non-NULL value
for this, it immediately makes the calling code require
DBus at build time. This has led to breakage of non-DBus
builds several times. It is desirable that only the virdbus.c
file should need WITH_DBUS conditionals, so we must ideally
remove the DBusError parameter from the method.

We can't simply raise a libvirt error, since the whole point
of this parameter is to give the callers a way to check if
the error is one they want to ignore, without having the logs
polluted with an error message. So, we add a virErrorPtr
parameter which the caller can then either ignore or raise
using the new virReportErrorObject method.

This new method is distinct from virSetError in that it
ensures the logging hooks are run.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
po/POTFILES.in
src/libvirt_private.syms
src/util/virdbus.c
src/util/virdbus.h
src/util/virerror.c
src/util/virerror.h
src/util/virfirewall.c
src/util/virsystemd.c