[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#26339: [PATCH v2 01/12] system: Pass <bootloader-parameter> to grub.
From: |
Danny Milosavljevic |
Subject: |
bug#26339: [PATCH v2 01/12] system: Pass <bootloader-parameter> to grub. |
Date: |
Mon, 8 May 2017 22:04:15 +0200 |
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.
bug#26339: [PATCH v2 01/12] system: Pass <bootloader-parameter> to grub., Mathieu Othacehe, 2017/05/08