On Sat, Aug 11, 2012 at 05:26:29PM +0100, Roger Leigh wrote: > On Wed, Jul 11, 2012 at 02:06:52PM +1000, James Tocknell wrote: > > On Mon, Jul 2, 2012 at 6:04 PM, Yves-Alexis Perez <cor...@debian.org> wrote: > > > On lun., 2012-07-02 at 10:02 +0200, Petter Reinholdtsen wrote: > > >> [Yves-Alexis Perez] > > >> > James, in any case, I think providing /etc/init.d/.depend.* might > > >> > help solving this. > > >> > > >> Actually, it is better to provide the output from > > >> /usr/share/insserv/make-testsuite, which allow the boot sequence to be > > >> reproduced using dummy init.d scripts. > > >> > > >> > Please keep me on CC: so I'm informed of the development. It might > > >> > be interesting to provide /var/log/lightdm.log and > > >> > /var/log/lightdm-gtk-greeeter.log, although the fact it works with > > >> > startx might means it won't show anything. > > >> > > >> To debug this, try to disable concurrent booting (and keep dependency > > >> based boot sequencing), and see if it work better. Also, it would be > > >> useful to get the boot log (/var/log/boot.log) after enabling > > >> bootlogd. > > I've installed lightdm on my system, and I do see the occasional > failure to start. This would indicate some race in parallelisation > which would be indicative of a missing dependency.
Please could you try sysvinit with the patch attached here? This special cases lightdm in startpar, since gdm/kdm/xdm are already special cased here. http://anonscm.debian.org/gitweb/?p=collab-maint/sysvinit;a=commitdiff;h=34ee13ba8d509183b568c70511289f7f7579502b If you're unable to build yourself, let me know and I'll prepare some packages. Thanks, Roger -- .''`. Roger Leigh : :' : Debian GNU/Linux http://people.debian.org/~rleigh/ `. `' schroot and sbuild http://alioth.debian.org/projects/buildd-tools `- GPG Public Key F33D 281D 470A B443 6756 147C 07B3 C8BC 4083 E800
>From 34ee13ba8d509183b568c70511289f7f7579502b Mon Sep 17 00:00:00 2001 From: Roger Leigh <rle...@debian.org> Date: Sat, 11 Aug 2012 22:16:00 +0100 Subject: [PATCH 1/2] startpar: Add patch to special case lightdm --- debian/patches/78_startpar_lightdm.patch | 15 +++++++++++++++ debian/patches/series | 1 + 2 files changed, 16 insertions(+), 0 deletions(-) create mode 100644 debian/patches/78_startpar_lightdm.patch diff --git a/debian/patches/78_startpar_lightdm.patch b/debian/patches/78_startpar_lightdm.patch new file mode 100644 index 0000000..e110b8a --- /dev/null +++ b/debian/patches/78_startpar_lightdm.patch @@ -0,0 +1,15 @@ +Purpose: Add lightdm to list of special cased *dms +Fixes: Intermittent failure to start lightdm +Author: Roger Leigh +Status: Not yet submitted upstream +--- +--- a/startpar/makeboot.c 2012-08-11 21:58:53.000000000 +0100 ++++ b/startpar/makeboot.c 2012-08-11 22:00:57.000000000 +0100 +@@ -273,6 +273,7 @@ + if (! strcmp(node->name, "xdm") + || ! strncmp(node->name, "gdm", 3) + || ! strncmp(node->name, "kdm", 3) ++ || ! strncmp(node->name, "lightdm", 7) + || ! strcmp(node->name, "boot.udev") + || ! strcmp(node->name, "udev")) + importance = 100; diff --git a/debian/patches/series b/debian/patches/series index e4139b0..b70da91 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -16,6 +16,7 @@ 75_man_startpar_see_also.patch 76_startpar_ldflags.patch 77_startpar_skip_reason.patch +78_startpar_lightdm.patch 91_sulogin_lockedpw.dpatch 92_kfreebsd_ifdown.patch 93_run_initctl.patch -- 1.7.2.5
