Update ebusd.service #502
Update ebusd.service #502
Conversation
eliminating the warning during booting the system: dmesg | grep ebus systemd[1]: /etc/systemd/system/ebusd.service:10: PIDFile= references a path below legacy directory /var/run/, updating /var/run/ebusd.pid → /run/ebusd.pid; please update the unit file accordingly.
| @@ -7,7 +7,7 @@ ConditionPathExists=/var/log | |||
| Type=forking | |||
| Restart=always | |||
| RestartSec=30 | |||
| PIDFile=/var/run/ebusd.pid | |||
| PIDFile=/run/ebusd.pid | |||
There was a problem hiding this comment.
according to https://www.gnu.org/prep/standards/html_node/Directory-Variables.html it should be /var/run and not /run
There was a problem hiding this comment.
according to systemd document:
https://www.freedesktop.org/software/systemd/man/systemd.exec.html#RuntimeDirectory=
RuntimeDirectory= /run/
There was a problem hiding this comment.
then it would have to be done differently for initd/systemd. plus you'd also have to tell ebusd about the changed location
There was a problem hiding this comment.
this you know better than, I just wanted to point it out that ev. path change is required.
thanks
There was a problem hiding this comment.
even GNUInstallDirs of recent cmake still has LOCALSTATEDIR/run in it with LOCALSTATEDIR usually being /var, so I'll keep it until that changes
|
A better option for Systemd would be to run it in foreground mode, no need for PID files, etc |
eliminating the warning during booting the system:
dmesg | grep ebus
systemd[1]: /etc/systemd/system/ebusd.service:10: PIDFile= references a path below legacy directory /var/run/, updating /var/run/ebusd.pid → /run/ebusd.pid; please update the unit file accordingly.