guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/3] gnu: Add python-screed.


From: Ben Woodcroft
Subject: Re: [PATCH 1/3] gnu: Add python-screed.
Date: Sat, 18 Jun 2016 08:57:38 +1000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0



On 17/06/16 12:07, Leo Famulari wrote:
On Fri, Jun 17, 2016 at 11:23:19AM +1000, Ben Woodcroft wrote:
* gnu/packages/bioinformatics.scm (python-screed, python2-screed):
New variables.
+       (uri (string-append
+             "https://pypi.python.org/packages/a0/6b/";
+             "a90c7ab7b0ad1eab211053c85c1242a81379f34c5f5933392079c9b36858"
Can you keep the hash on a single line? Just to reduce noise in the
commit log for future updates...

OK.


+    (synopsis "Short read sequence utils")
s/utils/utilities

"database utilities" would be even better I think.


+    (description "Screed parses FASTA and FASTQ files and generates databases.
Can you unpack these acronyms?

Acronyms are no fun in descriptions I agree. But the problem here is that FASTA and FASTQ aren't really acronyms, and besides these terms are extremely well known (like 'tar' or 'zip').

On a side note, FASTA was first used by the "FASTA" program circa 1985 (just like I am). And just like me it is still going - last release was April. I might have a go at packaging it.

ben



reply via email to

[Prev in Thread] Current Thread [Next in Thread]