[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#37868] [PATCH v2 2/2] system: Add kernel-module-packages to operati

From: Mathieu Othacehe
Subject: [bug#37868] [PATCH v2 2/2] system: Add kernel-module-packages to operating-system.
Date: Tue, 18 Feb 2020 13:31:49 +0100
User-agent: mu4e 1.2.0; emacs 26.3

Hello Danny,

Thanks for this patch! A few remarks below.

> +                               ; TODO: system, target.
> +                               #:system #f
> +                               #:target #f)))

We need to figure out what #:system and #:target to pass, otherwise it
will break system compilation with --system and --target. This is
somehow linked to this thread[1].

> +(define (linux-module-database manifest)

This is a rather long and over 80 columns procedure.  Maybe you should
consider split it into several functions.

> +                                (display "FAILED\n" (current-error-port))

This could be more specific and would need to be translated.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]