Volatile? argument seems to be ignored for system disk images

  • Done
  • quality assurance status badge
Details
One participant
  • Maxim Cournoyer
Owner
unassigned
Submitted by
Maxim Cournoyer
Severity
normal

Debbugs page

M
M
Maxim Cournoyer wrote on 17 Nov 2019 21:08
(name . Jelle Licht)(address . jlicht@fsfe.org)
87v9rhu5z7.fsf@gmail.com
Jelle Licht <jlicht@fsfe.org> writes:

Toggle quote (20 lines)
> I am trying to dive into some of the code that goes into generating
> system disk images for guix, and I ran into the following:
>
> It seems that commit 932e1f92404d917a91af28737d6631ec45b56875 did more
> than what the commit log said:
>
>> vm: Make sure disk image initrds load 'usb-storage.ko'.
>>
>> Reported by David Thompson <dthompson2@worcester.edu>.
>>
>> * gnu/system/vm.scm (system-disk-image): Add usb-storage.ko to the
>> initrd.
>
> This commit effectively made the `#:volatile?' kwarg to
> `system-disk-image' redundant. Should we remove `#:volatile?', or should
> it be passed along instead of #t?
>
> Or am I misunderstanding something subtle here, and is everything as it
> should be?

No, my understanding is the same as yours: it looks like a bug.

Fixed with commit 67ed544212.

Thank you!

Maxim
M
M
Maxim Cournoyer wrote on 27 Nov 2019 21:20
control message for bug #38253
(address . control@debbugs.gnu.org)
878so0tw3y.fsf@gmail.com
tags 38253 fixed
close 38253
quit
?
Your comment

This issue is archived.

To comment on this conversation send an email to 38253@patchwise.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 38253
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch