GNU bug report logs - #75782
[PATCH cuirass] remote-server: Disable url-fetch* fallback when triggering substitutes.

Previous Next

Package: guix-patches;

Reported by: vicvbcun <guix <at> ikherbers.com>

Date: Thu, 23 Jan 2025 11:38:01 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #8 received at 75782 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: vicvbcun <guix <at> ikherbers.com>
Cc: 75782 <at> debbugs.gnu.org
Subject: Re: bug#75782: [PATCH cuirass] remote-server: Disable url-fetch*
 fallback when triggering substitutes.
Date: Thu, 13 Feb 2025 16:09:20 +0100
Hi,

vicvbcun <guix <at> ikherbers.com> skribis:

> If an output is larger than the configured cache bypass threshold and the
> substitute hasn't already been baked, guix publish returns a 404 response and
> schedules baking.  This error code causes `url-fetch*' to instead try fetching
> the narinfo from the Internet Archive.  Suppress this fallback behaviour as
> the sole purpose is triggering the baking of substitutes.
>
> * src/cuirass/scripts/remote-server.scm (trigger-substitutes-baking): Bind
> `%download-methods' to the empty list befor calling `url-fetch*'.

Good catch!

> +       (parameterize
> +           ;; XXX: guix publish returns a 404 response when the output is
> +           ;; large and has not already been baked.  By default, this causes
> +           ;; `url-fetch*' to fall back to the Internet Archive.  Disable
> +           ;; this.
> +           ((%download-methods '()))
> +         (url-fetch* narinfo-url tmp-file))))))

Shouldn’t it be '(upstream) rather than '()?

(I’m looking at guix/build/download.scm.)

Thanks,
Ludo’.




This bug report was last modified 99 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.