[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/6] file-systems: Suppress fsck status completion bar.
From: |
Marius Bakke |
Subject: |
Re: [PATCH 3/6] file-systems: Suppress fsck status completion bar. |
Date: |
Mon, 07 Nov 2016 09:29:08 +0000 |
User-agent: |
Notmuch/0.23.1 (https://notmuchmail.org) Emacs/25.1.1 (x86_64-unknown-linux-gnu) |
Ludovic Courtès <address@hidden> writes:
> Hey!
>
> Marius Bakke <address@hidden> skribis:
>
>> Ludovic Courtès <address@hidden> writes:
>>
>>> Marius Bakke <address@hidden> skribis:
>>>
>>>> * gnu/build/file-systems.scm (check-file-system): Drop "-C" argument
>>>> from fsck for compatibility with other fscks.
>>>
>>> Oh so fsck.ext2 would no longer show any kind of progress report?
>>> That’s annoying.
>>>
>>> Could we address it differently? Not sure how, though.
>>
>> We would have to provide a custom check-file-system procedure for each
>> detected file-system. That might be needed in the long run anyway, but I
>> think this is a worthwhile compromise for now.
>
> What about adding a one-argument procedure as the ‘check-procedure’
> field of <file-system>, with a sane default, like:
>
> (define (default-file-system-check file-system)
> #~(system* (string-append "fsck." #$(file-system-type file-system))
> …))
>
> ?
>
> In fact, that would also remove the need for the special case to add
> dosfstools to the initrd because we could simply write:
>
> (define (fat-file-system-check file-system)
> #~(system* #$(file-append vfatfsck/static "/bin/fsck.vfat")
> …))
>
> and that would automatically bring vfatfsck/static to the initrd when
> it’s needed, and only then.
>
> WDYT?
>
> (Same design pattern as ‘open’ in <device-mapping>.)
This looks very promising. Will try to make it work -- new patches
hopefully later this week! :)
signature.asc
Description: PGP signature
- [PATCH 1/6] gnu: Add fatfsck-static., (continued)
[PATCH 4/6] gnu: Add efivar., Marius Bakke, 2016/11/05
[PATCH 5/6] gnu: Add efibootmgr., Marius Bakke, 2016/11/05
[PATCH 6/6] gnu: Add grub-efi., Marius Bakke, 2016/11/05