bug#26339: [PATCH v2 01/12] system: Pass <bootloader-parameter> to grub.

[Danny Milosavljevic]

Hi Ludo,

On Mon, 08 May 2017 21:47:29 +0200
address@hidden (Ludovic Courtès) wrote:

> This does not remove the circular dependency, it just expresses it
> differently, so I don’t think it will help.  Using
> 
>   (module-ref (resolve-interface '(gnu system grub))
>               'grub-configuration-file)
> 
> would have achieved the effect you wanted.

Hmm.... I've tested the @@-version locally by removing all the .go files and 
then doing guix system reconfigure ... which took ages every time.  And it 
worked just fine, no errors, no warnings, nothing.  Did I test it wrong?

> However, that’s pretty much a band-aid, so we should look for a way to
> improve this.

Yes, but as soon as the other bootloader patches are merged the band-aid would 
be gone.

We have multiple choices here, 
(1) revert all ~12 commits, or
(2) do the band-aid, leave the commits in and review and merge the remainder 
later, or
(3) review and merge all the other bootloader patches now...

I don't have a strong preference.  If I knew how flaky even simple changes in 
these modules are I wouldn't have been in favor of refactoring them at all... 
(really, just having u-boot with Grub as payload - and grub-efi as part of 
regular grub - would have been much simpler than this, in retrospect)

Note that there are now two other patch series by two different people which 
depend on these.