[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#755359: [kdm] systemd seem to fail to start display-manager.service



On Fri, Apr 03, 2015 at 09:05:17AM +0200, John Paul Adrian Glaubitz wrote:
> On 04/02/2015 10:15 PM, Moritz Mühlenhoff wrote:
> > My patch from https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=754314
> > retains the kdmrc customisation currently present in the sysvinit script;
> > maybe you can fold that one into your patch?
> 
> Hmm, I haven't realized that the display manager's configuration file is
> touch by the init script. I think that's definitely going beyond to what
> an init script is supposed to do.

Yeah, but I think the behaviour of the systemd unit should retain the
established behaviour of the svsvinit script.

> I also noticed that the systemd service file you wrote contains an
> [Install] section but this section is not allowed in service files
> for display managers at the time being since their service files
> are installed through the debconf mechanism when configuring the
> default display manager. This is also the reason why you are having
> problems activating the service.

Yeah, the logic to handle display managers appeared later, my patch
is nearly nine months old by now and much has changed in the systemd
Debian integration.

> OTOH, I was a bit surprised to see that you removed the ConsoleKit
> dependency. I wasn't even aware that kdm actually supports logind,
> does it?

See the references in my original bug submission.

> Maybe we can come up with a good patch if we merge both our patches
> into one that incorporates all important aspects discussed above.

I'd be in favour of adding the ExecStartPre of kdm-debian-setup-config
(but I'm unsure whether people actually use this), but I won't be able
to spend further time on this ATM.

Cheers,
        Moritz


Reply to: