bug#38715: Issue with guile-email: unbound variable

bug#38715: Issue with guile-email: unbound variable

[Jelle Licht]

Hey guix,

Pretty recently, it seems something weird is happening when playing with
guix on the REPL.

On master, 5a947e118fb72b8fb1fd0c8d0b783fc7cde0c461:

--8<---------------cut here---------------start------------->8---
$ guix repl
> (use-modules (gnu packages gnupg))
While compiling expression:
error: guile-email: unbound variable.
--8<---------------cut here---------------end--------------->8---

Loading up the `(gnu packages mail)' module first seems to work around
the compilation error. My best guess is that this has something to do
with a circular reference between guile modules, but I am not certain on
how to easily debug (and fix) this.

- Jelle

bug#38715: Issue with guile-email: unbound variable

[Tobias Geerinckx-Rice via Bug reports for GNU Guix]

[-- Attachment #1: Type: text/plain, Size: 3996 bytes --]

Jelle,

Jelle Licht 写道：
> Hey guix,
>
> Pretty recently, it seems something weird is happening when 
> playing with
> guix on the REPL.
>
> On master, 5a947e118fb72b8fb1fd0c8d0b783fc7cde0c461:
>
> --8<---------------cut 
> here---------------start------------->8---
> $ guix repl
>> (use-modules (gnu packages gnupg))
> While compiling expression:
> error: guile-email: unbound variable.
> --8<---------------cut 
> here---------------end--------------->8---

Ah!  Glad someone else is seeing this too.  Below's the full 
backtrace if it's of interest to anyone.

Kind regards,

T G-R

Backtrace:
In ice-9/boot-9.scm:
   222:29 19 (map1 (((gnu packages imagemagick)) ((gnu packages 
   w3m)) ((gnu packages wget)) ((gnu packages autotools)) ((gnu 
   packages base)) ((gnu packages compression)) ((gnu packages 
   node)) ((gnu packages xml)) ((# …)) …))
   222:29 18 (map1 (((gnu packages w3m)) ((gnu packages wget)) 
   ((gnu packages autotools)) ((gnu packages base)) ((gnu packages 
   compression)) ((gnu packages node)) ((gnu packages xml)) ((gnu 
   packages glib)) ((gnu # acl)) # …))
   222:29 17 (map1 (((gnu packages wget)) ((gnu packages 
   autotools)) ((gnu packages base)) ((gnu packages compression)) 
   ((gnu packages node)) ((gnu packages xml)) ((gnu packages 
   glib)) ((gnu packages acl)) ((gnu # mail)) …))
   222:29 16 (map1 (((gnu packages autotools)) ((gnu packages 
   base)) ((gnu packages compression)) ((gnu packages node)) ((gnu 
   packages xml)) ((gnu packages glib)) ((gnu packages acl)) ((gnu 
   packages mail)) ((gnu # #)) (#) …))
   222:29 15 (map1 (((gnu packages base)) ((gnu packages 
   compression)) ((gnu packages node)) ((gnu packages xml)) ((gnu 
   packages glib)) ((gnu packages acl)) ((gnu packages mail)) 
   ((gnu packages messaging)) ((gnu # #)) (#) …))
   222:29 14 (map1 (((gnu packages compression)) ((gnu packages 
   node)) ((gnu packages xml)) ((gnu packages glib)) ((gnu 
   packages acl)) ((gnu packages mail)) ((gnu packages messaging)) 
   ((gnu packages package-management)) # …))
   222:29 13 (map1 (((gnu packages node)) ((gnu packages xml)) 
   ((gnu packages glib)) ((gnu packages acl)) ((gnu packages 
   mail)) ((gnu packages messaging)) ((gnu packages 
   package-management)) ((gnu packages perl)) ((# …)) …))
   222:29 12 (map1 (((gnu packages xml)) ((gnu packages glib)) 
   ((gnu packages acl)) ((gnu packages mail)) ((gnu packages 
   messaging)) ((gnu packages package-management)) ((gnu packages 
   perl)) ((gnu packages pdf)) ((gnu …)) …))
   222:29 11 (map1 (((gnu packages glib)) ((gnu packages acl)) 
   ((gnu packages mail)) ((gnu packages messaging)) ((gnu packages 
   package-management)) ((gnu packages perl)) ((gnu packages pdf)) 
   ((gnu packages scheme)) ((…)) …))
   222:29 10 (map1 (((gnu packages acl)) ((gnu packages mail)) 
   ((gnu packages messaging)) ((gnu packages package-management)) 
   ((gnu packages perl)) ((gnu packages pdf)) ((gnu packages 
   scheme)) ((gnu packages speech)) (#) …))
   222:17  9 (map1 (((gnu packages mail)) ((gnu packages 
   messaging)) ((gnu packages package-management)) ((gnu packages 
   perl)) ((gnu packages pdf)) ((gnu packages scheme)) ((gnu 
   packages speech)) ((gnu packages xiph)) (#) …))
  2800:17  8 (resolve-interface (gnu packages mail) #:select _ 
  #:hide _ #:prefix _ #:renamer _ #:version _)
In ice-9/threads.scm:
    390:8  7 (_ _)
In ice-9/boot-9.scm:
  2726:13  6 (_)
In ice-9/threads.scm:
    390:8  5 (_ _)
In ice-9/boot-9.scm:
  2994:20  4 (_)
   2312:4  3 (save-module-excursion #<procedure 748881fa1db0 at 
   ice-9/boot-9.scm:2995:21 ()>)
  3014:26  2 (_)
In unknown file:
           1 (primitive-load-path "gnu/packages/mail" #<procedure 
           748881c88280 at ice-9/boot-9.scm:3001:37 ()>)
In gnu/packages/mail.scm:
   779:28  0 (_)

gnu/packages/mail.scm:779:28: error: guile-email: unbound variable

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

bug#38715: Issue with guile-email: unbound variable

[Arun Isaac]

[-- Attachment #1: Type: text/plain, Size: 391 bytes --]


> My best guess is that this has something to do with a circular
> reference between guile modules, but I am not certain on how to easily
> debug (and fix) this.

I updated the guile-email package two days ago. I hope that is not what
introduced this problem, and I don't see how it could have. Even `guix
build guile-email` on the terminal fails with a guile-email unbound
variable error.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

bug#38715: Issue with guile-email: unbound variable

[Jelle Licht]

Arun Isaac <arunisaac@systemreboot.net> writes:

>> My best guess is that this has something to do with a circular
>> reference between guile modules, but I am not certain on how to easily
>> debug (and fix) this.
>
> I updated the guile-email package two days ago. I hope that is not what
> introduced this problem, and I don't see how it could have. Even `guix
> build guile-email` on the terminal fails with a guile-email unbound
> variable error.

I do not think it was that commit, as I found the offending commit:
c7b2b539802eaa3f969e212c98eb671a1a75e9f3

This is the commit that adds mumimu to gnu/packages/mail.scm, as well as
to the inputs of mumi. Reverting this commit (at least) solves the issue
I had in the first mail.

Could it be that the reference to guile-email in the version field of
mumimu created this issue?
--8<---------------cut here---------------start------------->8---
(version (git-version (package-version guile-email) revision commit))
--8<---------------cut here---------------end--------------->8---

- Jelle

bug#38715: Issue with guile-email: unbound variable

[Christopher Baines]

[-- Attachment #1: Type: text/plain, Size: 1473 bytes --]


Jelle Licht <jlicht@fsfe.org> writes:

> Arun Isaac <arunisaac@systemreboot.net> writes:
>
>>> My best guess is that this has something to do with a circular
>>> reference between guile modules, but I am not certain on how to easily
>>> debug (and fix) this.
>>
>> I updated the guile-email package two days ago. I hope that is not what
>> introduced this problem, and I don't see how it could have. Even `guix
>> build guile-email` on the terminal fails with a guile-email unbound
>> variable error.
>
> I do not think it was that commit, as I found the offending commit:
> c7b2b539802eaa3f969e212c98eb671a1a75e9f3
>
> This is the commit that adds mumimu to gnu/packages/mail.scm, as well as
> to the inputs of mumi. Reverting this commit (at least) solves the issue
> I had in the first mail.
>
> Could it be that the reference to guile-email in the version field of
> mumimu created this issue?
> --8<---------------cut here---------------start------------->8---
> (version (git-version (package-version guile-email) revision commit))
> --8<---------------cut here---------------end--------------->8---

I also spotted this. It looks to me sort of like a copy/paste error, I'm
not sure why this variant of mu would take the version of the
guile-email package.

Anyway, to work around this issue I've pushed [1], which just replaces
the reference with the current value.

1: https://git.savannah.gnu.org/cgit/guix.git/commit/?id=03460d2bd327815535801e3780c224f91af9b445

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 962 bytes --]

bug#38715: Issue with guile-email: unbound variable

[Ludovic Courtès]

Hi Chris,

Christopher Baines <mail@cbaines.net> skribis:

> Jelle Licht <jlicht@fsfe.org> writes:

[...]

>> I do not think it was that commit, as I found the offending commit:
>> c7b2b539802eaa3f969e212c98eb671a1a75e9f3
>>
>> This is the commit that adds mumimu to gnu/packages/mail.scm, as well as
>> to the inputs of mumi. Reverting this commit (at least) solves the issue
>> I had in the first mail.
>>
>> Could it be that the reference to guile-email in the version field of
>> mumimu created this issue?
>> --8<---------------cut here---------------start------------->8---
>> (version (git-version (package-version guile-email) revision commit))
>> --8<---------------cut here---------------end--------------->8---
>
> I also spotted this. It looks to me sort of like a copy/paste error, I'm
> not sure why this variant of mu would take the version of the
> guile-email package.
>
> Anyway, to work around this issue I've pushed [1], which just replaces
> the reference with the current value.
>
> 1: https://git.savannah.gnu.org/cgit/guix.git/commit/?id=03460d2bd327815535801e3780c224f91af9b445

My bad!  I copied the definition of ‘mumimu’ from Mumi itself and didn’t
pay enough attention.

The version number makes little sense anyway.  You did the right thing,
thank you, and apologies for the breakage!

Ludo’.

bug#38715: Issue with guile-email: unbound variable

[Ricardo Wurmus]

Christopher Baines <mail@cbaines.net> writes:

>> Could it be that the reference to guile-email in the version field of
>> mumimu created this issue?
>> --8<---------------cut here---------------start------------->8---
>> (version (git-version (package-version guile-email) revision commit))
>> --8<---------------cut here---------------end--------------->8---
>
> I also spotted this. It looks to me sort of like a copy/paste error, I'm
> not sure why this variant of mu would take the version of the
> guile-email package.

This is a copy/paste error.  The origin of this expression is the
guix.scm file of mumi.  Ludo then wrote a mumi service and copied the
definition of mumimu from that guix.scm file.

(package-version guile-email) is obviously wrong.  It should have been
(package-version mu), because mumimu is a fork of mu.

--
Ricardo

bug#38715: Issue with guile-email: unbound variable

[Ricardo Wurmus]

Ricardo Wurmus <rekado@elephly.net> writes:

> Christopher Baines <mail@cbaines.net> writes:
>
>>> Could it be that the reference to guile-email in the version field of
>>> mumimu created this issue?
>>> --8<---------------cut here---------------start------------->8---
>>> (version (git-version (package-version guile-email) revision commit))
>>> --8<---------------cut here---------------end--------------->8---
>>
>> I also spotted this. It looks to me sort of like a copy/paste error, I'm
>> not sure why this variant of mu would take the version of the
>> guile-email package.
>
> This is a copy/paste error.  The origin of this expression is the
> guix.scm file of mumi.  Ludo then wrote a mumi service and copied the
> definition of mumimu from that guix.scm file.
>
> (package-version guile-email) is obviously wrong.  It should have been
> (package-version mu), because mumimu is a fork of mu.

I’m closing this now as mumi no longer depends on a fork of mu; instead
it uses guile-xapian with its own database.

-- 
Ricardo