Re: [libvirt] [PATCH] Add support for /run/initctl

2014-10-22 Thread Daniel P. Berrange
On Tue, Sep 30, 2014 at 12:01:13PM -0500, Rick Harris wrote: Newer versions of Debian use `/run/initctl` instead of `/dev/initctl`. This patch updates the code to search for the FIFO from a list of well-known locations. In the FreeBSD case, as before, we fall-back to the `/etc/.initctl`

Re: [libvirt] [PATCH] Add support for /run/initctl

2014-10-21 Thread Rick Harris
Hi all, A few weeks ago I proposed a patch to fix LXC shutdown for distros that use `/run/initctl` instead of `/dev/initctl`. Replying here so that it gets some more visibility. Would love some feedback here: are people happy with the general approach? This is my first libvirt patch: is the

[libvirt] [PATCH] Add support for /run/initctl

2014-09-30 Thread Rick Harris
Newer versions of Debian use `/run/initctl` instead of `/dev/initctl`. This patch updates the code to search for the FIFO from a list of well-known locations. In the FreeBSD case, as before, we fall-back to the `/etc/.initctl` stub. --- src/util/virinitctl.c | 38