Subject: [bug#41541] [PATCH 3/8] system: Add 'multiboot-modules' field to <boot-parameters>.

From: Ludovic Courtès <[email-masked]>
Date: Sat, 06 Jun 2020 12:55:28 +0200

Hi,

Mathieu Othacehe <[email-masked]> skribis:

>> +             (mach (if (%current-target-system)
>> +                       ;; A cross-built GNUmach does not work
>> +                       (with-parameters ((%current-system "i686-linux")
>> +                                         (%current-target-system #f))
>> +                         mach)
>
> Maybe it would make more sense to check if we are on a Hurd system,
> rather than just checking for %current-target-system to be set, don't
> know.

If this hack is still needed, can we move it away to hurd-os.tmpl, or to
the ‘gnumach’ package itself or something?

I think it’s best to keep (gnu system) free of any such assumptions.

>> +         (libc (if target
>> +                   (with-parameters ((%current-target-system #f))
>> +                     (cross-libc target))
>> +                   glibc))
>
> In the future, it would be nice to make this think transparent. Anyway,
> for now this is the best we can do I think.

Yeah, perhaps add a FIXME.

>> "--x-xattr-translator-records"

Do we need this yet?  If it can wait for later, that’s even nicer IMO as
it would allow us to separate concerns.

Ludo’.