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

Bernhard Schmidt berni at debian.org
Wed May 10 06:36:29 UTC 2017


On Mon, May 08, 2017 at 10:11:45PM +0200, Andreas Henriksson wrote:

Hi,

> I think it's much better to implement this logic natively in the daemon,
> so please see attached patch. Please note that it's gotten basically no
> testing at all, so review carefully.

I really have to admit that patching the upstream source to search in
some Debian specific runtime directory is not making me particularly
happy. This needs documentation and forward-porting etc.

Why not just make a little wrapper shell script that parses
/etc/default/ntp and does the cfg selection dance, then exec() ntp? This
would get all systemd features in a way easy to understand while
maintaining backward compatibility. Other than a "needless" invocation
of /bin/sh (which is an argument I'm still not buying) I see no
downsides.

Kurt, what do you think?

Bernhard
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-ntp-maintainers/attachments/20170510/f8e8bb7f/attachment.sig>


More information about the pkg-ntp-maintainers mailing list