mbox

[bug#45983,0/3] Fix non-recursive importers gnu and json

Message ID 20210119152518.3464-1-zimon.toutoune@gmail.com
Headers show

Message

Simon Tournier Jan. 19, 2021, 3:25 p.m. UTC 
Dear,

The first patch is cosmetic.  Even, the file guix/import/gnu.scm should not
contain UI messages, IMHO.  Another story.

The two other patches fix the ugly backtrace throw by:

    guix import gnu do-not-exist
    guix import json do-no-exist

with something like:

--8<---------------cut here---------------start------------->8---
$ ./pre-inst-env guix import gnu do-not-exist
guix import: error: failed to determine latest release of GNU do-not-exist

$ ./pre-inst-env guix import json do-not-exist
guix import: error: invalid file name
--8<---------------cut here---------------end--------------->8---

See bug#44115 <http://issues.guix.gnu.org/44115#3> for details about the bug.

All the best,
simon


zimoun (3):
  import: gnu: Add internationalized messages.
  guix: gnu-maintenance: Fix error handling.
  scripts: import: json: Fix error handling.

 guix/gnu-maintenance.scm     |  5 ++++-
 guix/import/gnu.scm          | 11 +++++++++--
 guix/scripts/import/json.scm |  7 +++++--
 3 files changed, 18 insertions(+), 5 deletions(-)


base-commit: 2d9c6542c804eb2ef3d8934e1e3ab8b24e9bbafb