bug#69291: closed ([PATCH 0/5] Start making substitute code less coupled)

[GNU bug Tracking System]

Your message dated Sun, 21 Apr 2024 12:24:56 +0100
with message-id <875xwb2c1j.fsf@cbaines.net>
and subject line Re: [bug#69291] [PATCH 5/5] scripts: substitute: Extract 
script specific output from download-nar.
has caused the debbugs.gnu.org bug report #69291,
regarding [PATCH 0/5] Start making substitute code less coupled
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
69291: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=69291
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH 0/5] Start making substitute code less coupled Date: Tue, 20 Feb 2024 19:05:07 +0000 User-agent: mu4e 1.10.8; emacs 29.1

These changes should help with using the substitute code in a Guile
implementation of the Guix daemon.


Christopher Baines (5):
  scripts: substitute: Remove side effect warning from network-error?.
  scripts: substitute: Allow not using with-timeout in download-nar.
  scripts: substitute: Replace some leave calls with raise.
  scripts: substitute: Untangle selecting fast vs small compressions.
  scripts: substitute: Extract script specific output from download-nar.

 guix/scripts/substitute.scm | 207 +++++++++++++++++++++---------------
 1 file changed, 123 insertions(+), 84 deletions(-)


base-commit: 3d061d9677027be7651f8e5a3a02e19daacd9a85
-- 
2.41.0

signature.asc
Description: PGP signature

--- End Message ---

--- Begin Message --- Subject: Re: [bug#69291] [PATCH 5/5] scripts: substitute: Extract script specific output from download-nar. Date: Sun, 21 Apr 2024 12:24:56 +0100 User-agent: mu4e 1.12.2; emacs 29.3

Christopher Baines <mail@cbaines.net> writes:

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Christopher Baines <mail@cbaines.net> skribis:
>>
>>> As this moves download-nar in a direction where it could be used outside the
>>> substitute script.
>>>
>>> * guix/scripts/substitute.scm (download-nar): Return more information and 
>>> move
>>> status-port output to…
>>> (guix-substitute): here.
>>>
>>> Change-Id: Icbddb9a47620b3520cdd2e8095f37a99824c1ce0
>>
>> LGTM.
>
> Looking at this patch some more, I missed the failure case in
> process-substitution/fallback, so I'll send an updated patch (for this
> and the previous patch).

I've sent a couple of updated patches as part of the series to
https://issues.guix.gnu.org/70494

signature.asc
Description: PGP signature

--- End Message ---