GNU bug report logs - #61574
[PATCH v2] scripts: repl: Extend REPL %load-path with all channels.

Previous Next

Package: guix;

Reported by: Simon Tournier <zimon.toutoune <at> gmail.com>

Date: Fri, 17 Feb 2023 14:57:01 UTC

Severity: normal

Merged with 61343

Full log


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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Simon Tournier <zimon.toutoune <at> gmail.com>
Cc: 61574 <at> debbugs.gnu.org, Josselin Poiret <dev <at> jpoiret.xyz>,
 61343 <at> debbugs.gnu.org, iyzsong <at> envs.net, Andrew Tropin <andrew <at> trop.in>
Subject: Re: bug#61574: bug#61343: bug#61574: [PATCH v2] scripts: repl:
 Extend REPL %load-path with all channels.
Date: Fri, 03 Mar 2023 12:14:00 +0100
Simon Tournier <zimon.toutoune <at> gmail.com> skribis:

> 1 file changed, 3 insertions(+), 3 deletions(-)
> guix/scripts/repl.scm | 6 +++---
>
> modified   guix/scripts/repl.scm
> @@ -27,7 +27,7 @@ (define-module (guix scripts repl)
>    #:use-module (srfi srfi-37)
>    #:use-module (ice-9 match)
>    #:use-module (rnrs bytevectors)
> -  #:autoload   (guix describe) (current-profile)
> +  #:autoload   (gnu packages) (%package-module-path)
>    #:autoload   (system repl repl) (start-repl)
>    #:autoload   (system repl server)
>                    (make-tcp-server-socket make-unix-domain-server-socket)
> @@ -193,7 +193,7 @@ (define script
>           ;; 'set-program-arguments'.  This in turn ensures that
>           ;; (%package-module-path) will contain entries for the channels
>           ;; available in the current profile.
> -         (current-profile)
> +         (%package-module-path)

Per the comment right above, this change would break the program name
memoization, wouldn’t it?

Ludo’.




This bug report was last modified 2 years and 103 days ago.

Previous Next


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