GNU bug report logs - #73927
[PATCH 00/16] Installer support for (cross) installing the Hurd.

Previous Next

Full log

Message #56 received at 73927 <at> debbugs.gnu.org (full text, mbox):

From: Mathieu Othacehe <othacehe <at> gnu.org>
To: Janneke Nieuwenhuizen <janneke <at> gnu.org>
Cc: 73927 <at> debbugs.gnu.org, Josselin Poiret <dev <at> jpoiret.xyz>,
 Ludovic Courtès <ludo <at> gnu.org>
Subject: Re: [bug#73927] [PATCH 14/16] installer: Add "Kernel" page to
 select the Hurd.
Date: Mon, 21 Oct 2024 20:14:24 +0200

Hello Janneke,

Really great to see that extension to the installer :)

The installer patches look OK to me, I will try to test them
on real hardware soon.

> Change-Id: I15d535a7a8a917e5f3492f8c01d922d652c32ee5
>
> geert none
>
> Change-Id: Ib6c5665638018f59a2690f603fad0702e042fb8b
>
> Change-Id: I01b854390240be60ce9fef8c9510a90bc6843ef3
>
> geert
>
> Change-Id: Ibb7205443969fc92d4fe62d4dfb4f956d03229b9

^
Maybe something to cleanup?

> +         ;; Ask the user to select the kernel for the system,
> +         ;; for x86 systems only.
> +         (installer-step
> +          (id 'kernel)
> +          (description (G_ "Kernel"))
> +          (compute (lambda _
> +                     (if (target-x86?)
> +                         ((installer-kernel-page current-installer))
> +                         '())))
> +          (configuration-formatter (lambda (result)
> +                                     (kernel->configuration result #$dry-run?))))
> +

If I remember correctly, new installer steps require some adaptations in
the (gnu installer tests) module.

Is make check-system TESTS="gui-installed-os" working correctly?

Thanks,

Mathieu

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.