Re: [PATCH v2 03/10] gnu: Add python-pytest-pep8.

2016-04-25 Thread Leo Famulari
On Mon, Apr 25, 2016 at 10:36:54PM +1000, Ben Woodcroft wrote: > On 25/04/16 18:43, Ricardo Wurmus wrote: > >Ben Woodcroft writes: > > > +(propagated-inputs > + `(("python-pytest-cache", python-pytest-cache) > + ("python-pytest" ,python-pytest) > + ("python-pep8

Re: [PATCH v2 03/10] gnu: Add python-pytest-pep8.

2016-04-25 Thread Ben Woodcroft
On 25/04/16 18:43, Ricardo Wurmus wrote: Ben Woodcroft writes: +(propagated-inputs + `(("python-pytest-cache", python-pytest-cache) + ("python-pytest" ,python-pytest) + ("python-pep8" ,python-pep8))) Are you sure all of these need to be propagated? That's what the install

Re: [PATCH v2 03/10] gnu: Add python-pytest-pep8.

2016-04-25 Thread Ricardo Wurmus
Ben Woodcroft writes: >>> +(propagated-inputs >>> + `(("python-pytest-cache", python-pytest-cache) >>> + ("python-pytest" ,python-pytest) >>> + ("python-pep8" ,python-pep8))) >> Are you sure all of these need to be propagated? > > That's what the install_requires in setup.py

Re: [PATCH v2 03/10] gnu: Add python-pytest-pep8.

2016-04-24 Thread Ben Woodcroft
On 25/04/16 03:04, Efraim Flashner wrote: On Sun, Apr 24, 2016 at 11:06:17PM +1000, Ben Woodcroft wrote: [..] + ;; 'setup.py test' does not run tests + (lambda* (#:key inputs outputs #:allow-other-keys) + (let ((out (assoc-ref outputs "out"))) + (

Re: [PATCH v2 03/10] gnu: Add python-pytest-pep8.

2016-04-24 Thread Ben Woodcroft
On 25/04/16 03:04, Efraim Flashner wrote: On Sun, Apr 24, 2016 at 11:06:17PM +1000, Ben Woodcroft wrote: [..] + (let ((out (assoc-ref outputs "out"))) + (setenv "PYTHONPATH" + (string-append +(getenv "PYTHONPATH") +

Re: [PATCH v2 03/10] gnu: Add python-pytest-pep8.

2016-04-24 Thread Efraim Flashner
On Sun, Apr 24, 2016 at 11:06:17PM +1000, Ben Woodcroft wrote: > * gnu/packages/python.scm (python-pytest-pep8, python2-pytest-pep8): New > variables. > --- > gnu/packages/python.scm | 60 > + > 1 file changed, 60 insertions(+) > > diff --git a/gnu

[PATCH v2 03/10] gnu: Add python-pytest-pep8.

2016-04-24 Thread Ben Woodcroft
* gnu/packages/python.scm (python-pytest-pep8, python2-pytest-pep8): New variables. --- gnu/packages/python.scm | 60 + 1 file changed, 60 insertions(+) diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm index c07dae6..b1a4969 100644 ---