[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#39873] [PATCH 1/3] substitute: Use the same port for multiple reque
From: |
Christopher Baines |
Subject: |
[bug#39873] [PATCH 1/3] substitute: Use the same port for multiple request batches. |
Date: |
Mon, 09 Mar 2020 00:07:46 +0000 |
User-agent: |
mu4e 1.2.0; emacs 26.3 |
Ludovic Courtès <address@hidden> writes:
> Hi Christopher!
>
> Christopher Baines <address@hidden> skribis:
>
>> In http-multiple-get.
>
> Leftover? :-)
No, just clarifying what function is being referred to. Although the
line below reveals that also.
>> * guix/scripts/substitute.scm (http-multiple-get): Switch port to p in one
>> occurrence.
>> ---
>> guix/scripts/substitute.scm | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/guix/scripts/substitute.scm b/guix/scripts/substitute.scm
>> index dfb975a24a..5ed43d36c9 100755
>> --- a/guix/scripts/substitute.scm
>> +++ b/guix/scripts/substitute.scm
>> @@ -545,7 +545,7 @@ initial connection on which HTTP requests are sent."
>> (()
>> (reverse result))
>> (remainder
>> - (connect port remainder result))))
>> + (connect p remainder result))))
>
> LGTM!
>
> Did you notice an occurrence of this bug somewhere?
I was looking for something that could be going wrong here as I was
seeing some kind of error talking to Cuirass using this in the Guix Data
Service, and happened to notice this in the code.
signature.asc
Description: PGP signature