unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#52350: pypi import: ungraceful exit when recursively importing unknown package
@ 2021-12-07 11:26 Ricardo Wurmus
  2021-12-07 13:20 ` zimoun
  2022-03-08 10:09 ` zimoun
  0 siblings, 2 replies; 3+ messages in thread
From: Ricardo Wurmus @ 2021-12-07 11:26 UTC (permalink / raw)
  To: 52350

When using the pypi importer, a typo in the package name leads to an
ugly backtrace:

--8<---------------cut here---------------start------------->8---
./pre-inst-env guix import pypi -r pytestrerunfailures
following redirection to `https://pypi.org/pypi/pytestrerunfailures/json/'...
Backtrace:
In ice-9/boot-9.scm:
  1752:10  9 (with-exception-handler _ _ #:unwind? _ #:unwind-for-type _)
In unknown file:
           8 (apply-smob/0 #<thunk 7f3f0d996f60>)
In ice-9/boot-9.scm:
    724:2  7 (call-with-prompt _ _ #<procedure default-prompt-handler (k proc)>)
In ice-9/eval.scm:
    619:8  6 (_ #(#(#<directory (guile-user) 7f3f0d990c80>)))
In guix/ui.scm:
   2206:7  5 (run-guix . _)
  2169:10  4 (run-guix-command _ . _)
In guix/scripts/import.scm:
   124:11  3 (guix-import . _)
In guix/scripts/import/pypi.scm:
    96:18  2 (guix-import-pypi . _)
In guix/import/utils.scm:
   498:27  1 (recursive-import "pytestrerunfailures" #:repo->guix-package _ #:guix-name _ …)
   488:33  0 (lookup-node "pytestrerunfailures" #f)

guix/import/utils.scm:488:33: In procedure lookup-node:
Wrong number of values returned to continuation (expected 2)
--8<---------------cut here---------------end--------------->8---

-- 
Ricardo




^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#52350: pypi import: ungraceful exit when recursively importing unknown package
  2021-12-07 11:26 bug#52350: pypi import: ungraceful exit when recursively importing unknown package Ricardo Wurmus
@ 2021-12-07 13:20 ` zimoun
  2022-03-08 10:09 ` zimoun
  1 sibling, 0 replies; 3+ messages in thread
From: zimoun @ 2021-12-07 13:20 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: 52350

Hi Ricardo,

On Tue, 7 Dec 2021 at 12:28, Ricardo Wurmus <rekado@elephly.net> wrote:

> When using the pypi importer, a typo in the package name leads to an
> ugly backtrace:

This is already tracker by patch#45984.  Well, the patch fixes the
issue of "pypi" and other ones too (although some had probably been
fixed independently, I have not checked or rebased this old patch).
The patch had not been merged because we (with jeko) tried a
pair-programming hacking session to revamp all the error handling.
Well, we were not very far (not easy pair programming, remotely
neither :-)).

I do not have much time these days to concretely make this revamp.
Maybe, the initial patch could be applied as a stopgap waiting a clean
revamp.

1: <http://issues.guix.gnu.org/issue/45984>


Cheers,
simon




^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#52350: pypi import: ungraceful exit when recursively importing unknown package
  2021-12-07 11:26 bug#52350: pypi import: ungraceful exit when recursively importing unknown package Ricardo Wurmus
  2021-12-07 13:20 ` zimoun
@ 2022-03-08 10:09 ` zimoun
  1 sibling, 0 replies; 3+ messages in thread
From: zimoun @ 2022-03-08 10:09 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: 52350

Hi Ricardo,

On mar., 07 déc. 2021 at 11:26, Ricardo Wurmus <rekado@elephly.net> wrote:

> ./pre-inst-env guix import pypi -r pytestrerunfailures

--8<---------------cut here---------------start------------->8---
$ guix import pypi pytestrerunfailures
following redirection to `https://pypi.org/pypi/pytestrerunfailures/json/'...
guix import: error: failed to download meta-data for package 'pytestrerunfailures'

$ guix import pypi pytestrerunfailures -r
following redirection to `https://pypi.org/pypi/pytestrerunfailures/json/'...
--8<---------------cut here---------------end--------------->8---

Fixed by f7bb161445903c51d299307610403cb3de0fcc56, at least the ugly
backtrace.  However, it is still unsatisfactory since the error is not
correctly handled.

I propose to close because the exit is now graceful -- at least less
ungraceful. :-)


Cheers,
simon




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-03-08 10:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-07 11:26 bug#52350: pypi import: ungraceful exit when recursively importing unknown package Ricardo Wurmus
2021-12-07 13:20 ` zimoun
2022-03-08 10:09 ` zimoun

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).