From patchwork Mon Dec 30 10:33:09 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Janneke Nieuwenhuizen X-Patchwork-Id: 19474 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id 95B7A1797C; Mon, 30 Dec 2019 10:34:58 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id 441F917960 for ; Mon, 30 Dec 2019 10:34:58 +0000 (GMT) Received: from localhost ([::1]:60232 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ilsO1-0000Fv-MX for patchwork@mira.cbaines.net; Mon, 30 Dec 2019 05:34:57 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:58997) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ilsNA-0007tF-0B for guix-patches@gnu.org; Mon, 30 Dec 2019 05:34:05 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ilsN8-0002OP-Jm for guix-patches@gnu.org; Mon, 30 Dec 2019 05:34:03 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:54538) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ilsN8-0002Ni-G4 for guix-patches@gnu.org; Mon, 30 Dec 2019 05:34:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1ilsN8-00009n-D5 for guix-patches@gnu.org; Mon, 30 Dec 2019 05:34:02 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#38809] [PATCH] bootloader: grub: Add resolution override. Resent-From: Jan Nieuwenhuizen Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Mon, 30 Dec 2019 10:34:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 38809 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 38809@debbugs.gnu.org X-Debbugs-Original-To: guix-patches@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.1577702001555 (code B ref -1); Mon, 30 Dec 2019 10:34:02 +0000 Received: (at submit) by debbugs.gnu.org; 30 Dec 2019 10:33:21 +0000 Received: from localhost ([127.0.0.1]:60511 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ilsMO-00008o-8L for submit@debbugs.gnu.org; Mon, 30 Dec 2019 05:33:21 -0500 Received: from lists.gnu.org ([209.51.188.17]:48857) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ilsMM-00008e-6M for submit@debbugs.gnu.org; Mon, 30 Dec 2019 05:33:15 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:58866) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ilsMK-0007Vc-Hr for guix-patches@gnu.org; Mon, 30 Dec 2019 05:33:14 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:33030) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ilsMK-000239-Cy; Mon, 30 Dec 2019 05:33:12 -0500 Received: from [2001:980:1b4f:1:42d2:832d:bb59:862] (port=58498 helo=dundal.peder.onsbrabantnet.nl) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1ilsMJ-0007p9-SI; Mon, 30 Dec 2019 05:33:12 -0500 From: Jan Nieuwenhuizen Date: Mon, 30 Dec 2019 11:33:09 +0100 Message-ID: <87lfquhzmi.fsf@gnu.org> MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches Hi, With this patch I finally have a readable GRUB menu. I am not sure about the name and semantics of `resolution': Better to use `gfxmode', and/or use a list of strings '("1024x768x32" "auto"), ... WDYT? Greetings, janneke From dfceb5702b0de5787f79e1098b4787e9d3adbe4e Mon Sep 17 00:00:00 2001 From: Jan Nieuwenhuizen Date: Mon, 30 Dec 2019 11:25:40 +0100 Subject: [PATCH] bootloader: grub: Add resolution override. * gnu/bootloader/grub.scm (): Add `resolution' entry. (eye-candy): Use it. * doc/guix.texi (Bootloader Configuration): Document it. --- doc/guix.texi | 26 ++++++++++++++++++++++++-- gnu/bootloader/grub.scm | 11 +++++++---- 2 files changed, 31 insertions(+), 6 deletions(-) diff --git a/doc/guix.texi b/doc/guix.texi index efc59c1aaf..02cecae043 100644 --- a/doc/guix.texi +++ b/doc/guix.texi @@ -25923,9 +25923,20 @@ must @emph{not} be an OS device name such as @file{/dev/sda1}. @end table @end deftp +@cindex HDPI +@cindex resolution @c FIXME: Write documentation once it's stable. -For now only GRUB has theme support. GRUB themes are created using -the @code{grub-theme} form, which is not documented yet. +For now only GRUB has theme support. GRUB themes are created using +the @code{grub-theme} form, which is not fully documented yet. + +@deftp {Data Type} grub-theme +Data type representing the configuration of the GRUB theme. + +@table @asis +@item @code{resolution} (default: @code{"auto"}) +The @code{gfxmode} to set (@pxref{gfxmode,,, grub, GNU GRUB manual}). +@end table +@end deftp @defvr {Scheme Variable} %default-theme This is the default GRUB theme used by the operating system if no @@ -25936,6 +25947,17 @@ It comes with a fancy background image displaying the GNU and Guix logos. @end defvr +For example, to override the default resolution, you may use something +like + +@lisp +(bootloader + (grub-configuration + ;; @dots{} + (theme (grub-theme + (inherit %default-theme) + (resolution "1024x786x32;auto"))))) +@end lisp @node Invoking guix system @section Invoking @code{guix system} diff --git a/gnu/bootloader/grub.scm b/gnu/bootloader/grub.scm index fd62f627a0..70de3edeb5 100644 --- a/gnu/bootloader/grub.scm +++ b/gnu/bootloader/grub.scm @@ -3,6 +3,7 @@ ;;; Copyright © 2016 Chris Marusich ;;; Copyright © 2017 Leo Famulari ;;; Copyright © 2017 Mathieu Othacehe +;;; Copyright © 2019 Jan (janneke) Nieuwenhuizen ;;; ;;; This file is part of GNU Guix. ;;; @@ -88,7 +89,9 @@ denoting a file name." (color-normal grub-theme-color-normal (default '((fg . cyan) (bg . blue)))) (color-highlight grub-theme-color-highlight - (default '((fg . white) (bg . blue))))) + (default '((fg . white) (bg . blue)))) + (resolution grub-resolution + (default "auto"))) (define %background-image (grub-image @@ -149,8 +152,8 @@ system string---e.g., \"x86_64-linux\"." ;; most other modern architectures have no other mode and therefore don't ;; need to be switched. (if (string-match "^(x86_64|i[3-6]86)-" system) - " - # Leave 'gfxmode' to 'auto'. + (string-append " + set gfxmode=" (grub-resolution (bootloader-configuration-theme config)) " insmod video_bochs insmod video_cirrus insmod gfxterm @@ -166,7 +169,7 @@ system string---e.g., \"x86_64-linux\"." insmod vbe insmod vga fi -" +") "")) (define (setup-gfxterm config font-file) -- 2.24.0