guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] Add MAFFT.


From: Ben Woodcroft
Subject: Re: [PATCH] Add MAFFT.
Date: Sat, 12 Sep 2015 22:20:11 +1000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0

On 07/09/15 18:55, Andreas Enge wrote:
On Tue, Aug 25, 2015 at 10:58:42PM +0200, Ludovic Courtès wrote:
It seems this message was left unanswered.  Andreas?
I had a cursory look at the patch, here are some comments:

[..]
+protein sequences.  For instance, it offers L-INS-i (accurate; for alignment
+of <???200 sequences) and FFT-NS-2 (fast; for alignment of <???30,000
Is this an artefact of the mailer?

I did not check whether the package passes "guix lint".

Could you send an updated patch?
Attached - I was only able to remove some of the long string-append lines. Apologies for that second "patch" - not intentional. Thanks for the review. I've changed the character to a different "~" so hopefully more standard now. Guix lint doesn't complain.

ben

Attachment: 0001-gnu-Add-mafft.patch
Description: Text Data


reply via email to

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