From: Tanguy Le Carrour <tanguy@bioneland.org>
To: Leo Famulari <leo@famulari.name>
Cc: 39777@debbugs.gnu.org
Subject: [bug#39777] [PATCH V3 04/11] gnu: python-jsonschema: Update to 3.2.0.
Date: Fri, 13 Mar 2020 09:45:41 +0100 [thread overview]
Message-ID: <20200313084541.zeditu3oaouqm2du@rafflesia> (raw)
In-Reply-To: <20200312174421.GA2140@jasmine.lan>
Le 03/12, Leo Famulari a écrit :
> On Thu, Mar 12, 2020 at 09:48:28AM +0100, Tanguy Le Carrour wrote:
> > The test relies on a call to `subprocess.check_output` to run
> > `python -m jsonschema --version`, but it behaves like `--version` was
> > never passed to the command?!
>
> How do you know it behaves like that? Is there some error output?
Actually, you're right, I don't know, because I didn't reproduce the exact same
error!
```
$ ./pre-inst-env guix build python-jsonschema -K
[…]
subprocess.CalledProcessError: Command '['/gnu/store/l8nphg0idd8pfddyad8f92lx8d1hc053-python-wrapper-3.7.4/bin/python', '-m', 'jsonschema', '--version']' returned non-zero exit status 1.
```
The exit status is 1.
To reproduce it in the build directory I did the following:
```
$ cd /tmp/guix-build-python-jsonschema-3.2.0.drv-0/jsonschema-3.2.0/
$ guix environment python-jsonschema --ad-hoc python-importlib-metadata
$ set -x PYTHONPATH .:$PYTHONPATH # I'm using Fish
$ trial jsonschema/tests/test_cli.py [env] GUIX
jsonschema.tests.test_cli
TestCLI
test_draft3_schema_draft4_validator ... [OK]
test_successful_validation ... [OK]
test_unsuccessful_validation ... [OK]
test_unsuccessful_validation_multiple_instances ... [OK]
test_version ... [ERROR]
===============================================================================
[ERROR]
Traceback (most recent call last):
[…]
subprocess.CalledProcessError: Command '['/gnu/store/m4kgg8z52kn6xspmd3brvivd129d4i3s-python-wrapper-3.7.4/bin/python', '-m', 'jsonschema', '--version']' returned non-zero exit status 2.
```
But the exit status was… 2! Which I didn't pay attention at first.
Then, I modified the code of the test to add some log:
```
$ trial jsonschema/tests/test_cli.py [env] GUIX
jsonschema.tests.test_cli
TestCLI
test_draft3_schema_draft4_validator ... [OK]
test_successful_validation ... [OK]
test_unsuccessful_validation ... [OK]
test_unsuccessful_validation_multiple_instances ... [OK]
test_version ... usage: __main__.py [-h] [-i INSTANCES] [-F ERROR_FORMAT] [-V VALIDATOR] schema
__main__.py: error: the following arguments are required: schema
[ERROR]
```
This error message in the output is the one I would get by calling
`python -m jsonschema` without `--version`.
This was the end of my investigation and… a mistake! ^_^'
Right after your question, I tried to do the same but directly from my package
definition by adding a `substitute`:
```
(add-before 'check 'replace-check-output
(lambda _
(substitute* "jsonschema/tests/test_cli.py"
(("check_output") "check_call"))
#t))
```
Re-built the package and… got the exit status 1 with a different error message
in the log:
```
jsonschema.tests.test_cli
TestCLI
test_draft3_schema_draft4_validator ... [OK]
test_successful_validation ... [OK]
test_unsuccessful_validation ... [OK]
test_unsuccessful_validation_multiple_instances ... [OK]
test_version ... /gnu/store/l8nphg0idd8pfddyad8f92lx8d1hc053-python-wrapper-3.7.4/bin/python: No module named jsonschema
[ERROR]
===============================================================================
[ERROR]
Traceback (most recent call last):
[…]
subprocess.CalledProcessError: Command '['/gnu/store/l8nphg0idd8pfddyad8f92lx8d1hc053-python-wrapper-3.7.4/bin/python', '-m', 'jsonschema', '--version']' returned non-zero exit status 1.
```
So, I'm left with the same question:
- why does it fail with `No module named jsonschema`? and with a second one
- why wasn't I able to reproduce the error from the build directory?!
I still have a lot to learn! :-(
Any help welcome!
--
Tanguy
next prev parent reply other threads:[~2020-03-13 8:46 UTC|newest]
Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-25 8:28 [bug#39777] [PATCH 0/5] gnu: poetry: Update to 1.0.3 Tanguy Le Carrour
2020-02-25 8:33 ` [bug#39777] [PATCH 1/5] gnu: python-pexpect: Update to 4.8.0 Tanguy Le Carrour
2020-02-25 8:33 ` [bug#39777] [PATCH 2/5] gnu: python-jsonschema: Update to 3.2.0 Tanguy Le Carrour
2020-02-25 8:33 ` [bug#39777] [PATCH 3/5] gnu: python-fakeredis: Update to 1.2.1 Tanguy Le Carrour
2020-02-25 8:33 ` [bug#39777] [PATCH 4/5] gnu: python-cachy: Update to 0.3.0 Tanguy Le Carrour
2020-02-25 8:33 ` [bug#39777] [PATCH 5/5] gnu: poetry: Update to 1.0.3 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 01/10] gnu: python-pexpect: Update to 4.8.0 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 02/10] gnu: python-cachy: Update to 0.3.0 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 03/10] gnu: python-fakeredis: Update to 1.2.1 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 04/10] gnu: python-jsonschema: Update to 3.2.0 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 05/10] gnu: python-pastel: Update to 0.2.0 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 06/10] gnu: python-msgpack: Update to 1.0.0 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 07/10] gnu: python-cachecontrol: Add missing propagated-input Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 08/10] gnu: python-tomlkit: Update to 0.5.11 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 09/10] gnu: python-clikit: Update to 0.4.2 Tanguy Le Carrour
2020-03-08 14:10 ` [bug#39777] [PATCH V2 10/10] gnu: poetry: Update to 1.0.5 Tanguy Le Carrour
2020-03-08 14:13 ` [bug#39777] [PATCH V2 0/10] " Tanguy Le Carrour
2020-03-09 8:16 ` [bug#39777] [PATCH V3 0/11] " Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 01/11] gnu: python-pexpect: Update to 4.8.0 Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 02/11] gnu: python-cachy: Update to 0.3.0 Tanguy Le Carrour
2020-03-11 19:14 ` Leo Famulari
2020-03-12 8:58 ` Tanguy Le Carrour
2020-03-14 17:30 ` Leo Famulari
2020-03-14 18:37 ` Tanguy Le Carrour
2020-03-14 17:39 ` Leo Famulari
2020-03-14 18:38 ` Tanguy Le Carrour
2020-03-19 17:27 ` Tanguy Le Carrour
2020-03-20 19:16 ` Leo Famulari
2020-03-09 8:18 ` [bug#39777] [PATCH V3 03/11] gnu: python-fakeredis: Update to 1.2.1 Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 04/11] gnu: python-jsonschema: Update to 3.2.0 Tanguy Le Carrour
2020-03-11 19:15 ` Leo Famulari
2020-03-12 8:48 ` Tanguy Le Carrour
2020-03-12 17:44 ` Leo Famulari
2020-03-13 8:45 ` Tanguy Le Carrour [this message]
2020-03-14 17:33 ` Leo Famulari
2020-03-14 17:36 ` Leo Famulari
2020-03-14 18:40 ` Tanguy Le Carrour
2020-03-19 7:42 ` Tanguy Le Carrour
2020-03-20 19:15 ` Leo Famulari
2020-03-21 8:50 ` Tanguy Le Carrour
2020-03-22 22:02 ` Leo Famulari
2020-03-23 7:26 ` Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 05/11] gnu: python-pastel: Update to 0.2.0 Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 06/11] gnu: python-msgpack: Update to 1.0.0 Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 07/11] gnu: python-cachecontrol: Add missing propagated-input Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 08/11] gnu: python-sortedcontainers: Add missing propagated-inputs Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 09/11] gnu: python-tomlkit: Update to 0.5.11 Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 10/11] gnu: python-clikit: Update to 0.4.2 Tanguy Le Carrour
2020-03-09 8:18 ` [bug#39777] [PATCH V3 11/11] gnu: poetry: Update to 1.0.5 Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 01/10] gnu: python-pexpect: Update to 4.8.0 Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 02/10] gnu: python-jsonschema: Update to 3.2.0 Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 03/10] gnu: python-pastel: Update to 0.2.0 Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 04/10] gnu: python-msgpack: Update to 1.0.0 Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 05/10] gnu: python-cachecontrol: Add missing propagated-input Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 06/10] gnu: python-sortedcontainers: Add missing propagated-inputs Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 07/10] gnu: python-clikit: Update to 0.4.2 Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 08/10] gnu: python-fakeredis: Update to 1.2.1 Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 09/10] gnu: python-cachy: Update to 0.3.0 Tanguy Le Carrour
2020-03-19 17:30 ` [bug#39777] [PATCH V4 10/10] gnu: poetry: Update to 1.0.5 Tanguy Le Carrour
2020-03-21 11:25 ` [bug#39777] [PATCH V5 0/14] " Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 01/14] gnu: python-pexpect: Update to 4.8.0 Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 02/14] gnu: python-jsonschema: Update to 3.2.0 Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 03/14] gnu: python-pastel: Update to 0.2.0 Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 04/14] gnu: python-msgpack: Update to 1.0.0 Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 05/14] gnu: python-cachecontrol: Add missing propagated-input Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 06/14] gnu: python-sortedcontainers: Add missing propagated-inputs Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 07/14] gnu: python-clikit: Update to 0.4.2 Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 08/14] gnu: python-fakeredis: Update to 1.2.1 Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 09/14] gnu: Add python-xattr Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 10/14] gnu: Add python-json-logger Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 11/14] gnu: Add python-daiquiri Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 12/14] gnu: Add python-pifpaf Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 13/14] gnu: python-cachy: Update to 0.3.0 Tanguy Le Carrour
2020-03-21 11:26 ` [bug#39777] [PATCH V5 14/14] gnu: poetry: Update to 1.0.5 Tanguy Le Carrour
2020-03-21 14:26 ` [bug#39777] [PATCH V6 0/X] " Tanguy Le Carrour
2020-03-24 21:19 ` bug#39777: " Leo Famulari
2020-03-21 14:37 ` [bug#39777] [PATCH V6] " Tanguy Le Carrour
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200313084541.zeditu3oaouqm2du@rafflesia \
--to=tanguy@bioneland.org \
--cc=39777@debbugs.gnu.org \
--cc=leo@famulari.name \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).