summary refs log tree commit diff
path: root/gnu/installer
diff options
context:
space:
mode:
authorLudovic Courtès <ludo@gnu.org>2019-04-14 23:19:35 +0200
committerLudovic Courtès <ludo@gnu.org>2019-04-14 23:40:52 +0200
commit7cd38788e6180eea64974c87a899d8e9613e3e9a (patch)
treea13e674037e684b97a06958238599002d990cea4 /gnu/installer
parent6a0287a1559a28ecb37fb59f9163cb068e0e5eea (diff)
downloadguix-7cd38788e6180eea64974c87a899d8e9613e3e9a.tar.gz
installer: Force user to choose exactly one network management method.
Previously, for non-desktop configs, users could choose any number of
network management methods, including 0, 2, or more.

* gnu/installer/newt/services.scm (run-networking-cbt-page): Remove
'network-management?' parameter and select only 'networking services.
(run-network-management-page): New procedure.
(run-services-page): Call it when DESKTOP is the empty list.
Diffstat (limited to 'gnu/installer')
-rw-r--r--gnu/installer/newt/services.scm41
1 files changed, 31 insertions, 10 deletions
diff --git a/gnu/installer/newt/services.scm b/gnu/installer/newt/services.scm
index 10c19115ca..2a221790d2 100644
--- a/gnu/installer/newt/services.scm
+++ b/gnu/installer/newt/services.scm
@@ -45,19 +45,14 @@ choose the one to use on the log-in screen.")
       (condition
        (&installer-step-abort))))))
 
-(define (run-networking-cbt-page network-management?)
-  "Run a page allowing the user to select networking services.  When
-NETWORK-MANAGEMENT? is true, include network management services like
-NetworkManager."
+(define (run-networking-cbt-page)
+  "Run a page allowing the user to select networking services."
   (run-checkbox-tree-page
    #:info-text (G_ "You can now select networking services to run on your \
 system.")
    #:title (G_ "Network service")
-   #:items (filter (let ((types (if network-management?
-                                    '(network-management networking)
-                                    '(networking))))
-                     (lambda (service)
-                       (memq (system-service-type service) types)))
+   #:items (filter (lambda (service)
+                     (eq? 'networking (system-service-type service)))
                    %system-services)
    #:item->text (compose G_ system-service-name)
    #:checkbox-tree-height 5
@@ -67,9 +62,35 @@ system.")
       (condition
        (&installer-step-abort))))))
 
+(define (run-network-management-page)
+  "Run a page to select among several network management methods."
+  (let ((title (G_ "Network management")))
+    (run-listbox-selection-page
+     #:title title
+     #:info-text (G_ "Choose the method to manage network connections.
+
+We recommend NetworkManager or Connman for a WiFi-capable laptop; the DHCP \
+client may be enough for a server.")
+     #:info-textbox-width 70
+     #:listbox-items (filter (lambda (service)
+                               (eq? 'network-management
+                                    (system-service-type service)))
+                             %system-services)
+     #:listbox-item->text (compose G_ system-service-name)
+     #:sort-listbox-items? #f
+     #:button-text (G_ "Exit")
+     #:button-callback-procedure
+     (lambda _
+       (raise
+        (condition
+         (&installer-step-abort)))))))
+
 (define (run-services-page)
   (let ((desktop (run-desktop-environments-cbt-page)))
     ;; When the user did not select any desktop services, and thus didn't get
     ;; '%desktop-services', offer network management services.
     (append desktop
-            (run-networking-cbt-page (null? desktop)))))
+            (run-networking-cbt-page)
+            (if (null? desktop)
+                (list (run-network-management-page))
+                '()))))