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")
+ ":"
+ out
+ "/lib/python"
some of the previous lines should be combined into one line
I think this is a matter of style, but I've amended it to below, which I
assume is OK - I have no strong opinions.
+ (setenv
+ "PYTHONPATH"
+ (string-append (getenv "PYTHONPATH")":" out "/lib/python"
+ (string-take (string-take-right
+ (assoc-ref inputs "python") 5) 3)
+ "/site-packages"))
+ (string-take (string-take-right
+ (assoc-ref inputs "python") 5) 3)
+ "/site-packages"))
+ (zero?
+ (system*
+ (string-append out "/bin/py.test") "test_pep8.py"))))))))
+ (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 is telling me.
Thanks,
ben
