~ruther/guix-local

bf8b09169c7d8ae48ac99ea237b82867ee9aa885 — Danny Milosavljevic 8 years ago 370ae08
system: grub: Use boot-parameters instead of menu-entry where possible.

* gnu/system/grub.scm (boot-parameters->menu-entry): New variable.
(grub-configuration-file): Use boot-parameters
instead of menu-entry where possible.
* guix/scripts/system.scm (profile-boot-parameters): Update docstring.
(reinstall-grub): Use profile-boot-parameters.
(perform-action): Use profile-boot-parameters.
2 files changed, 16 insertions(+), 6 deletions(-)

M gnu/system/grub.scm
M guix/scripts/system.scm
M gnu/system/grub.scm => gnu/system/grub.scm +12 -2
@@ 267,6 267,16 @@ code."
        (#f
         #~(format #f "search --file --set ~a" #$file)))))

(define (boot-parameters->menu-entry conf)
  "Convert a <boot-parameters> instance to a corresponding <menu-entry>."
  (menu-entry
   (label (boot-parameters-label conf))
   (device (boot-parameters-store-device conf))
   (device-mount-point (boot-parameters-store-mount-point conf))
   (linux (boot-parameters-kernel conf))
   (linux-arguments (boot-parameters-kernel-arguments conf))
   (initrd (boot-parameters-initrd conf))))

(define* (grub-configuration-file config entries
                                  #:key
                                  (system (%current-system))


@@ 276,7 286,7 @@ code."
<file-system> object.  OLD-ENTRIES is taken to be a list of menu entries
corresponding to old generations of the system."
  (define all-entries
    (append entries
    (append (map boot-parameters->menu-entry entries)
            (grub-configuration-menu-entries config)))

  (define entry->gexp


@@ 323,7 333,7 @@ set timeout=~a~%"
            #$@(if (pair? old-entries)
                   #~((format port "
submenu \"GNU system, old configurations...\" {~%")
                      #$@(map entry->gexp old-entries)
                      #$@(map entry->gexp (map boot-parameters->menu-entry old-entries))
                      (format port "}~%"))
                   #~()))))


M guix/scripts/system.scm => guix/scripts/system.scm +4 -4
@@ 365,7 365,7 @@ it atomically, and then run OS's activation script."

(define* (profile-boot-parameters #:optional (profile %system-profile)
                                  (numbers (generation-numbers profile)))
  "Return a list of 'menu-entry' for the generations of PROFILE specified by
  "Return a list of 'boot-parameters' for the generations of PROFILE specified by
NUMBERS, which is a list of generation numbers."
  (define (system->boot-parameters system number time)
    (unless-file-not-found


@@ 456,9 456,9 @@ open connection to the store."
         ;; from the actual past values for this generation's entry.
         (grub-config (grub-configuration (device root-device)))
         ;; Make the specified system generation the default entry.
         (entries (profile-grub-entries %system-profile (list number)))
         (entries (profile-boot-parameters %system-profile (list number)))
         (old-generations (delv number (generation-numbers %system-profile)))
         (old-entries (profile-grub-entries %system-profile old-generations))
         (old-entries (profile-boot-parameters %system-profile old-generations))
         (grub.cfg (run-with-store store
                     (grub-configuration-file grub-config
                                              entries


@@ 642,7 642,7 @@ output when building a system derivation, such as a disk image."
                      (operating-system-bootcfg os
                                                (if (eq? 'init action)
                                                    '()
                                                    (profile-grub-entries)))))
                                                    (profile-boot-parameters)))))

       ;; For 'init' and 'reconfigure', always build GRUB.CFG, even if
       ;; --no-grub is passed, because GRUB.CFG because we then use it as a GC