On 8/6/19 2:53 AM, Masahiro Honda wrote:
> Richard Laager wrote:
>> The -s seems to allow multiple ntpdate-debian processes to run
>> simultaneously, which seems bad. It seems like a correct change to
>> remove that, though it makes me wonder why it was there in the first
>> place. (This code was inherited from the ntp package.)
>>
>> Are you getting interesting logging when ntpdate-debian runs?
> 
> I was mistaken about the -s.

Me too. The -s is for ntpdate-debian, not flock.

> In ntpdate of the ntp package, -s means to divert logging output to the
> system syslog facility.

Right. In ntpsec's ntpdate, this is converted to -p for ntpdig, which is
wrong. This is what is breaking things for you.

rlaager@watermelon:~/src/ntpsec/ntpsec-pkg$ attic/ntpdate -n -s
ntpdig -s -j -t 1 -p

rlaager@watermelon:~/src/ntpsec/ntpsec-pkg$ ntpdig -s -j -t 1 -p
option -p requires argument

I've submitted a patch upstream to fix this correctly:
https://gitlab.com/NTPsec/ntpsec/merge_requests/1011/

I suppose I'll also need to "Depends: bsdutils" to guarantee that
logger(1) is available.

-- 
Richard

Reply via email to