From: Lennart Poettering <lennart@poettering.net>
Date: Tue, 1 Feb 2022 17:11:04 +0000 (+0100)
Subject: units: we need systemd-journald.service from systemd-journal-flush.service
X-Git-Tag: v251-rc1~387
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=23b1e8d087c9e8c5a2cdcc6a91510a4e7ca8f72f;p=thirdparty%2Fsystemd.git

units: we need systemd-journald.service from systemd-journal-flush.service

This is a follow-up for d5ee050ffc9d413253932d9340ade8c8fb111092, and
reintroduces a requirement dep from systemd-journal-flush.service onto
systemd-journald.service, but a weaker one than originally: a Wants= one
instead of a Requires= one.

Why? Simply because the service issues an IPC call to the journald,
hence it should pull it in. (Note that socket activation doesn't happen
for the Varlink socket it uses, hence we should pull in the service
itself.)
---

diff --git a/units/systemd-journal-flush.service b/units/systemd-journal-flush.service
index 6efb8734a77..5d0b811ae39 100644
--- a/units/systemd-journal-flush.service
+++ b/units/systemd-journal-flush.service
@@ -11,6 +11,7 @@
 Description=Flush Journal to Persistent Storage
 Documentation=man:systemd-journald.service(8) man:journald.conf(5)
 DefaultDependencies=no
+Wants=systemd-journald.service
 After=systemd-journald.service systemd-remount-fs.service
 Before=systemd-tmpfiles-setup.service
 RequiresMountsFor=/var/log/journal