public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
* [PATCH] fix broken fsck type in initramfs-fsck-hook-ext4
@ 2022-12-22 15:35 Schaffner, Tobias
  2022-12-22 16:01 ` Henning Schild
  0 siblings, 1 reply; 2+ messages in thread
From: Schaffner, Tobias @ 2022-12-22 15:35 UTC (permalink / raw)
  To: isar-users; +Cc: Gylstorff, Quirin

[-- Attachment #1: Type: text/plain, Size: 251 bytes --]

The initramfs-fsck-hook-ext4 script did not copy the fsck binary as the
wrong type variable was used to identify the needed filesystem type.

Set the type to ext4 as the script is ext4 only anyways.

(Had to exceptionally send this as attachment)

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-fix-broken-fsck-type-in-initramfs-fsck-hook-ext4.patch --]
[-- Type: text/x-patch; name="0001-fix-broken-fsck-type-in-initramfs-fsck-hook-ext4.patch", Size: 1312 bytes --]

From a6cdc01e5f53a08d3421a6567657b17c7e523800 Mon Sep 17 00:00:00 2001
From: Tobias Schaffner <tobias.schaffner@siemens.com>
Date: Thu, 22 Dec 2022 15:05:31 +0100
Subject: [PATCH] fix broken fsck type in initramfs-fsck-hook-ext4

The initramfs-fsck-hook-ext4 script did not copy the fsck binary as the
wrong type variable was used to identify the needed filesystem type.

Set the type to ext4 as the script is ext4 only anyways.

Signed-off-by: Tobias Schaffner <tobias.schaffner@siemens.com>
---
 .../initramfs-fsck-hook/files/initramfs.fsck.ext4.hook        | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta/recipes-support/initramfs-fsck-hook/files/initramfs.fsck.ext4.hook b/meta/recipes-support/initramfs-fsck-hook/files/initramfs.fsck.ext4.hook
index 23f7c28..e07db48 100644
--- a/meta/recipes-support/initramfs-fsck-hook/files/initramfs.fsck.ext4.hook
+++ b/meta/recipes-support/initramfs-fsck-hook/files/initramfs.fsck.ext4.hook
@@ -26,12 +26,10 @@ if [ ! -x /sbin/fsck ]; then
 	exit 0
 fi
 
-fsck_types="ext4"
-
 copy_exec /sbin/fsck
 copy_exec /sbin/logsave
 
-if prog="$(command -v fsck.${type})"; then
+if prog="$(command -v fsck.ext4)"; then
   copy_exec "$prog"
 else
   echo "Warning: /sbin/fsck.${type} doesn't exist, can't install to initramfs, ignoring."
-- 
2.34.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] fix broken fsck type in initramfs-fsck-hook-ext4
  2022-12-22 15:35 [PATCH] fix broken fsck type in initramfs-fsck-hook-ext4 Schaffner, Tobias
@ 2022-12-22 16:01 ` Henning Schild
  0 siblings, 0 replies; 2+ messages in thread
From: Henning Schild @ 2022-12-22 16:01 UTC (permalink / raw)
  To: Schaffner, Tobias; +Cc: isar-users, Gylstorff, Quirin

Am Thu, 22 Dec 2022 15:35:45 +0000
schrieb "Schaffner, Tobias" <tobias.schaffner@siemens.com>:

> The initramfs-fsck-hook-ext4 script did not copy the fsck binary as
> the wrong type variable was used to identify the needed filesystem
> type.
> 
> Set the type to ext4 as the script is ext4 only anyways.

that "type" was also used in the later warning

> (Had to exceptionally send this as attachment)

So you get your chance to send a v2 inline ;)

Henning 


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-12-22 16:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-22 15:35 [PATCH] fix broken fsck type in initramfs-fsck-hook-ext4 Schaffner, Tobias
2022-12-22 16:01 ` Henning Schild

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox