[libvirt] [PATCH v3] Introduce --without-pm-utils to get rid of pm-is-supported dependency

2014-04-02 Thread Cédric Bosdonnat
This uses the dbus api of systemd to check the power management capabilities of the node. --- Diff with v2: * Fixed a few dbus call problems configure.ac | 11 + libvirt.spec.in | 9 src/libvirt_private.syms | 3 +++ src/util/virnodesuspend.c | 32

Re: [libvirt] [PATCH v3] Introduce --without-pm-utils to get rid of pm-is-supported dependency

2014-04-02 Thread Daniel P. Berrange
On Wed, Apr 02, 2014 at 03:35:51PM +0200, Cédric Bosdonnat wrote: This uses the dbus api of systemd to check the power management capabilities of the node. --- Diff with v2: * Fixed a few dbus call problems configure.ac | 11 + libvirt.spec.in | 9

Re: [libvirt] [PATCH v3] Introduce --without-pm-utils to get rid of pm-is-supported dependency

2014-04-02 Thread Cedric Bosdonnat
On Wed, 2014-04-02 at 15:52 +0200, Daniel P. Berrange wrote: On Wed, Apr 02, 2014 at 03:35:51PM +0200, Cédric Bosdonnat wrote: This uses the dbus api of systemd to check the power management capabilities of the node. --- Diff with v2: * Fixed a few dbus call problems

Re: [libvirt] [PATCH v3] Introduce --without-pm-utils to get rid of pm-is-supported dependency

2014-04-02 Thread Daniel P. Berrange
On Wed, Apr 02, 2014 at 03:57:44PM +0200, Cedric Bosdonnat wrote: On Wed, 2014-04-02 at 15:52 +0200, Daniel P. Berrange wrote: On Wed, Apr 02, 2014 at 03:35:51PM +0200, Cédric Bosdonnat wrote: This uses the dbus api of systemd to check the power management capabilities of the node. ---

Re: [libvirt] [PATCH v3] Introduce --without-pm-utils to get rid of pm-is-supported dependency

2014-04-02 Thread Eric Blake
On 04/02/2014 07:35 AM, Cédric Bosdonnat wrote: This uses the dbus api of systemd to check the power management capabilities of the node. --- Diff with v2: * Fixed a few dbus call problems +++ b/configure.ac @@ -563,6 +563,10 @@ AC_ARG_WITH([chrdev-lock-files], [location for

Re: [libvirt] [PATCH v3] Introduce --without-pm-utils to get rid of pm-is-supported dependency

2014-04-02 Thread Eric Blake
On 04/02/2014 08:06 AM, Daniel P. Berrange wrote: Rather than having a hardcoded choice of pm-utils vs systemd I think we should check if the systemd service is running and if so use it, otherwise fallback to pm-utils. That way if someone has systemd installed, but is not running it as