guix pull: channel d8feee9 - extraneous field initializers (sha256)

  • Done
  • quality assurance status badge
Details
5 participants
  • Jan Nieuwenhuizen
  • Ludovic Courtès
  • Matias Jose Seco Baccanelli
  • Tobias Geerinckx-Rice
  • Ricardo Wurmus
Owner
unassigned
Submitted by
Matias Jose Seco Baccanelli
Severity
serious
Merged with
M
M
Matias Jose Seco Baccanelli wrote on 22 May 2020 12:14
(address . bug-guix@gnu.org)
20200522121438.658271a5.matias_jose_seco@autoproduzioni.net
Hello!

I've encountered the following exception, during regular 'guix pull'
command:

Updating channel 'guix' from Git repository at
d8feee9 Computing Guix derivation for 'x86_64-linux'...
ice-9/psyntax.scm:1585:32: In procedure expand-macro: Syntax error:
unknown location: %origin: extraneous field initializers (sha256) in
form (%origin (method url-fetch) (uri (string-append
"mirror://gnu/bash/bash-" version ".tar.gz")) (sha256 (base32
"0kgvfwqdcd90waczf4gx39xnrxzijhjrzyzv7s8v4w31qqm0za5l")) (patch-flags
(quote ("-p0"))) (patches (cons (search-patch
"bash-linux-pgrp-pipe.patch") %patch-series-5.0))) guix pull: error:
You found a bug: the program
'/gnu/store/fsch1y8hilmkg21rmgk256j3wfi2w7wk-compute-guix-derivation'
failed to compute the derivation for Guix (version:
"d8feee9f18ede0d1ea4d0e4876f09a0dab770295"; system: "x86_64-linux";
host version: "5c10d55206a4f7a9b932ff08512a4f50c1db35be"; pull-version:
1). Please report it by email to <bug-guix@gnu.org>.

Available for further informations,
saluti,
Matias
R
R
Ricardo Wurmus wrote on 22 May 2020 16:49
(name . Matias Jose Seco Baccanelli)(address . matias_jose_seco@autoproduzioni.net)
87imgoc9du.fsf@elephly.net
Hi,

I can reproduce this:

Toggle snippet (25 lines)
[rwurmus@bimsb-sys02.mdc-berlin.net:~/code/guix] (1005) $ ~/.config/guix/current/bin/guix pull
Updating channel 'guix' from Git repository at 'https://git.savannah.gnu.org/git/guix.git'...
Building from this channel:
guix https://git.savannah.gnu.org/git/guix.git cd5d5f5
Computing Guix derivation for 'x86_64-linux'... ice-9/psyntax.scm:1585:32: In procedure expand-macro:
Syntax error:
unknown location: %origin: extraneous field initializers (sha256) in form (%origin (method url-fetch) (uri (string-append "mirror://gnu/bash/bash-" version ".tar.gz")) (sha256 (base32 "0kgvfwqdcd90waczf4gx39xnrxzijhjrzyzv7s8v4w31qqm0za5l")) (patch-flags (quote ("-p0"))) (patches (cons (search-patch "bash-linux-pgrp-pipe.patch") %patch-series-5.0)))
-guix pull: error: You found a bug: the program '/gnu/store/fsch1y8hilmkg21rmgk256j3wfi2w7wk-compute-guix-derivation'
failed to compute the derivation for Guix (version: "cd5d5f53228fd5bf96d9f790aa2606ae71fa68d7"; system: "x86_64-linux";
host version: "be0ecfb1787b9e6954bf745bceeb1b9d2669d51a"; pull-version: 1).
Please report it by email to <bug-guix@gnu.org>.

[rwurmus@bimsb-sys02.mdc-berlin.net:~/code/guix] (1006) $ ~/.config/guix/current/bin/guix describe
Generation 8 Apr 15 2020 10:05:07 (current)
guix-bimsb 78833aa
repository URL: https://github.com/BIMSBbioinfo/guix-bimsb
branch: master
commit: 78833aa4c89fb317cdd67f17ff27111228d870a9
…
guix be0ecfb
repository URL: https://git.savannah.gnu.org/git/guix.git
branch: master
commit: be0ecfb1787b9e6954bf745bceeb1b9d2669d51a

It seems to be related to the commits between
c7d2dd69004b020de5d86898d2497ab3c8435c37 and
1ad5209d904d471ded6cf53b4e29b64e963dea3f.

I’m trying this now as a work-around:

~/.config/guix/current/bin/guix pull --commit=1ad5209d904d471ded6cf53b4e29b64e963dea3f

--
Ricardo
R
R
Ricardo Wurmus wrote on 22 May 2020 16:50
(address . control@debbugs.gnu.org)
87ftbsc9cq.fsf@elephly.net
severity 41457 serious
quit
L
L
Ludovic Courtès wrote on 22 May 2020 17:12
(name . Ricardo Wurmus)(address . rekado@elephly.net)
87mu609f5z.fsf@gnu.org
Hi,

Here’s a reproducer that works well for me:

guix time-machine --commit=be0ecfb1787b9e6954bf745bceeb1b9d2669d51a -- \
pull --commit=d8feee9f18ede0d1ea4d0e4876f09a0dab770295 -p /tmp/test

Before that I was able to pull from
df05842332be80ed7f53022402b95cf711163b41 to
d8feee9f18ede0d1ea4d0e4876f09a0dab770295 just fine.

Ludo’.
L
L
Ludovic Courtès wrote on 22 May 2020 20:01
(name . Ricardo Wurmus)(address . rekado@elephly.net)
878shjalwr.fsf@gnu.org
Hi,

Ricardo Wurmus <rekado@elephly.net> skribis:

Toggle quote (12 lines)
> [rwurmus@bimsb-sys02.mdc-berlin.net:~/code/guix] (1005) $ ~/.config/guix/current/bin/guix pull
> Updating channel 'guix' from Git repository at 'https://git.savannah.gnu.org/git/guix.git'...
> Building from this channel:
> guix https://git.savannah.gnu.org/git/guix.git cd5d5f5
> Computing Guix derivation for 'x86_64-linux'... ice-9/psyntax.scm:1585:32: In procedure expand-macro:
> Syntax error:
> unknown location: %origin: extraneous field initializers (sha256) in form (%origin (method url-fetch) (uri (string-append "mirror://gnu/bash/bash-" version ".tar.gz")) (sha256 (base32 "0kgvfwqdcd90waczf4gx39xnrxzijhjrzyzv7s8v4w31qqm0za5l")) (patch-flags (quote ("-p0"))) (patches (cons (search-patch "bash-linux-pgrp-pipe.patch") %patch-series-5.0)))
> -guix pull: error: You found a bug: the program '/gnu/store/fsch1y8hilmkg21rmgk256j3wfi2w7wk-compute-guix-derivation'
> failed to compute the derivation for Guix (version: "cd5d5f53228fd5bf96d9f790aa2606ae71fa68d7"; system: "x86_64-linux";
> host version: "be0ecfb1787b9e6954bf745bceeb1b9d2669d51a"; pull-version: 1).
> Please report it by email to <bug-guix@gnu.org>.

Good news, this is fixed by 50ea3135e0948a042cd3b899e970f6ade291a0c2!

As noted in the commit log, this issue stemmed from a change in
#:autoload semantics in Guile 3.0. In 3.0, the line:

#:autoload (gcrypt hash) (port-sha256)

means that only ‘port-sha256’ is imported. Conversely, in 2.2 (which is
the Guile version used by the ‘compute-guix-derivation’ program above),
that #:autoload line means that all the (gcrypt hash) bindings are
imported. That includes the ‘sha256’ bindings, which would thus prevent
matching the ‘sha256’ literal in the ‘origin-compatibility-helper’
macro. Terrible!

Ludo’.
Closed
J
J
Jan Nieuwenhuizen wrote on 22 May 2020 20:49
(address . ludo@gnu.org)(address . 41457@debbugs.gnu.org)
87a71zx0sx.fsf@gnu.org
Ludovic Courtès writes:

Hello,

Toggle quote (14 lines)
> Good news, this is fixed by 50ea3135e0948a042cd3b899e970f6ade291a0c2!
>
> As noted in the commit log, this issue stemmed from a change in
> #:autoload semantics in Guile 3.0. In 3.0, the line:
>
> #:autoload (gcrypt hash) (port-sha256)
>
> means that only ‘port-sha256’ is imported. Conversely, in 2.2 (which is
> the Guile version used by the ‘compute-guix-derivation’ program above),
> that #:autoload line means that all the (gcrypt hash) bindings are
> imported. That includes the ‘sha256’ bindings, which would thus prevent
> matching the ‘sha256’ literal in the ‘origin-compatibility-helper’
> macro. Terrible!

Huh? I just used #:autoload for the first time (in the native
qemu-image patch thaw way may not be using after all ...) and so I just
read the documentation.

]I am pretty sure that t]The 3.0 documentation says that "when a module
is autoloaded, all symbols become available"?


So...what's going on here?

janneke

--
Jan Nieuwenhuizen <janneke@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com| Avatar® http://AvatarAcademy.com
L
L
Ludovic Courtès wrote on 22 May 2020 22:13
(name . Jan Nieuwenhuizen)(address . janneke@gnu.org)(address . 41457@debbugs.gnu.org)
871rnbafsu.fsf@gnu.org
Hi,

Jan Nieuwenhuizen <janneke@gnu.org> skribis:

Toggle quote (7 lines)
> ]I am pretty sure that t]The 3.0 documentation says that "when a module
> is autoloaded, all symbols become available"?
>
> http://git.savannah.gnu.org/cgit/guile.git/tree/doc/ref/api-modules.texi#n298
>
> So...what's going on here?

Ah! It’s an omission, fixed in Guile commit
723e483e7d88897f1ad654028e86a87dd040b886.

The changed was made late in the 2.9.x series, in commit
cb14fd214365e50b6b1655616ae74d0228933bbd, and later documented in
‘NEWS’.

Thanks for the heads-up!

Ludo’.
R
R
Ricardo Wurmus wrote on 23 May 2020 06:51
Re: Processed (with 1 errors): Re: bug#41449: Bash package breaking guix build after hash system update
(name . GNU bug tracker automated control server)(address . control@debbugs.gnu.org)
87a71zckza.fsf@elephly.net
severity 41449 serious
close 41449
merge 41457 41449
quit
T
T
Tobias Geerinckx-Rice wrote on 23 May 2020 14:09
Re: Processed (with 1 errors): your mail
(name . GNU bug tracker automated control server)(address . control@debbugs.gnu.org)
87a71yx37l.fsf@nckx
severity 41462 serious
merge 41457 41462
-----BEGIN PGP SIGNATURE-----

iHUEARYKAB0WIQT12iAyS4c9C3o4dnINsP+IT1VteQUCXskSfgAKCRANsP+IT1Vt
eS9MAP983O2etSA4QUxLsAkra/94rbjk+A47EGwiSDTQHTfASwD/YZsp+w86KLOh
qxvPGrikmwJMEFEGpTDI2EWDLHj0cAg=
=hRgF
-----END PGP SIGNATURE-----

?
Your comment

This issue is archived.

To comment on this conversation send an email to 41457@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 41457
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch