guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#27558] [PATCH] gnu: Add python-ruamel.yaml


From: Marius Bakke
Subject: [bug#27558] [PATCH] gnu: Add python-ruamel.yaml
Date: Tue, 11 Jul 2017 21:21:38 +0200
User-agent: Notmuch/0.24.2 (https://notmuchmail.org) Emacs/25.2.1 (x86_64-unknown-linux-gnu)

Muriithi Frederick Muriuki <address@hidden> writes:

> * gnu/packages/python.scm (python-ruamel.yaml, python2-ruamel.yaml):
>   New variables.

[...]

> +(define-public python-ruamel.yaml
> +  (package
> +    (name "python-ruamel.yaml")
> +    (version "0.15.16")

0.15.18 is out already, sorry for the slow review! Can you try updating
to it?

> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (pypi-uri "ruamel.yaml" version))
> +       (sha256
> +        (base32
> +         "0yx1a1yqgm1phrmyvqzag5jwivn25ndlnwh2mgwgr7m8skn2h045"))))
> +    (build-system python-build-system)
> +    (arguments
> +     `(;; Tests fail with message:
> +       ;; ImportError: Failed to import test module: .ruamel
> +       ;; but the directory .ruamel exists in the environment
> +       ;; left when the package is built with --keep-failed
> +       #:tests? #f))

'python setup.py test' does not work for all packages and may return
bogus errors like these. Often you can find hints for how upstream runs
their tests in "tox.ini", or CI configuration files like ".travis.yml".

tox.ini in this case runs "bash -c 'pytest _test/test_*.py'". Can you
try replacing the check phase with a phase that invokes this command?

Apart from that looks good!

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]