[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#57861] [PATCH] gnu: Add ultrastar-deluxe.
From: |
Ludovic Courtès |
Subject: |
[bug#57861] [PATCH] gnu: Add ultrastar-deluxe. |
Date: |
Mon, 17 Oct 2022 18:44:21 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux) |
Maxime Devos <maximedevos@telenet.be> skribis:
> On 17-10-2022 10:33, Ludovic Courtès wrote:
>> Hello!
>> Lars-Dominik Braun <lars@6xq.net> skribis:
>>
>>> +;; Lazily resolve to avoid a circular dependency.
>>> +(define fpc*
>>> + (delay (module-ref (resolve-interface '(gnu packages pascal))
>>> + 'fpc)))
>> I think this is unnecessary: you can just #:use-module (gnu packages
>> pascal) and have ‘fpc’ in ‘native-inputs’. Everything will be fine
>> because ‘inputs’ is a thunked field (IOW, its evaluation is delayed).
>
> That's still a circular dependency. Why import (gnu packages pascal)
> when you don't need it
That’s what every package module does.
We can discuss the drawbacks of this and ways to avoid it, but this
particular patch and review is not the right place to do that IMO.
Thanks,
Ludo’.