[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: |
Fri, 11 Mar 2016 15:48:38 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
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’.