Bug#541396: mdadm: monitor init script should depend on $syslog

[Petter Reinholdtsen]

tags 541396 + patch
thanks

Here is a proposed patch to solve this issue.  Instead of depending on
checkroot and umountroot, I believe it is better to depend on the
virtual facility $local_fs, which represent the point when all local
file systems like /var/ are mounted.  The script need the mounted
/var/ file system, and thus checkroot is not enough when / and /var/
are on separate partitions.  Also, the $syslog dependendency should
also be used to make sure the syslog service is started before mdadm
starts.

Without this patch, the script will start too early in the boot
sequence, and some syslog messages might be lost.

diff -ur mdadm-3.0/debian/mdadm.init mdadm-3.0-pere/debian/mdadm.init
--- mdadm-3.0/debian/mdadm.init 2009-08-27 10:47:04.000000000 +0200
+++ mdadm-3.0-pere/debian/mdadm.init    2009-08-27 10:50:00.040714000 +0200
@@ -8,8 +8,8 @@
 #
 ### BEGIN INIT INFO
 # Provides:          mdadm
-# Required-Start:    checkroot
-# Required-Stop:     umountroot
+# Required-Start:    $local_fs $syslog
+# Required-Stop:     $local_fs $syslog
 # Should-Start:      module-init-tools
 # Default-Start:     2 3 4 5
 # Default-Stop:      0 1 6

Happy hacking,
-- 
Petter Reinholdtsen