Bug#719738: [PATCH] Please install the lvm2 systemd generator

[Bastian Blank]

On Thu, Aug 22, 2013 at 11:38:42PM +0200, Michael Stapelberg wrote:
> The systemd service file for udev, which we take directly from upstream,

There is no udev.service in the upstream systemd source. In the Debian
package it is a symlink to systemd-udevd.service.

>                                                               Instead,
> upstream’s view (specifically Kay’s view as the udev maintainer) is that
> Linux is event-driven and udev handles these events whenever they occur,

What upstream thinks is irrelevant if the system expecations are
different. Why can't this be a symlink to systemd-udev-settle.service,
which provides the backward compatible behaviour?

> Therefore, I ask you again to please include the patch I attached to
> this bugreport in order to make lvm work much better on a Debian machine
> using systemd.

First this needs a security check. The tool uses /tmp as default
directory and most likely runs as root. So at least this can be used for
DoS.

Second the init-script and the generated unit have different names, so
systemd won't be able to consider them equal. I have no idea how this
really works anyway.

Also please show me where the generator stuff is documented.

> I don’t quite understand why you are hesitant to do that, given that
> there are absolutely no changes to sysvinit users. Maybe you can clarify
> what your concern is, if any?

Because it deliberately breaks stuff. That it only supports one of the
kernel types supported by Debian does not help eather.

Bastian

-- 
I'm a soldier, not a diplomat.  I can only tell the truth.
		-- Kirk, "Errand of Mercy", stardate 3198.9