[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#28151] [PATCH] gnu: Add honcho.
From: |
Marius Bakke |
Subject: |
[bug#28151] [PATCH] gnu: Add honcho. |
Date: |
Tue, 22 Aug 2017 00:00:04 +0200 |
User-agent: |
Notmuch/0.25 (https://notmuchmail.org) Emacs/25.2.1 (x86_64-unknown-linux-gnu) |
Stefan Reichör <address@hidden> writes:
> * gnu/packages/python.scm (python-honcho): New variable.
Thanks! I noticed this runs no tests, can you try to add a phase that
invokes "pytest" instead of relying on the default "setup.py test"?
It looks like it might require some extra (native) inputs, but they
should already be in Guix:
https://github.com/nickstenning/honcho/blob/master/tox.ini#L17
Also, since this seems to be some kind of process manager, perhaps it
would be better suited in 'admin.scm' (without the 'python-' prefix)?
Just a question, anyway. The patch looks good to me (but make sure it
passes `guix lint`).
TIA!
signature.asc
Description: PGP signature