Control: reopen -1

On 2024-03-12 11:10, MOESSBAUER, Felix wrote:
On Tue, 2024-03-12 at 10:33 -0500, Richard Laager wrote:
This is intentional. The logging is optional and whether the
directory
exists controls this.

Hi Richard,

that's a quite uncommon interface. Is this at least documented
somewhere?

ntp.conf

Also the "error" should be downgraded to a "warning" at
least.

Didn't I do that?

commit b06f1d8177a9b0a2593947a2ebefcb43e94ac281
Author: Santiago Vila <sanv...@debian.org>
Date:   Sun Dec 24 15:36:25 2023 -0600

    Downgrade missing stats dir log severity

    The Debian package does not create /var/log/ntpsec by default.  The
    admin  is directed (by a comment in ntp.conf) to create it if and only
    if they want logging.  However, upstream ntpsec logs an error message
    if the log directory does not exist.

    Closes: 1049424
    Signed-off-by: Richard Laager <rlaa...@debian.org>
    [The commit message / patch description is my wording.]


One advantage of that is the ntpsec-ntpviz package can create that
directory to enable logging, which is required for ntpviz to be
useful.
Otherwise, it would have to edit the config file, which is riskier.

Well... for that conf.d style configs are usually used.

Yeah. ntpsec supports those _now_. (I don't know that it did at the time.) So maybe that's a better answer.

--
Richard

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature

Reply via email to