From 29294d21cf82323bf04a9dbb5a03d48d6f758822 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Mon, 1 Jul 2024 12:40:33 +0200 Subject: [PATCH] units: add dep on systemd-logind.service by user@.service Let's make sure logind is accessible by the time user@.service runs, and that logind stays around as long as it does so. Addresses an issue reported here: https://lists.freedesktop.org/archives/systemd-devel/2024-June/050468.html This addresses an issued introduced by 278e815bfa3e4c2e3914e00121c37fc844cb2025, which dropped the a dependency from user@.service systemd-user-sessions.service without replacement. While dropping that dependency does make sense, it should have been replaced with the weaker dependency on systemd-logind.service, hence fix that now. user@.service is after all a logind concept, hence logind really should be around for its lifetime. systemd-user-sessions.service is a later milestone that only really should apply to regular users (not root), hence it's too strong a requirement. --- units/user-runtime-dir@.service.in | 2 +- units/user@.service.in | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/units/user-runtime-dir@.service.in b/units/user-runtime-dir@.service.in index 5fb5cad36a6..241e9267bb0 100644 --- a/units/user-runtime-dir@.service.in +++ b/units/user-runtime-dir@.service.in @@ -10,7 +10,7 @@ [Unit] Description=User Runtime Directory /run/user/%i Documentation=man:user@.service(5) -After=dbus.service +After=systemd-logind.service dbus.service IgnoreOnIsolate=yes [Service] diff --git a/units/user@.service.in b/units/user@.service.in index 5efb12a8601..56954657472 100644 --- a/units/user@.service.in +++ b/units/user@.service.in @@ -11,7 +11,7 @@ Description=User Manager for UID %i Documentation=man:user@.service(5) BindsTo=user-runtime-dir@%i.service -After=user-runtime-dir@%i.service dbus.service systemd-oomd.service +After=systemd-logind.service user-runtime-dir@%i.service dbus.service systemd-oomd.service IgnoreOnIsolate=yes [Service]