teknoraver / rpms / systemd

Forked from rpms/systemd 3 months ago
Clone

Blame SOURCES/0347-man-clarify-that-the-ExecReload-command-should-be-sy.patch

84b277
From 191c7ee3810782c8968def5b23e46b442a8f530d Mon Sep 17 00:00:00 2001
84b277
From: Lennart Poettering <lennart@poettering.net>
84b277
Date: Fri, 16 May 2014 01:33:22 +0200
84b277
Subject: [PATCH] man: clarify that the ExecReload= command should be
84b277
 synchronous
84b277
84b277
http://lists.freedesktop.org/archives/systemd-devel/2014-May/019054.html
84b277
84b277
(cherry-picked from 33169701b0640d3629d4c36cf8c71dc26d2cb7e1)
84b277
84b277
Resolves: #1147524
84b277
---
84b277
 man/systemd.service.xml | 14 ++++++++++++++
84b277
 1 file changed, 14 insertions(+)
84b277
84b277
diff --git a/man/systemd.service.xml b/man/systemd.service.xml
84b277
index 80d6951..8df4b87 100644
84b277
--- a/man/systemd.service.xml
84b277
+++ b/man/systemd.service.xml
84b277
@@ -525,6 +525,20 @@ ExecStart=/bin/echo $ONE $TWO ${TWO}
84b277
                                 following:</para>
84b277
 
84b277
                                 <programlisting>/bin/kill -HUP $MAINPID</programlisting>
84b277
+
84b277
+                                <para>Note however that reloading a
84b277
+                                daemon by sending a signal (as with
84b277
+                                the example line above) is usually not
84b277
+                                a good choice, because this is an
84b277
+                                asynchronous operation and hence not
84b277
+                                suitable to order reloads of multiple
84b277
+                                services against each other. It is
84b277
+                                strongly recommended to set
84b277
+                                <varname>ExecReload=</varname> to a
84b277
+                                command that no only triggers a
84b277
+                                configuration reload of the daemon,
84b277
+                                but also synchronously waits for it
84b277
+                                complete.</para>
84b277
                                 </listitem>
84b277
                         </varlistentry>
84b277