diff options
| author | Tobias Geerinckx-Rice <me@tobias.gr> | 2017-09-03 01:19:38 +0200 |
|---|---|---|
| committer | Tobias Geerinckx-Rice <me@tobias.gr> | 2017-12-06 15:02:24 +0100 |
| commit | fdfdecdb7ebfd05b4ae66cc5140078dbfc10fc35 (patch) | |
| tree | a58dc6e21d8bf7458efb0c48a90947f61eab844a | |
| parent | 6c441efc5a554bdb6a0139d552dbb38fecbd93ba (diff) | |
gnu, doc, tests: Use ‘bootloader-configuration’ everywhere.
* doc/guix.texi (Proceeding with the Installation): Replace the old-style
‘grub-configuration’ with the newer ‘bootloader-configuration’ syntax.
* gnu/system/examples/vm-image.tmpl: Likewise.
* gnu/system/install.scm (installation-os): Likewise.
* gnu/tests.scm (%simple-os): Likewise.
* gnu/tests/install.scm (%minimal-os, %minimal-os-on-vda, %separate-home-os)
(%separate-store-os, %raid-root-os, %encrypted-root-os, %btrfs-root-os):
Likewise.
* gnu/tests/nfs.scm (%base-os): Likewise.
* tests/guix-system.scm (OS_BASE, make_user_config): Likewise.
* tests/system.scm (%os, %os-with-mapped-device): Likewise.
| -rw-r--r-- | doc/guix.texi | 4 | ||||
| -rw-r--r-- | gnu/system/examples/vm-image.tmpl | 6 | ||||
| -rw-r--r-- | gnu/system/install.scm | 5 | ||||
| -rw-r--r-- | gnu/tests.scm | 6 | ||||
| -rw-r--r-- | gnu/tests/install.scm | 29 | ||||
| -rw-r--r-- | gnu/tests/nfs.scm | 6 | ||||
| -rw-r--r-- | tests/guix-system.sh | 9 | ||||
| -rw-r--r-- | tests/system.scm | 9 |
8 files changed, 56 insertions, 18 deletions
diff --git a/doc/guix.texi b/doc/guix.texi index 2267fadd1db..3c33a19decd 100644 --- a/doc/guix.texi +++ b/doc/guix.texi | |||
| @@ -8241,8 +8241,8 @@ in particular: | |||
| 8241 | 8241 | ||
| 8242 | @itemize | 8242 | @itemize |
| 8243 | @item | 8243 | @item |
| 8244 | Make sure the @code{grub-configuration} form refers to the target you | 8244 | Make sure the @code{bootloader-configuration} form refers to the target |
| 8245 | want to install GRUB on. It should mention @code{grub-bootloader} if | 8245 | you want to install GRUB on. It should mention @code{grub-bootloader} if |
| 8246 | you are installing GRUB in the legacy way, or @code{grub-efi-bootloader} | 8246 | you are installing GRUB in the legacy way, or @code{grub-efi-bootloader} |
| 8247 | for newer UEFI systems. For legacy systems, the @code{target} field | 8247 | for newer UEFI systems. For legacy systems, the @code{target} field |
| 8248 | names a device, like @code{/dev/sda}; for UEFI systems it names a path | 8248 | names a device, like @code{/dev/sda}; for UEFI systems it names a path |
diff --git a/gnu/system/examples/vm-image.tmpl b/gnu/system/examples/vm-image.tmpl index 056b439c5fc..ce3653c8b4a 100644 --- a/gnu/system/examples/vm-image.tmpl +++ b/gnu/system/examples/vm-image.tmpl | |||
| @@ -26,8 +26,10 @@ partprobe, and then 2) resizing the filesystem with resize2fs.\n")) | |||
| 26 | 26 | ||
| 27 | ;; Assuming /dev/sdX is the target hard disk, and "my-root" is | 27 | ;; Assuming /dev/sdX is the target hard disk, and "my-root" is |
| 28 | ;; the label of the target root file system. | 28 | ;; the label of the target root file system. |
| 29 | (bootloader (grub-configuration (target "/dev/sda") | 29 | (bootloader (bootloader-configuration |
| 30 | (terminal-outputs '(console)))) | 30 | (bootloader grub-bootloader) |
| 31 | (target "/dev/sda") | ||
| 32 | (terminal-outputs '(console)))) | ||
| 31 | (file-systems (cons (file-system | 33 | (file-systems (cons (file-system |
| 32 | (device "my-root") | 34 | (device "my-root") |
| 33 | (title 'label) | 35 | (title 'label) |
diff --git a/gnu/system/install.scm b/gnu/system/install.scm index 78f2bf3a139..2e4dd05bb2e 100644 --- a/gnu/system/install.scm +++ b/gnu/system/install.scm | |||
| @@ -3,6 +3,7 @@ | |||
| 3 | ;;; Copyright © 2015 Mark H Weaver <mhw@netris.org> | 3 | ;;; Copyright © 2015 Mark H Weaver <mhw@netris.org> |
| 4 | ;;; Copyright © 2016 Andreas Enge <andreas@enge.fr> | 4 | ;;; Copyright © 2016 Andreas Enge <andreas@enge.fr> |
| 5 | ;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com> | 5 | ;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com> |
| 6 | ;;; Copyright © 2017 Tobias Geerinckx-Rice <me@tobias.gr> | ||
| 6 | ;;; | 7 | ;;; |
| 7 | ;;; This file is part of GNU Guix. | 8 | ;;; This file is part of GNU Guix. |
| 8 | ;;; | 9 | ;;; |
| @@ -301,7 +302,9 @@ You have been warned. Thanks for being so brave.\x1b[0m | |||
| 301 | (host-name "gnu") | 302 | (host-name "gnu") |
| 302 | (timezone "Europe/Paris") | 303 | (timezone "Europe/Paris") |
| 303 | (locale "en_US.utf8") | 304 | (locale "en_US.utf8") |
| 304 | (bootloader (grub-configuration (target "/dev/sda"))) | 305 | (bootloader (bootloader-configuration |
| 306 | (bootloader grub-bootloader) | ||
| 307 | (target "/dev/sda"))) | ||
| 305 | (file-systems | 308 | (file-systems |
| 306 | ;; Note: the disk image build code overrides this root file system with | 309 | ;; Note: the disk image build code overrides this root file system with |
| 307 | ;; the appropriate one. | 310 | ;; the appropriate one. |
diff --git a/gnu/tests.scm b/gnu/tests.scm index 97b9cc5107c..0caa922fdfb 100644 --- a/gnu/tests.scm +++ b/gnu/tests.scm | |||
| @@ -1,6 +1,7 @@ | |||
| 1 | ;;; GNU Guix --- Functional package management for GNU | 1 | ;;; GNU Guix --- Functional package management for GNU |
| 2 | ;;; Copyright © 2016, 2017 Ludovic Courtès <ludo@gnu.org> | 2 | ;;; Copyright © 2016, 2017 Ludovic Courtès <ludo@gnu.org> |
| 3 | ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com> | 3 | ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com> |
| 4 | ;;; Copyright © 2017 Tobias Geerinckx-Rice <me@tobias.gr> | ||
| 4 | ;;; | 5 | ;;; |
| 5 | ;;; This file is part of GNU Guix. | 6 | ;;; This file is part of GNU Guix. |
| 6 | ;;; | 7 | ;;; |
| @@ -21,6 +22,7 @@ | |||
| 21 | #:use-module (guix gexp) | 22 | #:use-module (guix gexp) |
| 22 | #:use-module (guix utils) | 23 | #:use-module (guix utils) |
| 23 | #:use-module (guix records) | 24 | #:use-module (guix records) |
| 25 | #:use-module (gnu bootloader) | ||
| 24 | #:use-module (gnu bootloader grub) | 26 | #:use-module (gnu bootloader grub) |
| 25 | #:use-module (gnu system) | 27 | #:use-module (gnu system) |
| 26 | #:use-module (gnu system file-systems) | 28 | #:use-module (gnu system file-systems) |
| @@ -206,7 +208,9 @@ the system under test." | |||
| 206 | (timezone "Europe/Berlin") | 208 | (timezone "Europe/Berlin") |
| 207 | (locale "en_US.UTF-8") | 209 | (locale "en_US.UTF-8") |
| 208 | 210 | ||
| 209 | (bootloader (grub-configuration (target "/dev/sdX"))) | 211 | (bootloader (bootloader-configuration |
| 212 | (bootloader grub-bootloader) | ||
| 213 | (target "/dev/sdX"))) | ||
| 210 | (file-systems (cons (file-system | 214 | (file-systems (cons (file-system |
| 211 | (device "my-root") | 215 | (device "my-root") |
| 212 | (title 'label) | 216 | (title 'label) |
diff --git a/gnu/tests/install.scm b/gnu/tests/install.scm index 0e97de0a878..3ac4a579da1 100644 --- a/gnu/tests/install.scm +++ b/gnu/tests/install.scm | |||
| @@ -1,5 +1,6 @@ | |||
| 1 | ;;; GNU Guix --- Functional package management for GNU | 1 | ;;; GNU Guix --- Functional package management for GNU |
| 2 | ;;; Copyright © 2016, 2017 Ludovic Courtès <ludo@gnu.org> | 2 | ;;; Copyright © 2016, 2017 Ludovic Courtès <ludo@gnu.org> |
| 3 | ;;; Copyright © 2017 Tobias Geerinckx-Rice <me@tobias.gr> | ||
| 3 | ;;; | 4 | ;;; |
| 4 | ;;; This file is part of GNU Guix. | 5 | ;;; This file is part of GNU Guix. |
| 5 | ;;; | 6 | ;;; |
| @@ -60,7 +61,9 @@ | |||
| 60 | (timezone "Europe/Paris") | 61 | (timezone "Europe/Paris") |
| 61 | (locale "en_US.UTF-8") | 62 | (locale "en_US.UTF-8") |
| 62 | 63 | ||
| 63 | (bootloader (grub-configuration (target "/dev/vdb"))) | 64 | (bootloader (bootloader-configuration |
| 65 | (bootloader grub-bootloader) | ||
| 66 | (target "/dev/vdb"))) | ||
| 64 | (kernel-arguments '("console=ttyS0")) | 67 | (kernel-arguments '("console=ttyS0")) |
| 65 | (file-systems (cons (file-system | 68 | (file-systems (cons (file-system |
| 66 | (device "my-root") | 69 | (device "my-root") |
| @@ -343,7 +346,9 @@ per %test-installed-os, this test is expensive in terms of CPU and storage.") | |||
| 343 | (timezone "Europe/Paris") | 346 | (timezone "Europe/Paris") |
| 344 | (locale "en_US.UTF-8") | 347 | (locale "en_US.UTF-8") |
| 345 | 348 | ||
| 346 | (bootloader (grub-configuration (target "/dev/vda"))) | 349 | (bootloader (bootloader-configuration |
| 350 | (bootloader grub-bootloader) | ||
| 351 | (target "/dev/vda"))) | ||
| 347 | (kernel-arguments '("console=ttyS0")) | 352 | (kernel-arguments '("console=ttyS0")) |
| 348 | (file-systems (cons (file-system | 353 | (file-systems (cons (file-system |
| 349 | (device "my-root") | 354 | (device "my-root") |
| @@ -418,7 +423,9 @@ reboot\n") | |||
| 418 | (timezone "Europe/Paris") | 423 | (timezone "Europe/Paris") |
| 419 | (locale "en_US.utf8") | 424 | (locale "en_US.utf8") |
| 420 | 425 | ||
| 421 | (bootloader (grub-configuration (target "/dev/vdb"))) | 426 | (bootloader (bootloader-configuration |
| 427 | (bootloader grub-bootloader) | ||
| 428 | (target "/dev/vdb"))) | ||
| 422 | (kernel-arguments '("console=ttyS0")) | 429 | (kernel-arguments '("console=ttyS0")) |
| 423 | (file-systems (cons* (file-system | 430 | (file-systems (cons* (file-system |
| 424 | (device "my-root") | 431 | (device "my-root") |
| @@ -476,7 +483,9 @@ partition. In particular, home directories must be correctly created (see | |||
| 476 | (timezone "Europe/Paris") | 483 | (timezone "Europe/Paris") |
| 477 | (locale "en_US.UTF-8") | 484 | (locale "en_US.UTF-8") |
| 478 | 485 | ||
| 479 | (bootloader (grub-configuration (target "/dev/vdb"))) | 486 | (bootloader (bootloader-configuration |
| 487 | (bootloader grub-bootloader) | ||
| 488 | (target "/dev/vdb"))) | ||
| 480 | (kernel-arguments '("console=ttyS0")) | 489 | (kernel-arguments '("console=ttyS0")) |
| 481 | (file-systems (cons* (file-system | 490 | (file-systems (cons* (file-system |
| 482 | (device "root-fs") | 491 | (device "root-fs") |
| @@ -552,7 +561,9 @@ where /gnu lives on a separate partition.") | |||
| 552 | (timezone "Europe/Paris") | 561 | (timezone "Europe/Paris") |
| 553 | (locale "en_US.utf8") | 562 | (locale "en_US.utf8") |
| 554 | 563 | ||
| 555 | (bootloader (grub-configuration (target "/dev/vdb"))) | 564 | (bootloader (bootloader-configuration |
| 565 | (bootloader grub-bootloader) | ||
| 566 | (target "/dev/vdb"))) | ||
| 556 | (kernel-arguments '("console=ttyS0")) | 567 | (kernel-arguments '("console=ttyS0")) |
| 557 | (initrd (lambda (file-systems . rest) | 568 | (initrd (lambda (file-systems . rest) |
| 558 | ;; Add a kernel module for RAID-0 (aka. "stripe"). | 569 | ;; Add a kernel module for RAID-0 (aka. "stripe"). |
| @@ -635,7 +646,9 @@ by 'mdadm'.") | |||
| 635 | (timezone "Europe/Paris") | 646 | (timezone "Europe/Paris") |
| 636 | (locale "en_US.UTF-8") | 647 | (locale "en_US.UTF-8") |
| 637 | 648 | ||
| 638 | (bootloader (grub-configuration (target "/dev/vdb"))) | 649 | (bootloader (bootloader-configuration |
| 650 | (bootloader grub-bootloader) | ||
| 651 | (target "/dev/vdb"))) | ||
| 639 | 652 | ||
| 640 | ;; Note: Do not pass "console=ttyS0" so we can use our passphrase prompt | 653 | ;; Note: Do not pass "console=ttyS0" so we can use our passphrase prompt |
| 641 | ;; detection logic in 'enter-luks-passphrase'. | 654 | ;; detection logic in 'enter-luks-passphrase'. |
| @@ -762,7 +775,9 @@ build (current-guix) and then store a couple of full system images.") | |||
| 762 | (timezone "Europe/Paris") | 775 | (timezone "Europe/Paris") |
| 763 | (locale "en_US.UTF-8") | 776 | (locale "en_US.UTF-8") |
| 764 | 777 | ||
| 765 | (bootloader (grub-configuration (target "/dev/vdb"))) | 778 | (bootloader (bootloader-configuration |
| 779 | (bootloader grub-bootloader) | ||
| 780 | (target "/dev/vdb"))) | ||
| 766 | (kernel-arguments '("console=ttyS0")) | 781 | (kernel-arguments '("console=ttyS0")) |
| 767 | (file-systems (cons (file-system | 782 | (file-systems (cons (file-system |
| 768 | (device "my-root") | 783 | (device "my-root") |
diff --git a/gnu/tests/nfs.scm b/gnu/tests/nfs.scm index 889f578d018..d58cf7aefd3 100644 --- a/gnu/tests/nfs.scm +++ b/gnu/tests/nfs.scm | |||
| @@ -2,6 +2,7 @@ | |||
| 2 | ;;; Copyright © 2016, 2017 Ludovic Courtès <ludo@gnu.org> | 2 | ;;; Copyright © 2016, 2017 Ludovic Courtès <ludo@gnu.org> |
| 3 | ;;; Copyright © 2016 John Darrington <jmd@gnu.org> | 3 | ;;; Copyright © 2016 John Darrington <jmd@gnu.org> |
| 4 | ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com> | 4 | ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com> |
| 5 | ;;; Copyright © 2017 Tobias Geerinckx-Rice <me@tobias.gr> | ||
| 5 | ;;; | 6 | ;;; |
| 6 | ;;; This file is part of GNU Guix. | 7 | ;;; This file is part of GNU Guix. |
| 7 | ;;; | 8 | ;;; |
| @@ -20,6 +21,7 @@ | |||
| 20 | 21 | ||
| 21 | (define-module (gnu tests nfs) | 22 | (define-module (gnu tests nfs) |
| 22 | #:use-module (gnu tests) | 23 | #:use-module (gnu tests) |
| 24 | #:use-module (gnu bootloader) | ||
| 23 | #:use-module (gnu bootloader grub) | 25 | #:use-module (gnu bootloader grub) |
| 24 | #:use-module (gnu system) | 26 | #:use-module (gnu system) |
| 25 | #:use-module (gnu system file-systems) | 27 | #:use-module (gnu system file-systems) |
| @@ -41,7 +43,9 @@ | |||
| 41 | (timezone "Europe/Berlin") | 43 | (timezone "Europe/Berlin") |
| 42 | (locale "en_US.UTF-8") | 44 | (locale "en_US.UTF-8") |
| 43 | 45 | ||
| 44 | (bootloader (grub-configuration (target "/dev/sdX"))) | 46 | (bootloader (bootloader-configuration |
| 47 | (bootloader grub-bootloader) | ||
| 48 | (target "/dev/sdX"))) | ||
| 45 | (file-systems %base-file-systems) | 49 | (file-systems %base-file-systems) |
| 46 | (users %base-user-accounts) | 50 | (users %base-user-accounts) |
| 47 | (packages (cons* | 51 | (packages (cons* |
diff --git a/tests/guix-system.sh b/tests/guix-system.sh index 213864833a8..ed8563c8aa2 100644 --- a/tests/guix-system.sh +++ b/tests/guix-system.sh | |||
| @@ -1,5 +1,6 @@ | |||
| 1 | # GNU Guix --- Functional package management for GNU | 1 | # GNU Guix --- Functional package management for GNU |
| 2 | # Copyright © 2014, 2015, 2016, 2017 Ludovic Courtès <ludo@gnu.org> | 2 | # Copyright © 2014, 2015, 2016, 2017 Ludovic Courtès <ludo@gnu.org> |
| 3 | # Copyright © 2017 Tobias Geerinckx-Rice <me@tobias.gr> | ||
| 3 | # | 4 | # |
| 4 | # This file is part of GNU Guix. | 5 | # This file is part of GNU Guix. |
| 5 | # | 6 | # |
| @@ -134,7 +135,9 @@ OS_BASE=' | |||
| 134 | (timezone "Europe/Paris") | 135 | (timezone "Europe/Paris") |
| 135 | (locale "en_US.UTF-8") | 136 | (locale "en_US.UTF-8") |
| 136 | 137 | ||
| 137 | (bootloader (grub-configuration (device "/dev/sdX"))) | 138 | (bootloader (bootloader-configuration |
| 139 | (bootloader grub-bootloader) | ||
| 140 | (device "/dev/sdX"))) | ||
| 138 | (file-systems (cons (file-system | 141 | (file-systems (cons (file-system |
| 139 | (device "root") | 142 | (device "root") |
| 140 | (title (string->symbol "label")) | 143 | (title (string->symbol "label")) |
| @@ -205,7 +208,9 @@ make_user_config () | |||
| 205 | (timezone "Europe/Paris") | 208 | (timezone "Europe/Paris") |
| 206 | (locale "en_US.UTF-8") | 209 | (locale "en_US.UTF-8") |
| 207 | 210 | ||
| 208 | (bootloader (grub-configuration (device "/dev/sdX"))) | 211 | (bootloader (bootloader-configuration |
| 212 | (bootloader grub-bootloader) | ||
| 213 | (device "/dev/sdX"))) | ||
| 209 | (file-systems (cons (file-system | 214 | (file-systems (cons (file-system |
| 210 | (device "root") | 215 | (device "root") |
| 211 | (title 'label) | 216 | (title 'label) |
diff --git a/tests/system.scm b/tests/system.scm index a661544a5fe..6a7f45c59c2 100644 --- a/tests/system.scm +++ b/tests/system.scm | |||
| @@ -1,5 +1,6 @@ | |||
| 1 | ;;; GNU Guix --- Functional package management for GNU | 1 | ;;; GNU Guix --- Functional package management for GNU |
| 2 | ;;; Copyright © 2016 Ludovic Courtès <ludo@gnu.org> | 2 | ;;; Copyright © 2016 Ludovic Courtès <ludo@gnu.org> |
| 3 | ;;; Copyright © 2017 Tobias Geerinckx-Rice <me@tobias.gr> | ||
| 3 | ;;; | 4 | ;;; |
| 4 | ;;; This file is part of GNU Guix. | 5 | ;;; This file is part of GNU Guix. |
| 5 | ;;; | 6 | ;;; |
| @@ -36,7 +37,9 @@ | |||
| 36 | (host-name "komputilo") | 37 | (host-name "komputilo") |
| 37 | (timezone "Europe/Berlin") | 38 | (timezone "Europe/Berlin") |
| 38 | (locale "en_US.utf8") | 39 | (locale "en_US.utf8") |
| 39 | (bootloader (grub-configuration (target "/dev/sdX"))) | 40 | (bootloader (bootloader-configuration |
| 41 | (bootloader grub-bootloader) | ||
| 42 | (target "/dev/sdX"))) | ||
| 40 | (file-systems (cons %root-fs %base-file-systems)) | 43 | (file-systems (cons %root-fs %base-file-systems)) |
| 41 | 44 | ||
| 42 | (users %base-user-accounts))) | 45 | (users %base-user-accounts))) |
| @@ -51,7 +54,9 @@ | |||
| 51 | (host-name "komputilo") | 54 | (host-name "komputilo") |
| 52 | (timezone "Europe/Berlin") | 55 | (timezone "Europe/Berlin") |
| 53 | (locale "en_US.utf8") | 56 | (locale "en_US.utf8") |
| 54 | (bootloader (grub-configuration (target "/dev/sdX"))) | 57 | (bootloader (bootloader-configuration |
| 58 | (bootloader grub-bootloader) | ||
| 59 | (target "/dev/sdX"))) | ||
| 55 | (mapped-devices (list %luks-device)) | 60 | (mapped-devices (list %luks-device)) |
| 56 | (file-systems (cons (file-system | 61 | (file-systems (cons (file-system |
| 57 | (inherit %root-fs) | 62 | (inherit %root-fs) |
