On Fri, Nov 21, 2014 at 04:19:06PM +0100, Alessandro Ghedini wrote:
> On lun, set 15, 2014 at 12:14:07 +0200, Agustin Martin wrote:
> > Control: tag 748668 +patch
> 
> Maybe I'm missing something but your email doesn't provide any patch.

Thanks,

As I wrote when forcemerging 768774, 

  That info should be enough for a fix, so I tagged that bug report
  as +patch even if no explicit patch was added.

> Anyway, I wrote a patch that fixes this bug (see attached) as you suggested. 
> The
> patch does:
> 
> 1 Patches the systemd service file to not start unless slim is the default
>   manager (as per /etc/X11/default-display-manager).
> 2 Removes the [Install] section in the systemd service file so that it doesn't
>   take over the system on install. This also removes the WantedBy statement 
> that
>   was added as part of the fix for #741511 which is IMO wrong.
> 3 Creates the /etc/systemd/system/display-manager.service in postinst if slim 
> is
>   the dfault manager.
> 
> Note that 1 and 3 above actually come from the lightdm package (I only removed
> the part where it reloads dbus, which is not used by slim).

Thanks again. 

I did not write the actual patch because I am not yet using systemd, so I could 
not
test it properly. 
 
> I think that this patch should be included in jessie.

Me too, 

Maintainers?

Regards,

-- 
Agustin


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to