[acpid/f14/master] * Fri Sep 03 2010 Jiri Skala <jskala at redhat.com> - 2.0.5-3 - fixed #629740 - acpid doesn't fork, but

Jiri Skala jskala at fedoraproject.org
Fri Sep 10 13:35:32 UTC 2010


commit d5394ae954c24bc6558508ed8fe0f554ea848a80
Author: Jiri Skala <jskala at localhost.localdomain>
Date:   Fri Sep 10 15:35:36 2010 +0200

    * Fri Sep 03 2010 Jiri Skala <jskala at redhat.com> - 2.0.5-3
    - fixed #629740 - acpid doesn't fork, but systemd unit file claims otherwise

 acpid.service |   11 +++++++++--
 1 files changed, 9 insertions(+), 2 deletions(-)
---
diff --git a/acpid.service b/acpid.service
index 4b2dbed..937e8fd 100644
--- a/acpid.service
+++ b/acpid.service
@@ -2,9 +2,16 @@
 Description=ACPI Event Daemon
 After=syslog.target
 
+# This could probably benefit from socket activation, but honestly I think it
+# is time for acpid to go away, and hence I am not planning to spend the time
+# to add socket activation here. We use Type=forking to ensure that the
+# communication sockets are in place before boot proceeds with any service
+# needing this service. Would acpid support socket activation we could use
+# Type=simple here.
+
 [Service]
-Type=simple
-ExecStart=/usr/sbin/acpid -f
+Type=forking
+ExecStart=/usr/sbin/acpid
 
 [Install]
 WantedBy=multi-user.target


More information about the scm-commits mailing list