bug#48682: closed (recursive import between (gnu packages chez) and (gnu packages racket))

[GNU bug Tracking System]

Your message dated Sat, 29 May 2021 22:15:30 +0200
with message-id <87o8ct5mal.fsf@gnu.org>
and subject line Re: bug#48682: recursive import between (gnu packages chez) 
and (gnu packages racket)
has caused the debbugs.gnu.org bug report #48682,
regarding recursive import between (gnu packages chez) and (gnu packages racket)
to be marked as done.

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


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

--- Begin Message --- Subject: recursive import between (gnu packages chez) and (gnu packages racket) Date: Wed, 26 May 2021 20:54:55 +0200

bisected it to this commit:
ae88e30a0f8403e781f8b01262766cdc46b1018a

* bisect.sh: the script used for git bisect run
* BISECT_LOG: obvious
* test.scm: more detailed test, with backtrace. guile dies while
  printing the backtrace.

Issue doesn't occur when i comment out the use-module line in
racket.scm that imports (@ (gnu packages chez) chez-scheme).

IRC discussion:
roptat suggests it might be triggered by something inheriting from
chez-scheme, cbaines says recursive modules usually don't cause issues,
only recursive inputs do. i haven't found any relevant inherits with a
quick grep, but maybe it's transitive or something.
I also haven't found any other mention of chez-scheme in racket.scm
other than the line that imports it, which is weird, given that Racket
is now built on Chez, so I'd expect it to use it as an input. I guess
it's a bundled version? In any case I don't think I should just remove
the import, because it will be needed eventually, so this issue needs
to be fixed by then.

That's as far as I've gotten today, will hopefully have time to look
into it tomorrow.

ps.: should I forward this to the guile list?

test.scm
Description: Text Data

bisect.sh
Description: application/shellscript

BISECT_LOG
Description: Binary data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#48682: recursive import between (gnu packages chez) and (gnu packages racket)
>
>
>
>
> Date: 
>
> Sat, 29 May 2021 22:15:30 +0200
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)
>
>
>
>
> Hi,
>
> raingloom <raingloom@riseup.net> skribis:
>
> >>From cef344639ccbf45b5397368d643a36fa48dac098 Mon Sep 17 00:00:00 2001
> > From: raingloom <raingloom@riseup.net>
> > Date: Fri, 28 May 2021 03:59:16 +0200
> > Subject: [PATCH] gnu: Break up import loop between (gnu packages racket) and
> >  (gnu packages chez).
> >
> > * gnu/packages/racket.scm: Remove (gnu packages chez) import.
> > ---
> >  gnu/packages/racket.scm | 2 --
> >  1 file changed, 2 deletions(-)
>
> Applied, thanks!
>
> > diff --git a/gnu/packages/racket.scm b/gnu/packages/racket.scm
> > index be33270b30..2d606071fe 100644
> > --- a/gnu/packages/racket.scm
> > +++ b/gnu/packages/racket.scm
> > @@ -32,8 +32,6 @@
> >    #:use-module (ice-9 match)
> >    #:use-module (gnu packages)
> >    #:use-module (gnu packages bash)
> > -  #:use-module ((gnu packages chez)
> > -                #:select (chez-scheme))
> >    #:use-module (gnu packages compression)
>
> In general we cannot use #:select for (gnu packages …) modules because
> that doesn’t play well with circular module dependencies.
>
> That said… what problem is this fixing raingloom?
>
> Usually, circular top-level references lead things like ‘guix show
> chez-scheme’ (this amounts to loading (gnu packages chez)), but it
> worked fine for me.
>
> Thanks,
> Ludo’.
>
>
> --- End Message ---