‘guix refresh’ broken on master?

‘guix refresh’ broken on master?

[Tobias Geerinckx-Rice]

[-- Attachment #1.1: Type: text/plain, Size: 1542 bytes --]

Guix, David,

Commit 3e0c036 (‘import: Add importer for rust crates’) reliably breaks
all calls to ‘guix refresh’ for me:

  nckx@ubuntu$ ~/guix/pre-inst-env guix refresh --help
  ;;; note: source file /home/nckx/guix/guix/utils.scm
  ;;;       newer than compiled /home/nckx/guix/guix/utils.go
  ;;; note: source file /home/nckx/guix/guix/store.scm
  ;;;       newer than compiled /home/nckx/guix/guix/store.go
  warning: failed to install locale: Invalid argument
  ;;; note: source file /home/nckx/guix/guix/scripts/refresh.scm
  ;;;       newer than compiled /home/nckx/guix/guix/scripts/refresh.go
  ;;; note: source file /home/nckx/guix/guix/upstream.scm
  ;;;       newer than compiled /home/nckx/guix/guix/upstream.go
  ;;; note: source file /home/nckx/guix/guix/gnu-maintenance.scm
  ;;;       newer than compiled /home/nckx/guix/guix/gnu-maintenance.go
  ;;; note: source file /home/nckx/guix/guix/import/utils.scm
  ;;;       newer than compiled /home/nckx/guix/guix/import/utils.go
  ;;; note: source file /home/nckx/guix/guix/import/json.scm
  ;;;       newer than compiled /home/nckx/guix/guix/import/json.go
  guix: refresh: command not found
  Try `guix --help' for more information.

Running the same command one commit earlier spews a lot more ‘note’s
before showing the help message, suggesting that one of the above files
is to blame. I'm not sure how to coax more information out of Guile, though.

Can anyone reproduce this, or suggest a way Forward?

Kind regards,

T G-R


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 476 bytes --]

Re: ‘guix refresh’ broken on master?

[David Craven]

Ah there does seem to be a problem.

First this hunk is in the wrong patch:

 diff --git a/guix/scripts/refresh.scm b/guix/scripts/refresh.scm
index f8fb3f80c..72f51cbff 100644
--- a/guix/scripts/refresh.scm
+++ b/guix/scripts/refresh.scm
@@ -39,6 +39,7 @@
                           %kernel.org-updater))
   #:use-module (guix import elpa)
   #:use-module (guix import cran)
+  #:use-module (guix import crate)
   #:use-module (guix import hackage)
   #:use-module (guix gnupg)
   #:use-module (gnu packages)

It should have gone in the next one called Add updater for rust
crates. The updater depends on guile-json so this means that all of
guix refresh depends on guile-json.

Re: ‘guix refresh’ broken on master?

[Ludovic Courtès]

Hello!

Tobias Geerinckx-Rice <me@tobias.gr> skribis:

> Commit 3e0c036 (‘import: Add importer for rust crates’) reliably breaks
> all calls to ‘guix refresh’ for me:

This is because your installation lacks Guile-JSON, probably.  Fixed in
commit 99ec0cb397453802788e7095724dcfbca3ca1723.

The ‘list-updaters’ macro in (guix scripts refresh) ensures that only
updaters that are actually available at run time are visible.

Thanks!

Ludo’.

Re: ‘guix refresh’ broken on master?

[David Craven]

> This is because your installation lacks Guile-JSON, probably.  Fixed in
> commit 99ec0cb397453802788e7095724dcfbca3ca1723.

Ah that was easier than expected.