[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