[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 22/22] gnu: Add roary.
From: |
Leo Famulari |
Subject: |
Re: [PATCH 22/22] gnu: Add roary. |
Date: |
Fri, 29 Jul 2016 17:07:46 -0400 |
User-agent: |
Mutt/1.6.0 (2016-04-01) |
On Wed, Jul 27, 2016 at 10:54:23PM +1000, Ben Woodcroft wrote:
> * gnu/packages/bioinformatics.scm (roary): New variable.
Wow, this is a complicated package! Thanks for doing the work!
> + `(#:phases
> + (modify-phases %standard-phases
> + (add-after 'unpack 'delete-bundled-binaries
> + (lambda _
> + (delete-file-recursively "binaries")
> + #t))
The only thing I would suggest you change is to delete these binaries in
an origin snippet. The effect would be that the binaries would not be
provided by `guix build --source roary`.
If the binaries aren't provided with their source code, then I think
this change is required.
- [PATCH 14/22] gnu: Add bpp-core., (continued)
- [PATCH 14/22] gnu: Add bpp-core., Ben Woodcroft, 2016/07/27
- [PATCH 15/22] gnu: Add bpp-seq., Ben Woodcroft, 2016/07/27
- [PATCH 16/22] gnu: Add bpp-phyl., Ben Woodcroft, 2016/07/27
- [PATCH 17/22] gnu: Add bpp-popgen., Ben Woodcroft, 2016/07/27
- [PATCH 18/22] gnu: Add bppsuite., Ben Woodcroft, 2016/07/27
- [PATCH 19/22] gnu: Add exonerate., Ben Woodcroft, 2016/07/27
- [PATCH 20/22] gnu: Add prank., Ben Woodcroft, 2016/07/27
- [PATCH 21/22] gnu: Add raxml., Ben Woodcroft, 2016/07/27
- [PATCH 22/22] gnu: Add roary., Ben Woodcroft, 2016/07/27
- Re: [PATCH 22/22] gnu: Add roary.,
Leo Famulari <=
- Re: [PATCH 01/22] gnu: parallel: Use full paths for executables., Leo Famulari, 2016/07/28
- Re: PATCH [0/22]: Add roary., Ludovic Courtès, 2016/07/31