[syslog-ng] [Bug 126] [systemd] Add installation of systemd unit file via build-sys

[bugzilla at bugzilla.balabit.com]

https://bugzilla.balabit.com/show_bug.cgi?id=126


Tom Gundersen <teg at jklm.no> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |teg at jklm.no




--- Comment #6 from Tom Gundersen <teg at jklm.no>  2011-06-26 17:49:16 ---
@Balazs,Dave: it's great to see work on systemd support in syslog-ng!

A few (independent) comments:

1) the installation directories upstream should care about are:
  - /lib/systemd/system, where the unit files should be installed by the syslog-ng package, and
  - /usr/local/lib/systemd/system, where the unit files should be installed when you compile your own package.

   We should never install anything into /etc/systemd/system, this is for the admin only.

2) Just a quick note about something I noticed in #125: we should not need to alter syslog.socket.
   The correct behavior is that syslog.sockt, should start systemd-kmsg-syslogd.service if nothing is running,
   and when syslog-ng.service/rsyslog.service is started they take over the socket. At least that's the way I
   understood it.

3) The reason ExecReload was removed from rsyslog.service was that rsyslog does not support HUP at all
   <http://git.adiscon.com/?p=rsyslog.git;a=commit;h=fd26a42bdc04eaf497cafd9ef806a54f3de1a7e9>. While
   a synchronous reload would be ideal, I agree that keeping the HUP stuff is the correct thing to do for the
   time being.

Looking forward to this work trickling down to the Arch packages soon :-)


-- 
Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.