On Wed, 22 Jul 2015 19:37:28 +0200 Andreas Steinel <a.stei...@gmail.com> 
wrote:
> Package: watchdog
> Version: 5.14-3
> Severity: important
> 
> If you now add the following line to /lib/systemd/system/watchdog.service to
> the install section, then it works
> 
>     [Install]
>     WantedBy=multi-user.target
> 
> as expected. Systemctl enable works and the daemon gets started on boot.

I can confirm this. 
What is odd is that it is available in  
http://http.debian.net/debian/pool/main/w/watchdog/watchdog_5.14.orig.tar.gz

But it is actively stripped out in 
http://http.debian.net/debian/pool/main/w/watchdog/watchdog_5.14-3.diff.gz :

--- watchdog-5.14.orig/debian/watchdog.service
+++ watchdog-5.14/debian/watchdog.service
@@ -1,15 +1,14 @@
 [Unit]
 Description=watchdog daemon
-After=systemd-update-utmp.service
-After=runlevel1.target runlevel2.target runlevel3.target runlevel4.target 
runlevel5.target
+Conflicts=wd_keepalive.service
+After=multi-user.target
+OnFailure=wd_keepalive.service
 
 [Service]
 Type=forking
 EnvironmentFile=/etc/default/watchdog
-ExecStartPre=/bin/sh -c '[ -z "${watchdog_module}" ] || [ 
"${watchdog_module}" = "none" ] || echo /sbin/modprobe $watchdog_module'
-ExecStartPre=/bin/systemctl stop wd_keepalive
-ExecStart=/usr/sbin/watchdog $watchdog_options
-ExecStopPost=/bin/systemctl start wd_keepalive
+ExecStartPre=/bin/sh -c '[ -z "${watchdog_module}" ] || [ 
"${watchdog_module}" = "none" ] || /sbin/modprobe $watchdog_module
+ExecStart=/bin/sh -c '[ $run_watchdog != 1 ] || exec /usr/sbin/watchdog 
$watchdog_options'
+ExecStopPost=/bin/sh -c '[ $run_wd_keepalive != 1 ] || false'
 
 [Install]
-WantedBy=multi-user.target

I have no idea for the motivation of that change.

The above quote also shows the incorrect modification of the ExecStartPre line 
(bug #798294 and bug #783166), namely the missing trailing ' character.

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to