Default 'Is the disk partition alrady mounted?' to 'no' on install
authorkn <kn@openbsd.org>
Fri, 7 Apr 2023 13:48:42 +0000 (13:48 +0000)
committerkn <kn@openbsd.org>
Fri, 7 Apr 2023 13:48:42 +0000 (13:48 +0000)
For installation, no partitions whatsoever are mounted, so 'yes' is always
wrong unless the user manually mounted stuff before that question.

No change for upgrade, where partitions on the root disk are mounted.

Idea from naddy, diff from Mikhail (mp39590 AT gmail)
Tweaks from me
OK op afresh1

distrib/miniroot/install.sub

index a365bd5..af1457a 100644 (file)
@@ -1,5 +1,5 @@
 #!/bin/ksh
-#      $OpenBSD: install.sub,v 1.1240 2023/04/07 09:40:18 kn Exp $
+#      $OpenBSD: install.sub,v 1.1241 2023/04/07 13:48:42 kn Exp $
 #
 # Copyright (c) 1997-2015 Todd Miller, Theo de Raadt, Ken Westerback
 # Copyright (c) 2015, Robert Peichaer <rpe@openbsd.org>
@@ -2048,7 +2048,12 @@ install_cdrom() {
 # Install sets from disk.
 # Ask for the disk device containing the set files.
 install_disk() {
-       if ! ask_yn "Is the disk partition already mounted?" yes; then
+       local _ismounted=yes
+
+       # No partitions are mounted prior to regular installation.
+       [[ $MODE == install ]] && _ismounted=no
+
+       if ! ask_yn "Is the disk partition already mounted?" $_ismounted; then
                ask_which "disk" "contains the $MODE media" \
                        '$(bsort $(get_dkdevs))' \
                        '$(get_dkdevs_uninitialized)'