Justus Winter writes:
> Quoting Ben Woodcroft (2016-02-15 02:13:07)
>> Thanks for the patch, but it appears you've included many other
>> changes.
>
> Yeah :/
>
>> Can you resend without them please?
>
> I will, as soon as I have revised the package not to use some
> libraries it bundles.
Feel
Quoting Ben Woodcroft (2016-02-15 02:13:07)
> Thanks for the patch, but it appears you've included many other
> changes.
Yeah :/
> Can you resend without them please?
I will, as soon as I have revised the package not to use some
libraries it bundles.
Justus
On 15/02/16 10:47, Ben Woodcroft wrote:
On 15/02/16 03:49, Justus Winter wrote:
* gnu-system.am (dist_patch_DATA): Add new patch.
* gnu/packages/bioinformatics.scm (genometools): New variable.
* gnu/packages/patches/genometools-fix-testsuite.patch: New file.
---
gnu-system.am
Quoting Justus Winter (2016-02-14 18:19:35)
> I'll send a follow-up.
Eww, that went bad :(
Furthermore, the application bundles some libraries, I need to take
care of that first.
I'll send a revised patch soonish...
Justus
Hi,
Quoting Andreas Enge (2016-02-14 16:47:10)
> On Sun, Feb 14, 2016 at 03:56:21PM +0100, Justus Winter wrote:
> > this is my first package, please be nice ;)
>
> we always are :-)
:)
> The package looks very nice, I only have a few minor comments.
>
> On Sun, Feb 14, 2016 at 03:56:22PM +010
PS: I forgot the magic words: Could you please send an updated patch?
Ah, and add a copyright line for yourself.
Hello,
On Sun, Feb 14, 2016 at 03:56:21PM +0100, Justus Winter wrote:
> this is my first package, please be nice ;)
we always are :-)
The package looks very nice, I only have a few minor comments.
On Sun, Feb 14, 2016 at 03:56:22PM +0100, Justus Winter wrote:
> * gnu-system.am (dist_patch_DATA
* gnu-system.am (dist_patch_DATA): Add new patch.
* gnu/packages/bioinformatics.scm (genometools): New package.
* gnu/packages/patches/genometools-fix-testsuite.patch: New file.
---
gnu-system.am | 1 +
gnu/packages/bioinformatics.scm| 40 +
