bug#64406: [PATCH] Improve commands to manage Python imports

[Augusto Stoffel]

On Sat,  1 Jul 2023 at 22:16, Matthias Meulien wrote:

> -from sys import argv, stdin
> +from sys import argv, exit, stdin

FWIW, exit is already in the global namespace.

> -        (unless (eq 0 status)
> +        (cond
> +         ((eq 1 status)
>            (error "%s exited with status %s (maybe isort is missing?)"
>                   python-interpreter status))
> +         ((eq 2 status)
> +          (error "%s exited with status %s (maybe isort version is <5.7.0?)"
> +                 python-interpreter status)))

This change implies that the "success" branch may run if the exit code
is nonzero (without knowing all the isort internals, it can't be
excluded that an exit code > 2 is used somewhere).  I suggest instead a
(pcase status ...) to construct the " (maybe... ?)" segment of the error
message.

We could also take this opportunity to distinguish between random
exceptions happening in the script (which likely leads to exit code 1)
and the ModuleNotFoundError case.