all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH]: gnu: Add RSeQC.
@ 2015-02-18 14:12 Ricardo Wurmus
  2015-02-18 17:43 ` Ricardo Wurmus
  0 siblings, 1 reply; 5+ messages in thread
From: Ricardo Wurmus @ 2015-02-18 14:12 UTC (permalink / raw)
  To: Guix-devel

[-- Attachment #1: Type: text/plain, Size: 673 bytes --]

Attached is a patch to add RSeQC to the bioinformatics module.  It only
works with Python 2.7.  I removed the bundled copy of pysam to be able
to use our python-pysam package instead.

The license isn't very clear.  The Google Code website[1] lists the
license as "GPLv3", but nowhere in the source is the license version
specified.  "PKG-INFO" declares "License :: GPL", whereas
"./doc/COPYING" contains the MIT license.  It's all pretty messy, so I
contacted one of the authors to ask for clarification.

Apart from the license issues (which need to be solved before adding
RSeQC), does this patch look okay?

~~ Ricardo

___________
[1]: http://code.google.com/p/rseqc


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-RSeQC.patch --]
[-- Type: text/x-patch, Size: 2991 bytes --]

From e99a3cdb587afcf874495519c4dcf380295bd6f1 Mon Sep 17 00:00:00 2001
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Date: Wed, 18 Feb 2015 15:00:04 +0100
Subject: [PATCH] gnu: Add RSeQC.

* gnu/packages/bioinformatics.scm (rseqc): New variable.
---
 gnu/packages/bioinformatics.scm | 47 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 47 insertions(+)

diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
index 5bd668e..cf7e674 100644
--- a/gnu/packages/bioinformatics.scm
+++ b/gnu/packages/bioinformatics.scm
@@ -22,6 +22,7 @@
   #:use-module (guix download)
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system cmake)
+  #:use-module (guix build-system python)
   #:use-module (guix build-system trivial)
   #:use-module (gnu packages)
   #:use-module (gnu packages base)
@@ -310,6 +311,52 @@ several alignment strategies enable effective alignment of RNA-seq reads, in
 particular, reads spanning multiple exons.")
     (license license:gpl3+)))
 
+(define-public rseqc
+  (package
+    (name "rseqc")
+    (version "2.6.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri
+        (string-append "mirror://sourceforge/rseqc/"
+                       version "/RSeQC-" version ".tar.gz"))
+       (sha256
+        (base32 "09rf0x9d6apjja5l01cgprj7vigpw6kiqhy34ibwwlxil0db0ri4"))
+       (modules '((guix build utils)))
+       (snippet
+        '(begin
+           ;; remove bundled copy of pysam
+           (delete-file-recursively "lib/pysam")
+           (substitute* "setup.py"
+             ;; remove dependency on outdated "distribute" module
+             (("^from distribute_setup import use_setuptools") "")
+             (("^use_setuptools\\(\\)") "")
+             ;; do not use bundled copy of pysam
+             (("^have_pysam = False") "have_pysam = True"))))))
+    (build-system python-build-system)
+    (arguments `(#:python ,python-2))
+    (inputs
+     `(("python-cython" ,python2-cython)
+       ("python-pysam" ,python2-pysam)
+       ("python-numpy" ,python2-numpy)
+       ("python-setuptools" ,python2-setuptools)
+       ("zlib" ,zlib)))
+    (native-inputs
+     `(("python-nose" ,python2-nose)))
+    (home-page "http://rseqc.sourceforge.net/")
+    (synopsis "RNA-seq quality control package")
+    (description
+     "RSeQC provides a number of modules that can comprehensively evaluate
+high throughput sequence data, especially RNA-seq data.  Some basic modules
+inspect sequence quality, nucleotide composition bias, PCR bias and GC bias,
+while RNA-seq specific modules evaluate sequencing saturation, mapped reads
+distribution, coverage uniformity, strand specificity, etc.")
+    ;; The license version is only mentioned on the Google Code website; there
+    ;; are no license headers.  I have contacted one of the authors to ask for
+    ;; clarification.
+    (license license:gpl3)))
+
 (define-public samtools
   (package
     (name "samtools")
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH]: gnu: Add RSeQC.
  2015-02-18 14:12 [PATCH]: gnu: Add RSeQC Ricardo Wurmus
@ 2015-02-18 17:43 ` Ricardo Wurmus
  2015-02-19 10:29   ` Ricardo Wurmus
  0 siblings, 1 reply; 5+ messages in thread
From: Ricardo Wurmus @ 2015-02-18 17:43 UTC (permalink / raw)
  To: Guix-devel


Ricardo Wurmus writes:

> The license isn't very clear.  The Google Code website[1] lists the
> license as "GPLv3", but nowhere in the source is the license version
> specified.  "PKG-INFO" declares "License :: GPL", whereas
> "./doc/COPYING" contains the MIT license.  It's all pretty messy, so I
> contacted one of the authors to ask for clarification.

The author has told me that the license is GPLv3; the website will be
updated accordingly.  I will change the license field as soon as the
change is publicly visible.

~~ Ricardo

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH]: gnu: Add RSeQC.
  2015-02-18 17:43 ` Ricardo Wurmus
@ 2015-02-19 10:29   ` Ricardo Wurmus
  2015-02-19 11:51     ` Andreas Enge
  0 siblings, 1 reply; 5+ messages in thread
From: Ricardo Wurmus @ 2015-02-19 10:29 UTC (permalink / raw)
  To: Guix-devel

[-- Attachment #1: Type: text/plain, Size: 338 bytes --]


Ricardo Wurmus writes:

> The author has told me that the license is GPLv3; the website will be
> updated accordingly.  I will change the license field as soon as the
> change is publicly visible.

The license is now officially declared to be GPLv3:

    http://rseqc.sourceforge.net/#license

Attached is an updated patch.

~~ Ricardo


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-RSeQC.patch --]
[-- Type: text/x-patch, Size: 2808 bytes --]

From 88a5924c36ccd5317de0307ee4aa9b959ffeb9bd Mon Sep 17 00:00:00 2001
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Date: Wed, 18 Feb 2015 15:00:04 +0100
Subject: [PATCH] gnu: Add RSeQC.

* gnu/packages/bioinformatics.scm (rseqc): New variable.
---
 gnu/packages/bioinformatics.scm | 44 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 44 insertions(+)

diff --git a/gnu/packages/bioinformatics.scm b/gnu/packages/bioinformatics.scm
index 5bd668e..905246a 100644
--- a/gnu/packages/bioinformatics.scm
+++ b/gnu/packages/bioinformatics.scm
@@ -22,6 +22,7 @@
   #:use-module (guix download)
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system cmake)
+  #:use-module (guix build-system python)
   #:use-module (guix build-system trivial)
   #:use-module (gnu packages)
   #:use-module (gnu packages base)
@@ -310,6 +311,49 @@ several alignment strategies enable effective alignment of RNA-seq reads, in
 particular, reads spanning multiple exons.")
     (license license:gpl3+)))
 
+(define-public rseqc
+  (package
+    (name "rseqc")
+    (version "2.6.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri
+        (string-append "mirror://sourceforge/rseqc/"
+                       version "/RSeQC-" version ".tar.gz"))
+       (sha256
+        (base32 "09rf0x9d6apjja5l01cgprj7vigpw6kiqhy34ibwwlxil0db0ri4"))
+       (modules '((guix build utils)))
+       (snippet
+        '(begin
+           ;; remove bundled copy of pysam
+           (delete-file-recursively "lib/pysam")
+           (substitute* "setup.py"
+             ;; remove dependency on outdated "distribute" module
+             (("^from distribute_setup import use_setuptools") "")
+             (("^use_setuptools\\(\\)") "")
+             ;; do not use bundled copy of pysam
+             (("^have_pysam = False") "have_pysam = True"))))))
+    (build-system python-build-system)
+    (arguments `(#:python ,python-2))
+    (inputs
+     `(("python-cython" ,python2-cython)
+       ("python-pysam" ,python2-pysam)
+       ("python-numpy" ,python2-numpy)
+       ("python-setuptools" ,python2-setuptools)
+       ("zlib" ,zlib)))
+    (native-inputs
+     `(("python-nose" ,python2-nose)))
+    (home-page "http://rseqc.sourceforge.net/")
+    (synopsis "RNA-seq quality control package")
+    (description
+     "RSeQC provides a number of modules that can comprehensively evaluate
+high throughput sequence data, especially RNA-seq data.  Some basic modules
+inspect sequence quality, nucleotide composition bias, PCR bias and GC bias,
+while RNA-seq specific modules evaluate sequencing saturation, mapped reads
+distribution, coverage uniformity, strand specificity, etc.")
+    (license license:gpl3)))
+
 (define-public samtools
   (package
     (name "samtools")
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH]: gnu: Add RSeQC.
  2015-02-19 10:29   ` Ricardo Wurmus
@ 2015-02-19 11:51     ` Andreas Enge
  2015-02-19 12:37       ` Ricardo Wurmus
  0 siblings, 1 reply; 5+ messages in thread
From: Andreas Enge @ 2015-02-19 11:51 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: Guix-devel

On Thu, Feb 19, 2015 at 11:29:47AM +0100, Ricardo Wurmus wrote:
> +    (license license:gpl3)))

From the text printed on the license page you give, it should be gpl3+.
Otherwise it looks good, pleas push!

Andreas

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH]: gnu: Add RSeQC.
  2015-02-19 11:51     ` Andreas Enge
@ 2015-02-19 12:37       ` Ricardo Wurmus
  0 siblings, 0 replies; 5+ messages in thread
From: Ricardo Wurmus @ 2015-02-19 12:37 UTC (permalink / raw)
  To: Andreas Enge; +Cc: Guix-devel


Andreas Enge writes:

> On Thu, Feb 19, 2015 at 11:29:47AM +0100, Ricardo Wurmus wrote:
>> +    (license license:gpl3)))
>
> From the text printed on the license page you give, it should be gpl3+.
> Otherwise it looks good, pleas push!

You are of course right!  Will push with that change.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2015-02-19 12:38 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-02-18 14:12 [PATCH]: gnu: Add RSeQC Ricardo Wurmus
2015-02-18 17:43 ` Ricardo Wurmus
2015-02-19 10:29   ` Ricardo Wurmus
2015-02-19 11:51     ` Andreas Enge
2015-02-19 12:37       ` Ricardo Wurmus

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.