From: Leo Famulari <leo@famulari.name>
To: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH] Update deeptools.
Date: Wed, 2 Mar 2016 20:50:05 -0500 [thread overview]
Message-ID: <20160303015005.GA28589@jasmine> (raw)
In-Reply-To: <idjegbv1vdz.fsf@bimsb-sys02.mdc-berlin.net>
On Mon, Feb 29, 2016 at 02:47:04PM +0100, Ricardo Wurmus wrote:
> Hi Guix,
>
> the goal of this series of patches is to update deeptools.
>
> First I moved the “*-pysam” packages from python.scm to
> bioinformatics.scm, then updated them. Then added “pybigwig”, cleaned
> up “deeptools” (used “propagated-inputs” for no good reason), and
> finally updated the package.
All look good, although can you say if pysam needs setuptools at
run-time? Or does it make sense to change it to a native-input?
>
> ~~ Ricardo
>
> From f960fda0d38d0a71519072ad8fae8edd7b90b2c6 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
> Date: Thu, 25 Feb 2016 14:01:14 +0100
> Subject: [PATCH 1/5] gnu: python-pysam, python2-pysam: Move to
> bioinformatics.scm.
>
> * gnu/packages/python.scm (python-pysam, python2-pysam): Move to...
> * gnu/packages/bioinformatics.scm (python-pysam, python2-pysam):
> ...here.
> ---
> gnu/packages/bioinformatics.scm | 38 ++++++++++++++++++++++++++++++++++++++
> gnu/packages/python.scm | 38 --------------------------------------
> 2 files changed, 38 insertions(+), 38 deletions(-)
>
> diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
> index e570487..a86f7d6 100644
> --- a/gnu/packages/bioinformatics.scm
> +++ b/gnu/packages/bioinformatics.scm
> @@ -808,6 +808,44 @@ and more accurate. BWA-MEM also has better performance than BWA-backtrack for
> multiple sequence alignments.")
> (license license:expat)))
>
> +(define-public python-pysam
> + (package
> + (name "python-pysam")
> + (version "0.8.1")
> + (source
> + (origin
> + (method url-fetch)
> + (uri (string-append "https://pypi.python.org/packages/source/p/pysam/pysam-"
> + version ".tar.gz"))
> + (sha256
> + (base32
> + "1fb6i6hbpzxaxb62kyyp5alaidwhj40f7c6gwbhr6njzlqd5l459"))))
> + (build-system python-build-system)
> + (arguments
> + `(#:tests? #f ; tests are excluded in the manifest
> + #:phases
> + (alist-cons-before
> + 'build 'set-flags
> + (lambda _
> + (setenv "LDFLAGS" "-lncurses")
> + (setenv "CFLAGS" "-D_CURSES_LIB=1"))
> + %standard-phases)))
> + (inputs
> + `(("python-cython" ,python-cython)
> + ("python-setuptools" ,python-setuptools)
> + ("ncurses" ,ncurses)
> + ("zlib" ,zlib)))
> + (home-page "https://github.com/pysam-developers/pysam")
> + (synopsis "Python bindings to the SAMtools C API")
> + (description
> + "Pysam is a Python module for reading and manipulating files in the
> +SAM/BAM format. Pysam is a lightweight wrapper of the SAMtools C API. It
> +also includes an interface for tabix.")
> + (license license:expat)))
> +
> +(define-public python2-pysam
> + (package-with-python2 python-pysam))
> +
> (define-public clipper
> (package
> (name "clipper")
> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> index 46eda76..214a8ea 100644
> --- a/gnu/packages/python.scm
> +++ b/gnu/packages/python.scm
> @@ -1039,44 +1039,6 @@ get the local timezone information, unless you know the zoneinfo name, and
> under several distributions that's hard or impossible to figure out.")
> (license cc0)))
>
> -(define-public python-pysam
> - (package
> - (name "python-pysam")
> - (version "0.8.1")
> - (source
> - (origin
> - (method url-fetch)
> - (uri (string-append "https://pypi.python.org/packages/source/p/pysam/pysam-"
> - version ".tar.gz"))
> - (sha256
> - (base32
> - "1fb6i6hbpzxaxb62kyyp5alaidwhj40f7c6gwbhr6njzlqd5l459"))))
> - (build-system python-build-system)
> - (arguments
> - `(#:tests? #f ; tests are excluded in the manifest
> - #:phases
> - (alist-cons-before
> - 'build 'set-flags
> - (lambda _
> - (setenv "LDFLAGS" "-lncurses")
> - (setenv "CFLAGS" "-D_CURSES_LIB=1"))
> - %standard-phases)))
> - (inputs
> - `(("python-cython" ,python-cython)
> - ("python-setuptools" ,python-setuptools)
> - ("ncurses" ,ncurses)
> - ("zlib" ,zlib)))
> - (home-page "https://github.com/pysam-developers/pysam")
> - (synopsis "Python bindings to the SAMtools C API")
> - (description
> - "Pysam is a Python module for reading and manipulating files in the
> -SAM/BAM format. Pysam is a lightweight wrapper of the SAMtools C API. It
> -also includes an interface for tabix.")
> - (license license:expat)))
> -
> -(define-public python2-pysam
> - (package-with-python2 python-pysam))
> -
> (define-public python2-pysqlite
> (package
> (name "python2-pysqlite")
> --
> 2.1.0
>
> From 473a6f8b1e56984b308533a9baa44979c6450e7d Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
> Date: Mon, 29 Feb 2016 14:36:25 +0100
> Subject: [PATCH 2/5] gnu: python-pysam: Update to 0.8.4.
>
> * gnu/packages/bioinformatics.scm (python-pysam): Update to 0.8.4.
> ---
> gnu/packages/bioinformatics.scm | 16 +++++++---------
> 1 file changed, 7 insertions(+), 9 deletions(-)
>
> diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
> index a86f7d6..28754dc 100644
> --- a/gnu/packages/bioinformatics.scm
> +++ b/gnu/packages/bioinformatics.scm
> @@ -811,15 +811,13 @@ multiple sequence alignments.")
> (define-public python-pysam
> (package
> (name "python-pysam")
> - (version "0.8.1")
> - (source
> - (origin
> - (method url-fetch)
> - (uri (string-append "https://pypi.python.org/packages/source/p/pysam/pysam-"
> - version ".tar.gz"))
> - (sha256
> - (base32
> - "1fb6i6hbpzxaxb62kyyp5alaidwhj40f7c6gwbhr6njzlqd5l459"))))
> + (version "0.8.4")
> + (source (origin
> + (method url-fetch)
> + (uri (pypi-uri "pysam" version))
> + (sha256
> + (base32
> + "1slx5mb94mzm5qzk52q270sab0sar95j67w1g1k452nz3s9j7krh"))))
> (build-system python-build-system)
> (arguments
> `(#:tests? #f ; tests are excluded in the manifest
> --
> 2.1.0
>
> From 1d58bb0aa9894295f6f949d463bcdfa54b4d2980 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
> Date: Mon, 29 Feb 2016 14:37:38 +0100
> Subject: [PATCH 3/5] gnu: Add python-pybigwig.
>
> * gnu/packages/bioinformatics.scm (python-pybigwig, python2-pybigwig):
> New variables.
> ---
> gnu/packages/bioinformatics.scm | 41 +++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 41 insertions(+)
>
> diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
> index 28754dc..d7aae6e 100644
> --- a/gnu/packages/bioinformatics.scm
> +++ b/gnu/packages/bioinformatics.scm
> @@ -1164,6 +1164,47 @@ other types of unwanted sequence from high-throughput sequencing reads.")
> files.")
> (license license:expat)))
>
> +(define-public python-pybigwig
> + (package
> + (name "python-pybigwig")
> + (version "0.2.5")
> + (source (origin
> + (method url-fetch)
> + (uri (pypi-uri "pyBigWig" version))
> + (sha256
> + (base32
> + "0yrpdxg3y0sny25x4w22lv1k47jzccqjmg7j4bp0hywklvp0hg7d"))
> + (modules '((guix build utils)))
> + (snippet
> + '(begin
> + ;; Delete bundled libBigWig sources
> + (delete-file-recursively "libBigWig")))))
> + (build-system python-build-system)
> + (arguments
> + `(#:phases
> + (modify-phases %standard-phases
> + (add-after 'unpack 'link-with-libBigWig
> + (lambda* (#:key inputs #:allow-other-keys)
> + (substitute* "setup.py"
> + (("libs=\\[") "libs=[\"BigWig\", "))
> + #t)))))
> + (inputs
> + `(("libbigwig" ,libbigwig)
> + ("zlib" ,zlib)
> + ("curl" ,curl)))
> + (home-page "https://github.com/dpryan79/pyBigWig")
> + (synopsis "Access bigWig files in Python using libBigWig")
> + (description
> + "This package provides Python bindings to the libBigWig library for
> +accessing bigWig files.")
> + (license license:expat)))
> +
> +(define-public python2-pybigwig
> + (let ((pybigwig (package-with-python2 python-pybigwig)))
> + (package (inherit pybigwig)
> + (native-inputs
> + `(("python-setuptools" ,python2-setuptools))))))
> +
> (define-public deeptools
> (package
> (name "deeptools")
> --
> 2.1.0
>
> From 37837afefee0df1459322b04b1e26257539b8d02 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
> Date: Mon, 29 Feb 2016 14:38:57 +0100
> Subject: [PATCH 4/5] gnu: deeptools: Change "propagated-inputs" to "inputs".
>
> * gnu/packages/bioinformatics.scm (deeptools)[propagated-inputs]: Move
> all inputs from here...
> [inputs]: ...to here.
> ---
> gnu/packages/bioinformatics.scm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
> index d7aae6e..472219e 100644
> --- a/gnu/packages/bioinformatics.scm
> +++ b/gnu/packages/bioinformatics.scm
> @@ -1221,7 +1221,7 @@ accessing bigWig files.")
> (build-system python-build-system)
> (arguments
> `(#:python ,python-2))
> - (propagated-inputs
> + (inputs
> `(("python-scipy" ,python2-scipy)
> ("python-numpy" ,python2-numpy)
> ("python-matplotlib" ,python2-matplotlib)
> --
> 2.1.0
>
> From 197ebd69cc27de3c724d3e01e6c08f843b8b3f07 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
> Date: Mon, 29 Feb 2016 14:40:10 +0100
> Subject: [PATCH 5/5] gnu: deeptools: Update to 2.1.1.
>
> * gnu/packages/bioinformatics.scm (deeptools): Update to 2.1.1.
> [inputs]: Add "python2-numpydoc" and "python2-pybigwig".
> ---
> gnu/packages/bioinformatics.scm | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
> index 472219e..6df7d5c 100644
> --- a/gnu/packages/bioinformatics.scm
> +++ b/gnu/packages/bioinformatics.scm
> @@ -1208,25 +1208,26 @@ accessing bigWig files.")
> (define-public deeptools
> (package
> (name "deeptools")
> - (version "1.5.11")
> + (version "2.1.1")
> (source (origin
> (method url-fetch)
> - (uri (string-append
> - "https://github.com/fidelram/deepTools/archive/"
> - version ".tar.gz"))
> + (uri (string-append "https://github.com/fidelram/deepTools/"
> + "archive/" version ".tar.gz"))
> (file-name (string-append name "-" version ".tar.gz"))
> (sha256
> (base32
> - "1kaagygcbvjs9sxd9cqmskd02wcfp9imvb735r087w7hwqpvz6fs"))))
> + "1nmfin0zjdby3vay3r4flvz94dr6qjhj41ax4yz3vx13j6wz8izd"))))
> (build-system python-build-system)
> (arguments
> `(#:python ,python-2))
> (inputs
> `(("python-scipy" ,python2-scipy)
> ("python-numpy" ,python2-numpy)
> + ("python-numpydoc" ,python2-numpydoc)
> ("python-matplotlib" ,python2-matplotlib)
> ("python-bx-python" ,python2-bx-python)
> - ("python-pysam" ,python2-pysam)))
> + ("python-pysam" ,python2-pysam)
> + ("python-pybigwig" ,python2-pybigwig)))
> (native-inputs
> `(("python-mock" ,python2-mock) ;for tests
> ("python-pytz" ,python2-pytz) ;for tests
> --
> 2.1.0
>
next prev parent reply other threads:[~2016-03-03 1:50 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-29 13:47 [PATCH] Update deeptools Ricardo Wurmus
2016-03-03 1:50 ` Leo Famulari [this message]
2016-03-03 12:32 ` Ricardo Wurmus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160303015005.GA28589@jasmine \
--to=leo@famulari.name \
--cc=guix-devel@gnu.org \
--cc=ricardo.wurmus@mdc-berlin.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).