all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] Add bwa.
@ 2015-03-12 11:37 Ricardo Wurmus
  2015-03-14 13:40 ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Ricardo Wurmus @ 2015-03-12 11:37 UTC (permalink / raw)
  To: guix-devel

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

Hi Guix,

here's a patch for an aligner tool:


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

From 6c03af7501c53c06a808906ce7d4fbd431bbf1ba Mon Sep 17 00:00:00 2001
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Date: Thu, 12 Mar 2015 12:35:14 +0100
Subject: [PATCH] gnu: Add bwa.

* gnu/packages/bioinformatics.scm (bwa): 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 22ce71b..67326aa 100644
--- a/gnu/packages/bioinformatics.scm
+++ b/gnu/packages/bioinformatics.scm
@@ -238,6 +238,53 @@ gapped, local, and paired-end alignment modes.")
     (supported-systems '("x86_64-linux"))
     (license license:gpl3+)))
 
+(define-public bwa
+  (package
+    (name "bwa")
+    (version "0.7.12")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "mirror://sourceforge/bio-bwa/bwa-"
+                                  version ".tar.bz2"))
+              (sha256
+               (base32
+                "1330dpqncv0px3pbhjzz1gwgg39kkcv2r9qp2xs0sixf8z8wl7bh"))))
+    (build-system gnu-build-system)
+    (arguments
+     '(#:tests? #f ;no "check" target
+       #:phases
+       (alist-replace
+        'install
+        (lambda* (#:key outputs #:allow-other-keys)
+          (let ((bin (string-append
+                      (assoc-ref outputs "out") "/bin"))
+                (doc (string-append
+                      (assoc-ref outputs "out") "/share/doc/bwa"))
+                (man (string-append
+                      (assoc-ref outputs "out") "/share/man/man1")))
+            (mkdir-p bin)
+            (mkdir-p doc)
+            (mkdir-p man)
+            (copy-file "bwa" (string-append bin "/bwa"))
+            (copy-file "README.md" (string-append doc "/README.md"))
+            (copy-file "bwa.1" (string-append man "/bwa.1"))))
+        ;; no "configure" script
+        (alist-delete 'configure %standard-phases))))
+    (inputs `(("zlib" ,zlib)))
+    (home-page "http://bio-bwa.sourceforge.net/")
+    (synopsis "Burrows-Wheeler aligner")
+    (description
+     "BWA is a software package for mapping low-divergent sequences against a
+large reference genome, such as the human genome.  It consists of three
+algorithms: BWA-backtrack, BWA-SW and BWA-MEM.  The first algorithm is
+designed for Illumina sequence reads up to 100bp, while the rest two for
+longer sequences ranged from 70bp to 1Mbp.  BWA-MEM and BWA-SW share similar
+features such as long-read support and split alignment, but BWA-MEM, which is
+the latest, is generally recommended for high-quality queries as it is faster
+and more accurate.  BWA-MEM also has better performance than BWA-backtrack for
+70-100bp Illumina reads.")
+    (license license:gpl3)))
+
 (define-public clipper
   (package
     (name "clipper")
-- 
2.1.0


[-- Attachment #3: Type: text/plain, Size: 168 bytes --]


The license is really just GPLv3; there are no license headers in the
files that would imply otherwise and the only mention of the GPLv3 is in
the README.

~~ Ricardo

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

* Re: [PATCH] Add bwa.
  2015-03-12 11:37 [PATCH] Add bwa Ricardo Wurmus
@ 2015-03-14 13:40 ` Ludovic Courtès
  2015-03-14 13:52   ` Ricardo Wurmus
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2015-03-14 13:40 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> skribis:

> From 6c03af7501c53c06a808906ce7d4fbd431bbf1ba Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
> Date: Thu, 12 Mar 2015 12:35:14 +0100
> Subject: [PATCH] gnu: Add bwa.
>
> * gnu/packages/bioinformatics.scm (bwa): New variable.

[...]

> +    (synopsis "Burrows-Wheeler aligner")

s/aligner/genome sequence aligner/ or something like that.

> +    (license license:gpl3)))

GPLv3-only?

Other than that LGTM, thanks!

Ludo’.

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

* Re: [PATCH] Add bwa.
  2015-03-14 13:40 ` Ludovic Courtès
@ 2015-03-14 13:52   ` Ricardo Wurmus
  2015-03-14 18:30     ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Ricardo Wurmus @ 2015-03-14 13:52 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel


Ludovic Courtès writes:

>> +    (synopsis "Burrows-Wheeler aligner")
>
> s/aligner/genome sequence aligner/ or something like that.

Is a change to "Burrows-Wheeler sequence aligner" acceptable?  I don't
think "sequence aligner" has any other meaning in this context (and in
combination with "Burrows-Wheeler").

>> +    (license license:gpl3)))
>
> GPLv3-only?

Yes, just GPLv3 as far as I can tell.  The usual upgrade clause is
nowhere to be found in the source tree.

~~ Ricardo

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

* Re: [PATCH] Add bwa.
  2015-03-14 13:52   ` Ricardo Wurmus
@ 2015-03-14 18:30     ` Ludovic Courtès
  0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2015-03-14 18:30 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

Ricardo Wurmus <rekado@elephly.net> skribis:

> Ludovic Courtès writes:
>
>>> +    (synopsis "Burrows-Wheeler aligner")
>>
>> s/aligner/genome sequence aligner/ or something like that.
>
> Is a change to "Burrows-Wheeler sequence aligner" acceptable?  I don't
> think "sequence aligner" has any other meaning in this context (and in
> combination with "Burrows-Wheeler").

Yes, sounds good.

>>> +    (license license:gpl3)))
>>
>> GPLv3-only?
>
> Yes, just GPLv3 as far as I can tell.  The usual upgrade clause is
> nowhere to be found in the source tree.

If there are no file headers or documents specifying the version, then
it’s effectively GPLv3+ (actually it’s even any version, per Section 14,
but we usually do GPLv3+ in this case.)

Thanks,
Ludo’.

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

end of thread, other threads:[~2015-03-14 18:30 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-12 11:37 [PATCH] Add bwa Ricardo Wurmus
2015-03-14 13:40 ` Ludovic Courtès
2015-03-14 13:52   ` Ricardo Wurmus
2015-03-14 18:30     ` Ludovic Courtès

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.