Control: tags -1 patch

Hi all,

On Sun, 30 Apr 2017 12:38:33 +0200 Lukas <h...@zwergenfeste.ch> wrote:
> Hi
> 
> Got the same problem on stretch.
> 
> Changing the systemd service type to simple solves the problem for me:
> 
> echo "Type=simple" >> /lib/systemd/system/slim.service
> systemctl daemon-reload
> 
> So, i guess the dbus name can't be acquired...
> 
> BR
> 
> Lukas 

So, to recap, this is due to the fix for #860465, which is part of 
1.3.6-5. It appears that in order to fix plymouth handling, the service 
file was modified to include a number of things slim currently does 
*not* support:

 - BusName=org.freedesktop.DisplayManager which was added to the unit 
   file silently converts the unit to Type=dbus and instructs systemd to 
   expect slim to bind the org.freedesktop.DisplayManager endpoint on 
   the system bus before it marks the service as running. However, 
   grepping the source indicates that slim is completely unaware of 
   dbus. Thus this directive causes systemd to endlessly restart slim 
   every TimeoutStartSec seconds.

 - Moreover, slim seems to not be able to stop plymouth by itself, 
   contrary to what is stated in the service file. This is also 
   reflected by the manual handling of plymouth in the upstart and 
   initscript files.

The attached debdiff resolves the restart loop, while ensuring that slim 
plays nice with plymouth. If nobody responds, I intend to NMU the 
package tomorrow.

Regards,
Apollon
diff -Nru slim-1.3.6/debian/changelog slim-1.3.6/debian/changelog
--- slim-1.3.6/debian/changelog	2017-04-23 19:14:21.000000000 +0300
+++ slim-1.3.6/debian/changelog	2017-05-02 16:48:36.000000000 +0300
@@ -1,3 +1,11 @@
+slim (1.3.6-5.1) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * slim.service: drop BusName and Conflicts/OnFailure=plymouth-quit.service
+    (Closes: #861474)
+
+ -- Apollon Oikonomopoulos <apoi...@debian.org>  Tue, 02 May 2017 16:48:36 +0300
+
 slim (1.3.6-5) unstable; urgency=medium
 
   * Fix bugged Debian-Lines theme. (Closes: #776693)
diff -Nru slim-1.3.6/debian/patches/fix-missing-plymouth-handling.patch slim-1.3.6/debian/patches/fix-missing-plymouth-handling.patch
--- slim-1.3.6/debian/patches/fix-missing-plymouth-handling.patch	2017-04-23 18:38:28.000000000 +0300
+++ slim-1.3.6/debian/patches/fix-missing-plymouth-handling.patch	2017-05-02 16:46:20.000000000 +0300
@@ -5,20 +5,11 @@
 
 --- a/slim.service
 +++ b/slim.service
-@@ -2,7 +2,16 @@
+@@ -1,6 +1,6 @@
+ [Unit]
  Description=SLiM Simple Login Manager
- After=systemd-user-sessions.service
+-After=systemd-user-sessions.service
++After=systemd-user-sessions.service plymouth-quit.service
  
-+# replaces plymouth-quit since slim quits plymouth on its own
-+Conflicts=plymouth-quit.service
-+After=plymouth-quit.service
-+
-+# slim takes responsibility for stopping plymouth, so if it fails
-+# for any reason, make sure plymouth still stops
-+OnFailure=plymouth-quit.service
-+
  [Service]
  ExecStartPre=/bin/sh -c '[ "$(cat /etc/X11/default-display-manager 2>/dev/null)" = "/usr/bin/slim" ]'
- ExecStart=/usr/bin/slim -nodaemon
- Restart=always
-+BusName=org.freedesktop.DisplayManager

Reply via email to