unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
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

  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).