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

[Marius Bakke <mbakke@fastmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1267 bytes --]

Muriithi Frederick Muriuki <fredmanglis@gmail.com> 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 #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]