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. 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. My patch basically copies the behavior of the lightdm package which is already in the archives and works reasonably, so I wouldn't want to make this patch any more complicated at this point. 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? Maybe we can come up with a good patch if we merge both our patches into one that incorporates all important aspects discussed above. Adrian -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer - glaub...@debian.org `. `' Freie Universitaet Berlin - glaub...@physik.fu-berlin.de `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913 -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
