chiark / gitweb /
login: revert lazy session-activation on non-VT seats
authorDavid Herrmann <dh.herrmann@gmail.com>
Thu, 28 Nov 2013 09:52:18 +0000 (10:52 +0100)
committerDavid Herrmann <dh.herrmann@gmail.com>
Thu, 28 Nov 2013 09:52:18 +0000 (10:52 +0100)
Existing applications like gdm already depend on new sessions to get
immediately activated on seats without VTs. Fixes a bug reported as:
  [systemd-devel] systemd 208:trouble with inactive user sessions at non-seat0 seats

This patch restores the original behavior. We either need to add a new
flag for session-creation or some other heuristic to avoid activating new
sessions in the future.

src/login/logind-seat.c

index b30c4ce..ca0e8d7 100644 (file)
@@ -413,8 +413,8 @@ int seat_attach_session(Seat *s, Session *session) {
         seat_send_changed(s, "Sessions", NULL);
 
         /* On seats with VTs, the VT logic defines which session is active. On
         seat_send_changed(s, "Sessions", NULL);
 
         /* On seats with VTs, the VT logic defines which session is active. On
-         * seats without VTs, we automatically activate the first session. */
-        if (!seat_has_vts(s) && !s->active)
+         * seats without VTs, we automatically activate new sessions. */
+        if (!seat_has_vts(s))
                 seat_set_active(s, session);
 
         return 0;
                 seat_set_active(s, session);
 
         return 0;