On Wed, Oct 07, 2009 at 01:53:52PM +0200, Petter Reinholdtsen wrote: > > Package: wdm > Version: 1.28-2.4 > Severity: important > User: initscripts-ng-de...@lists.alioth.debian.org > Usertags: incorrect-dependency > > I got a report from a Debian user about problems with wdm with > dependency based boot sequencing, and had a look. The init.d/wdm > dependency information look like this: > > ### BEGIN INIT INFO > # Provides: wdm > # Required-Start: $remote_fs > # Required-Stop: $remote_fs > # Should-Start: console-screen > # Should-Stop: console-screen > # Default-Start: 2 3 4 5 > # Default-Stop: 0 1 6 > # Short-Description: start or stop the WINGs display manager > ### END INIT INFO > > This will make it start very early in the boot. The report indicated > that X now need dbus and hal, and the gdm init.d script depend on > acpid and network-manager as well. If wdm need the same dependencies, > I recommend changing the should-start line to look like this: > > # Should-Start: console-screen kbd acpid dbus hal network-manager > > This will move the init.d script further back in the boot sequence, > and might increase the chance of wdm starting as it should during > boot. It will also solve bug #548105. I do not really know the > requirement fo wdm, so I hope you as the maintainer can review this > list and adjust it to better fit the need of wdm.
I do not know the details about the reported problem, but here it is weird making system unusable with dependency based boot. With dependency based boot, in my box wdm is started too early, launched in the middle of tty creation, locking tty2. getty tries hard to open tty2, and I get that system gets completely locked soon after login (not sure if because of getty), with no other choice than the reset button (Not even Magic SysRQ works). While this package works with traditional boot, so it does not fail with all systems (and strictly speaking important severity may be correct), I think that failing with dependency based boot in that way makes it more grave than plain important and so, NMU candidate. I have prepared a NMU to deal with this. Besides suggestions by you and Michael Schutte it contains some other changes in the dependencies inspired in xdm/kdm/gdm dependencies. This NMU also deals with a couple of lintian errors about obsolete build-deps. diff is attached. Hope this helps, -- Agustin
diff -wu wdm-1.28/debian/wdm.init wdm-1.28/debian/wdm.init --- wdm-1.28/debian/wdm.init +++ wdm-1.28/debian/wdm.init @@ -1,10 +1,9 @@ #!/bin/sh ### BEGIN INIT INFO # Provides: wdm -# Required-Start: $remote_fs -# Required-Stop: $remote_fs -# Should-Start: console-screen -# Should-Stop: console-screen +# Required-Start: $local_fs $remote_fs x11-common +# Required-Stop: $local_fs $remote_fs +# Should-Start: console-screen kbd acpid dbus hal network-manager # Default-Start: 2 3 4 5 # Default-Stop: 0 1 6 # Short-Description: start or stop the WINGs display manager diff -wu wdm-1.28/debian/control wdm-1.28/debian/control --- wdm-1.28/debian/control +++ wdm-1.28/debian/control @@ -3,7 +3,7 @@ Priority: optional Maintainer: Vlad Shakhov <lumpen.intellect...@gmail.com> Standards-Version: 3.7.3 -Build-Depends: debhelper (>= 5), libjpeg62-dev, libpam0g-dev, libwings-dev, libpng12-dev, libungif4-dev, libwraster3-dev, libxpm-dev, zlib1g-dev, libx11-dev, libxau-dev, libxdmcp-dev, libxinerama-dev, libxkbfile-dev, libxmu-dev, libxpm-dev, libxt-dev, x-dev, libxft-dev, dpatch, po-debconf +Build-Depends: debhelper (>= 5), libjpeg62-dev, libpam0g-dev, libwings-dev, libpng12-dev, libgif-dev, libwraster3-dev, libxpm-dev, zlib1g-dev, libx11-dev, libxau-dev, libxdmcp-dev, libxinerama-dev, libxkbfile-dev, libxmu-dev, libxpm-dev, libxt-dev, x11proto-core-dev, libxft-dev, dpatch, po-debconf Package: wdm Architecture: any diff -wu wdm-1.28/debian/changelog wdm-1.28/debian/changelog --- wdm-1.28/debian/changelog +++ wdm-1.28/debian/changelog @@ -1,3 +1,17 @@ +wdm (1.28-3.2) unstable; urgency=low + + * Non-maintainer upload. + * Fix dependencies in init.d script so it is started later + in the boot sequence.(Closes: #548105, #550060), + - Should-Start: Add kbd acpid dbus hal network-manager + - Add $local_fs to Required-Start and Required-Stop. + - Add x11-common to Required-Start. + * Fix lintian error: build-depends-on-obsolete-package: + - Change x-dev Build-Dep to x11proto-core-dev (Closes: #515470). + - Change libungif4-dev Build-Dep to libgif-dev. + + -- Agustin Martin Domingo <agmar...@debian.org> Tue, 03 Nov 2009 13:00:12 +0100 + wdm (1.28-3.1) unstable; urgency=low * Non-maintainer upload.
