[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#29100] [PATCH 1/3] build: Use overlayfs instead of unionfs.
From: |
Ludovic Courtès |
Subject: |
[bug#29100] [PATCH 1/3] build: Use overlayfs instead of unionfs. |
Date: |
Sun, 05 Nov 2017 22:08:49 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Heya,
Hartmut Goebel <address@hidden> skribis:
> Overlayfs is part of the kernel, while unionfs needs FUSE. This also reduces
> the size of the initrd by ca. 4.3% (487K).
Awesome! It’s definitely an improvement, and I’m happy that it fixes
<http://bugs.gnu.org/23056>.
> * gnu/build/linux-boot.scm (mount-root-file-system): Remove optional parameter
> "unionfs"; mount using overlayfs instead of unionfs; new directory layout
> requied by overlayfs; update documentation;
> [mark-as-not-killable]: remove now unused function
>
> * gnu/system/linux-intrd.scm (file-system-packages): Remove now unused
^^^
Typo.
> packages "unionfs-fuse/static".
> (linux-modules): Replace "fuse" by "overlay".
Nitpick: please capitalize sentences, add missing periods.
> + ;; Make /root a overlay of the tmpfs and the actual root.
s/a/an/
> --- a/gnu/system/linux-initrd.scm
> +++ b/gnu/system/linux-initrd.scm
> @@ -236,7 +236,7 @@ FILE-SYSTEMS."
> (list btrfs-progs/static)
> '())
> ,@(if volatile-root?
> - (list unionfs-fuse/static)
> + (list ) ;; nothing requierd in this case
> '())))
You can remove the whole ‘if’ here.
OK with these changes, thank you!
Ludo’.
[bug#29100] [PATCH 1/3] build: Use overlayfs instead of unionfs.,
Ludovic Courtès <=
bug#29100: Updated, tested again and merged, Hartmut Goebel, 2017/11/08