10/12: services: openntpd: Remove support for deprecated "-s" option.

[guix-commits]

civodul pushed a commit to branch master
in repository guix.

commit 181f290aa262eae22819c6234b4065786afb14cf
Author: Simon South <simon@simonsouth.net>
AuthorDate: Mon Jan 11 12:04:59 2021 -0500

    services: openntpd: Remove support for deprecated "-s" option.
    
    * gnu/services/networking.scm (openntpd-configuration): Remove
    "allow-large-adjustment?" field.
    (openntpd-shepherd-service): Remove use of "allow-large-adjustment?"
    configuration field and "-s" daemon option.
    * tests/networking.scm (%openntpd-conf-sample): Remove
    "allow-large-adjustment?" field.
    * doc/guix.texi (Networking Services)[openntpd-service-type]: Remove
    "allow-large-adjustment?" field from sample configuration.
    [openntpd-configuration]: Remove description of "allow-large-adjustment?"
    field.
    
    Signed-off-by: Ludovic Courtès <ludo@gnu.org>
---
 doc/guix.texi               |  6 +-----
 gnu/services/networking.scm | 12 +++---------
 tests/networking.scm        |  3 +--
 3 files changed, 5 insertions(+), 16 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 1234834..c92f3a5 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -16246,8 +16246,7 @@ clock synchronized with that of the given servers.
   (listen-on '("127.0.0.1" "::1"))
   (sensor '("udcf0 correction 70000"))
   (constraint-from '("www.gnu.org"))
-  (constraints-from '("https://www.google.com/";))
-  (allow-large-adjustment? #t)))
+  (constraints-from '("https://www.google.com/";))))
 
 @end lisp
 @end deffn
@@ -16285,9 +16284,6 @@ a constraint.
 As with constraint from, specify a list of URLs, IP addresses or hostnames of
 HTTPS servers to provide a constraint.  Should the hostname resolve to multiple
 IP addresses, @code{ntpd} will calculate a median constraint from all of them.
-@item @code{allow-large-adjustment?} (default: @code{#f})
-Determines if @code{ntpd} is allowed to make an initial adjustment of more
-than 180 seconds.
 @end table
 @end deftp
 
diff --git a/gnu/services/networking.scm b/gnu/services/networking.scm
index dd40613..a4d4ac0 100644
--- a/gnu/services/networking.scm
+++ b/gnu/services/networking.scm
@@ -560,9 +560,7 @@ make an initial adjustment of more than 1,000 seconds."
   (constraint-from         openntpd-constraint-from
                            (default '()))
   (constraints-from        openntpd-constraints-from
-                           (default '()))
-  (allow-large-adjustment? openntpd-allow-large-adjustment?
-                           (default #f))) ; upstream default
+                           (default '())))
 
 (define (openntpd-configuration->string config)
 
@@ -594,8 +592,7 @@ make an initial adjustment of more than 1,000 seconds."
      "\n")))                              ;add a trailing newline
 
 (define (openntpd-shepherd-service config)
-  (let ((openntpd (openntpd-configuration-openntpd config))
-        (allow-large-adjustment? (openntpd-allow-large-adjustment? config)))
+  (let ((openntpd (openntpd-configuration-openntpd config)))
 
     (define ntpd.conf
       (plain-file "ntpd.conf" (openntpd-configuration->string config)))
@@ -607,10 +604,7 @@ make an initial adjustment of more than 1,000 seconds."
            (start #~(make-forkexec-constructor
                      (list (string-append #$openntpd "/sbin/ntpd")
                            "-f" #$ntpd.conf
-                           "-d" ;; don't daemonize
-                           #$@(if allow-large-adjustment?
-                                  '("-s")
-                                  '()))
+                           "-d") ;; don't daemonize
                      ;; When ntpd is daemonized it repeatedly tries to respawn
                      ;; while running, leading shepherd to disable it.  To
                      ;; prevent spamming stderr, redirect output to logfile.
diff --git a/tests/networking.scm b/tests/networking.scm
index c494a48..f242137 100644
--- a/tests/networking.scm
+++ b/tests/networking.scm
@@ -68,8 +68,7 @@
    (listen-on '("127.0.0.1" "::1"))
    (sensor '("udcf0 correction 70000"))
    (constraint-from '("www.gnu.org"))
-   (constraints-from '("https://www.google.com/";))
-   (allow-large-adjustment? #t)))
+   (constraints-from '("https://www.google.com/";))))
 
 (test-assert "openntpd configuration generation sanity check"