Re: [PATCH] Add preseq.

[Ludovic Courtès]

Ricardo Wurmus <address@hidden> skribis:

> From 3ffe6847848f6ba107e0757728f1c6a6e7807d56 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <address@hidden>
> Date: Tue, 23 Jun 2015 10:54:00 +0200
> Subject: [PATCH] gnu: Add preseq.
>
> * gnu/packages/bioinformatics.scm (preseq): New variable.

[...]

> +              (snippet
> +               ;; Remove bundled samtools.  Leave the headers.
> +               '(for-each delete-file
> +                          (find-files "preseq-master/samtools"
> +                                      "\\.c$")))))

Why leave the headers?  Isn’t it risky to leave headers that may be
different from the Samtools that we’re actually linking against?

Otherwise LGTM.

Thanks,
Ludo’.