public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Andrew Ammerlaan <andrewammerlaan@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: [gentoo-dev] [PATCH 1/2] dist-kernel-utils.eclass: fix extension of generated efi file
Date: Sat, 17 Jun 2023 20:17:30 +0200	[thread overview]
Message-ID: <a14a1d48-69b2-8fe9-3d55-7954bdfd6511@gentoo.org> (raw)

This fixes a bug. The 90-uki-copy.install kernel-install plugin insists 
uki's have the .efi extension. You only hit this if you set layout=uki 
in /etc/kernel/install.conf.

layout=uki is probably what you want if you set uefi=yes in dracut.conf 
because in this layout the uki's end up on the ESP. This is required to 
load the uki directly from efi firmware.


 From 1c406ada60d3493203c1fbd6333caf74e53ee8ac Mon Sep 17 00:00:00 2001
From: Andrew Ammerlaan <andrewammerlaan@gentoo.org>
Date: Fri, 16 Jun 2023 16:33:56 +0200
Subject: [PATCH] dist-kernel-utils.eclass: fix extension of generated 
efi file

If kernel-install is configured to use the uki layout the extension of 
the efi
file that we install has to be .efi otherwise kernel-install returns an 
error.

Signed-off-by: Andrew Ammerlaan <andrewammerlaan@gentoo.org>
---
  eclass/dist-kernel-utils.eclass | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/eclass/dist-kernel-utils.eclass 
b/eclass/dist-kernel-utils.eclass
index 439bdc87695df..c6892c2f01278 100644
--- a/eclass/dist-kernel-utils.eclass
+++ b/eclass/dist-kernel-utils.eclass
@@ -1,4 +1,4 @@
-# Copyright 2020-2022 Gentoo Authors
+# Copyright 2020-2023 Gentoo Authors
  # Distributed under the terms of the GNU General Public License v2

  # @ECLASS: dist-kernel-utils.eclass
@@ -104,7 +104,7 @@ dist-kernel_install_kernel() {
  	if [[ ${magic} == MZ ]]; then
  		einfo "Combined UEFI kernel+initramfs executable found"
  		# install the combined executable in place of kernel
-		image=${initrd}.uefi
+		image=${initrd}.efi
  		mv "${initrd}" "${image}" || die
  		# put an empty file in place of initrd.  installing a duplicate
  		# file would waste disk space, and removing it entirely provokes


             reply	other threads:[~2023-06-17 18:17 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-17 18:17 Andrew Ammerlaan [this message]
2023-06-17 18:21 ` [gentoo-dev] [PATCH 2/2] dist-kernel-utils.eclass: skip initrd installation when using the uki layout Andrew Ammerlaan
2023-06-19 12:33   ` [gentoo-dev] [PATCH 2/2 v2] " Andrew Ammerlaan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=a14a1d48-69b2-8fe9-3d55-7954bdfd6511@gentoo.org \
    --to=andrewammerlaan@gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox