Package: screen
Version: 4.0.3-8
Severity: important
Tags: patch
User: [EMAIL PROTECTED]
Usertags: incorrect-dependency

Hi,

On insserv enabled system, a conflict between LSB Default-Start data and
options passed to update-rc.d in screen postinst was exposed at package upgrade
time.

Setting up screen (4.0.3-8) ...
Installing new version of config file /etc/init.d/screen-cleanup ...
insserv: Warning, current start runlevel(s) of script `screen-cleanup' (S ) 
overwrites defaults (2 3 4 5 ).

screen debian/rules
---
dh_installinit --no-start --init-script='screen-cleanup' 
--update-rcd-params='start 70 S .'

/var/lib/dpkg/info/screen.postinst
---
# Automatically added by dh_installinit
if [ -x "/etc/init.d/screen-cleanup" ]; then
        update-rc.d screen-cleanup start 70 S . >/dev/null || exit $?
fi
# End automatically added section

/etc/init.d/screen-cleanup
---
# Default-Start:     2 3 4 5

Either the options given to dh_installinit are wrong, or the LSB header is
wrong. My guess is that the LSB header is incorrect. Attached a patch.

As a side note, I believe that if you declare a dependency on virtual facility
"$remote_fs", that implies "$local_fs", therefore you could remove $local_fs
from the Required-Start and Required-Stop field.

For more information, see http://wiki.debian.org/LSBInitScripts

Thanks, Kel.
---
diff -Nrup screen-4.0.3/debian/init screen-4.0.3.insserv/debian/init
--- screen-4.0.3/debian/init    2008-04-25 17:33:05.000000000 +1000
+++ screen-4.0.3.insserv/debian/init    2008-04-25 17:32:40.000000000 +1000
@@ -8,7 +8,7 @@
 # Provides:          screen-cleanup
 # Required-Start:    $local_fs $remote_fs
 # Required-Stop:     $local_fs $remote_fs
-# Default-Start:     2 3 4 5
+# Default-Start:     S
 # Default-Stop:
 # Short-Description: screen sessions cleaning
 ### END INIT INFO



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to