Re: [PATCH] Add package definition for GNU XaoS.

[Alex Kost]

Ludovic Courtès (2015-12-30 18:51 +0300) wrote:

> Alex Kost <address@hidden> skribis:
[...]
>> Originally Fabian sent a patch for separate "xaos.scm" file.  Then
>> Andreas suggested to move it to "maths.scm".  But this will lead to the
>> mentioned circularity:
>>   (gnu packages image) already uses (gnu packages maths)
>> and after this patch:
>>   (gnu packages maths) will use (gnu packages image).
>>
>> Is it OK to leave it like this, or should 'xaos' be left in a
>> separate file?
>
> It’s OK to leave it as is.
>
> Module circularity are not a problem, unless there are circular
> *top-level* references.

Great!  Though I believe it's preferable to avoid module circularities
if possible, right?

> That is, if maths.scm does, say:
>
>   (define foo libpng)
>
> and image.scm does:
>
>   (define bar lapack)
>
> then we have a problem.

Thanks for the explanation!

-- 
Alex