Package: sleepd Version: 1.11 Severity: important Tags: patch User: [email protected] Usertags: incorrect-dependency User: [email protected] Usertags: piuparts
With dependency based boot sequencing, I discovered what I believe is a bug in the init.d script of this package. The init.d script depend on hal and dbus, while the package only recommend on the packages providing these scripts. This break installation of the package on systems where hal and dbus isn't installed. I recommend making the init.d dependencies optional. The installation problem was discovered by piuparts, see <URL: http://piuparts.debian.org/sid/fail/sleepd_2.00.log > for the test log. These are the error messages: insserv: Service dbus has to be enabled to start service sleepd insserv: Service hal has to be enabled to start service sleepd insserv: exiting now! I'm setting severity to important, as the recommended packages will be installed in the normal case. This patch should solve the issue. diff -ur ../sleepd-2.00/debian/init ../sleepd-2.00-pere/debian/init --- ../sleepd-2.00/debian/init 2009-05-15 20:57:04.000000000 +0200 +++ ../sleepd-2.00-pere/debian/init 2009-09-27 20:25:42.000000000 +0200 @@ -2,8 +2,10 @@ # ### BEGIN INIT INFO # Provides: sleepd -# Required-Start: $syslog $local_fs $remote_fs dbus hal -# Required-Stop: $syslog $local_fs $remote_fs dbus hal +# Required-Start: $syslog $local_fs $remote_fs +# Required-Stop: $syslog $local_fs $remote_fs +# Should-Start: dbus hal +# Should-Stop: dbus hal # Default-Start: 2 3 4 5 # Default-Stop: 1 # Short-Description: Sleep daemon Happy hacking, -- Petter Reinholdtsen -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
