Re: [PATCH] Futures: Avoid creating the worker pool more than once

[Mark H Weaver]

Hi Ludovic,

Sorry for not responding to this earlier.

address@hidden (Ludovic Courtès) writes:
> Mark H Weaver <address@hidden> skribis:
>
>> diff --git a/module/ice-9/futures.scm b/module/ice-9/futures.scm
>> index 0f64b5c..7fbccf6 100644
>> --- a/module/ice-9/futures.scm
>> +++ b/module/ice-9/futures.scm
>> @@ -19,6 +19,7 @@
>>  (define-module (ice-9 futures)
>>    #:use-module (srfi srfi-1)
>>    #:use-module (srfi srfi-9)
>> +  #:use-module (ice-9 threads)
>>    #:use-module (ice-9 q)
>>    #:export (future make-future future? touch))
>
> I just realized that this patch introduces a circular dependency between
> the two modules.

Why is this a problem?  I think circular dependencies are fine as long
as there is not a cycle composed entirely of syntactic keyword
dependencies.

After introducing this circular dependency, I successfully bootstrapped
from scratch without difficulties.

What am I missing here?

      Mark