guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [RFCv4] install: Create a GC root during install-grub.


From: Ludovic Courtès
Subject: Re: [RFCv4] install: Create a GC root during install-grub.
Date: Sun, 13 Mar 2016 22:44:38 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Jookia <address@hidden> skribis:

> On Fri, Mar 11, 2016 at 03:48:38PM +0100, Ludovic Courtès wrote:
>> Jookia <address@hidden> skribis:
>> 
>> > While previously creating a GC root for GRUB's resources was the caller's
>> > responsibility, it's much less repetitive to put it in install-grub now 
>> > that
>> > it's wrapped by error handling. This also means we can replace the 
>> > install-grub*
>> > function with a small definition inside perform-action named 
>> > 'install-boot'.
>> >
>> > * gnu/build/install.scm (install-grub): Make a GC root for grub.cfg on 
>> > success.
>> >   (register-grub.cfg-root): Remove function, install-grub does this now.
>> > * gnu/system/vm.scm (qemu-image): Don't explicitly make a GC root.
>> > * guix/scripts/system.scm (install-grub*): Move useful parts to 
>> > perform-action.
>> >   (perform-action): Use inline definition install-boot to install GRUB.
>> 
>> Is it different from v1 to v3?
>> 
>> I’m asking because I feel it might be difficult for me to find out what
>> the right version is when I start reviewing these.  :-)
>> 
>> Ludo’.
>
> Yes, apologies since I'm not exactly sure how to add text that would be taken
> out in final patches. v2 fixes an issue with reconfigure breaking since it
> assumed there was always a target (not true), I can't remember what v3 fixed, 
> v4
> fixed v2's fix since I used the wrong variable name. 

OK.

> Perhaps I should write diffs to patches assuming all the patches are
> in a single directory?

Diffs to patches would be hard to read, but if you could simply add a
cover letter with a short summary of the changes compared to previous
versions, that’d be perfect.  :-)

Ludo’.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]