[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 10/10] gnu: Add python-scikit-bio.
From: |
Leo Famulari |
Subject: |
Re: [PATCH v2 10/10] gnu: Add python-scikit-bio. |
Date: |
Mon, 25 Apr 2016 14:47:04 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Sun, Apr 24, 2016 at 11:06:24PM +1000, Ben Woodcroft wrote:
> * gnu/packages/python.scm (python-scikit-bio, python2-scikit-bio):
> New variables.
Thanks!
> + (arguments
> + `(#:phases
> + (modify-phases %standard-phases
> + ;; Install procedure installs extraneous binaries.
> + (add-after 'install 'remove-extraneous-files
> + (lambda* (#:key outputs #:allow-other-keys)
> + (let* ((out (assoc-ref outputs "out"))
> + (bin (string-append out "/bin")))
> + (delete-file-recursively bin))
> + #t))
That's interesting. How are they extraneous? Would nobody want to use
them?
> + ;; Tests are intended to be run using 'make test' not 'setup.py
> + ;; test', but there are many test failures anyway
> + ;; e.g. https://github.com/biocore/scikit-bio/issues/1308. Run a
> + ;; simple import test instead.
> + (delete 'check)
> + (add-after 'install 'check-after-install
Are the tests broken, or does scikit-bio simply not work with the
current version of pandas?
More context:
https://github.com/biocore/scikit-bio/pull/1325
> +(define-public python2-scikit-bio
> + (let ((base (package-with-python2
> + (strip-python2-variant python-scikit-bio))))
> + (package
> + (inherit base)
> + (native-inputs
> + (append (package-native-inputs base)
> + `(("python2-setuptools" ,python2-setuptools)
> + ("python2-mock" ,python2-mock)))))))
Same suggestion as for python-natsort.
- Re: [PATCH v2 05/10] gnu: Add python-future., (continued)
- [PATCH v2 06/10] gnu: Add python-cachecontrol., Ben Woodcroft, 2016/04/24
- [PATCH v2 07/10] gnu: python-tornado: Use 'python2-variant'., Ben Woodcroft, 2016/04/24
- [PATCH v2 08/10] gnu: python-ipython: Use 'python2-variant'., Ben Woodcroft, 2016/04/24
- [PATCH v2 09/10] gnu: Add python-natsort., Ben Woodcroft, 2016/04/24
- [PATCH v2 10/10] gnu: Add python-scikit-bio., Ben Woodcroft, 2016/04/24
- Re: [PATCH v2 10/10] gnu: Add python-scikit-bio.,
Leo Famulari <=