On Sat, Nov 21, 2015 at 05:16:33PM +0000, Richard W.M. Jones wrote:
On Fri, Nov 20, 2015 at 08:38:40AM -0500, Stephen Gallagher wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 11/19/2015 09:04 PM, Zbigniew Jędrzejewski-Szmek wrote:
> > On Thu, Nov 19, 2015 at 10:28:25PM +0000, Richard W.M. Jones
> > wrote:
> >> On Wed, Nov 18, 2015 at 05:32:02PM +0100, Jan Kurik wrote:
> >>> * systemd-udev.rpm will consist of systemd-udevd, udevadm, udev
> >>> rules, and the hardware database.
> >>
> >> So close! libguestfs needs udevd and /sbin/reboot.
> >> Unfortunately the latter is still part of the main systemd
> >> package.
> > reboot is systemctl is a thin client for systemd, so no hope here
> > :)
>
> Why does libguestfs need /sbin/reboot? That seems a little suspect to me.
It runs it:
https://github.com/libguestfs/libguestfs/blob/master/appliance/init#L196
But actually it only runs `reboot -f', because we have no systemd in
the appliance -- as there is only one process + udevd in the
appliance, using systemd would be overkill. So that could be replaced
with a simple system call to `reboot (RB_AUTOBOOT)'.
Filesystems should be
synced first (that's what systemctl reboot -f / reboot -f does).
I don't know if libguestfs does that separately.
Zbyszek