[pkg-ntp-maintainers] Bug#635752: Bug#635752: ntp: Please include systemd service file

[Kurt Roeckx]

On Thu, Jul 28, 2011 at 04:25:16PM +0200, Michael Stapelberg wrote:
> Package: ntp
> Severity: wishlist
> Tags: patch
> 
> 
> Hello,
> 
> I created a systemd service file for ntp.
> systemd is a sysvinit replacement (see [1] for more information).
> 
> Please include this file in /lib/systemd/system/ in the next upload of this 
> package.
> 
> I also sent this file upstream, so hopefully it will be included in future 
> releases anyway.
> 
> If you have any questions, please do not hesitate to contact me.

I'm not sure what Debian's policy is for this.  As far as I know,
this is all still open for debate.

Also, the current file is missing alot of the configuarion options
that are curruntly supported, like having an ntp.conf provided
by dhcp.

We now pass the uid/gid of $RUNASUSER, not some (fixed) name and group.

It's now passing -n and not passing the PID file.  I guess at least
the PID file part is normal for systemd.

It has:
After=network.target

Which might be different then:
# Required-Start:  $network $remote_fs $syslog

And:
[Install]
WantedBy=multi-user.target

Whatever that means.

Please also note that in Debian currently nothing should wait for
ntp to be started, and if something does it's probably broken.

I will need more information about systemd than what you pointed
to.


Kurt