Bug#796625: [PATCH] clvm: ship native systemd units in debian package

[Bastian Blank]

On Mon, Jun 13, 2016 at 01:31:49PM +0200, Andreas Henriksson wrote:
> Hope this helps. Would be much appreciated if we could resolve this
> bug report (very) soon. I'd like to avoid NMUing this package but
> if progress is blocked on this bug report I'll go ahead if there's
> no feedback by then.

Did you at least test it once?  My gut say: no.

> --- lvm2-2.02.153/debian/patches/clvm-systemd-unit-debian-adaptions.patch	1970-01-01 01:00:00.000000000 +0100
> +++ lvm2-2.02.153/debian/patches/clvm-systemd-unit-debian-adaptions.patch	2016-06-13 13:05:16.000000000 +0200

There is already a patch for systemd modifications, use it.

> +--- a/scripts/lvm2_clvmd_systemd_red_hat.service.in
> ++++ b/scripts/lvm2_clvmd_systemd_red_hat.service.in
> +@@ -1,9 +1,10 @@
> + [Unit]
> + Description=Clustered LVM daemon
> + Documentation=man:clvmd(8)
> +-After=dlm.service corosync.service
> ++After=cman.service corosync.service

Where did you find a cman.service?  The package does not even build cman
support.

Bastian

-- 
Spock: The odds of surviving another attack are 13562190123 to 1, Captain.