On 05/11/2018 02:56 AM, Julien Goodwin wrote: > I also think the test in the init script should probably change to > if -e /run/ntp.conf.dhcp, since /run is created on boot, otherwise > there's surprising behaviour if the admin edits ntp.conf, restarts the > service, then discovers a DHCP update has reset the config. An > unconditional test on file existence is likely less astonishing for > users.
This makes sense. This change would bring ntpsec in line with ntp. It also brings the ntpsec sysvinit script in line with the ntpsec systemd wrapper. I looked through the git history, and I can't find a good reason for why this "-nt" test existed. It's unclear if that came from an older version of ntp or was something I cooked up. I'll make that change. I have some pending fixes to deal with, and the exact order and timing is contingent on a few different things. -- Richard
