Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-30 Thread Ludovic Courtès
Hello, Marius Bakke skribis: > I'm not sure, we would have to do this with every package that uses > the pysam cython interface. This particular issue is described in the > manual: > > https://www.gnu.org/software/guix/manual/html_node/package-Reference.html > > The propagated-inputs section say

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-27 Thread Ben Woodcroft
On 27/09/16 04:32, Marius Bakke wrote: Ben Woodcroft writes: On 24/09/16 13:15, Marius Bakke wrote: Ben Woodcroft writes: Htslib is propagated because it is included by the installed samtools and bcftools headers; at least one of pysams dependencies failed to build when it was a regular in

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-26 Thread Marius Bakke
Ben Woodcroft writes: > On 24/09/16 13:15, Marius Bakke wrote: >> Ben Woodcroft writes: >> Htslib is propagated because it is included by the installed samtools and bcftools headers; at least one of pysams dependencies failed to build when it was a regular input. >>> Ah, I see. Ca

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-26 Thread Ben Woodcroft
On 24/09/16 13:15, Marius Bakke wrote: Ben Woodcroft writes: Htslib is propagated because it is included by the installed samtools and bcftools headers; at least one of pysams dependencies failed to build when it was a regular input. Ah, I see. Can this be avoided with patchelf? I don't th

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-23 Thread Marius Bakke
Ben Woodcroft writes: >> Htslib is propagated because it >> is included by the installed samtools and bcftools headers; at least one >> of pysams dependencies failed to build when it was a regular input. > Ah, I see. Can this be avoided with patchelf? I don't think so. The htslib headers need to

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-23 Thread Ben Woodcroft
On 24/09/16 01:56, Marius Bakke wrote: Ben Woodcroft writes: Also, I notice that pysam bundles htslib, bcftools and samtools C code. Hopefully it should be straightforward enough to remove htslib as there are install instructions, I'm not sure about the other two. This shouldn't block the pa

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-23 Thread Marius Bakke
Marius Bakke writes: > Ben Woodcroft writes: > >>> Also, I notice that pysam bundles htslib, bcftools and samtools C code. Hopefully it should be straightforward enough to remove htslib as there are install instructions, I'm not sure about the other two. This shouldn't block

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-23 Thread Marius Bakke
Ben Woodcroft writes: >> >>> Also, I notice that pysam bundles htslib, bcftools and samtools C code. >>> Hopefully it should be straightforward enough to remove htslib as there >>> are install instructions, I'm not sure about the other two. This >>> shouldn't block the patch here, but would you m

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-22 Thread Ben Woodcroft
On 09/23/2016 01:15 AM, Marius Bakke wrote: Ben Woodcroft writes: Subject: [PATCH 1/3] gnu: python-pysam: Update to 0.9.1.4. I'm not sure whether this is a product of the upgrade or not, but I notice this in the build log. I think it is harmless though, WDYT? starting phase `validate-runpa

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-22 Thread Marius Bakke
Ben Woodcroft writes: >> Subject: [PATCH 1/3] gnu: python-pysam: Update to 0.9.1.4. > I'm not sure whether this is a product of the upgrade or not, but I > notice this in the build log. I think it is harmless though, WDYT? > > starting phase `validate-runpath' > validating RUNPATH of 10 binaries

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-21 Thread Ben Woodcroft
On 22/09/16 07:14, Marius Bakke wrote: [..] Hi Ben, Thanks for the review. Updated patches attached. Hi, Subject: [PATCH 1/3] gnu: python-pysam: Update to 0.9.1.4. I'm not sure whether this is a product of the upgrade or not, but I notice this in the build log. I think it is harmless though

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-21 Thread Marius Bakke
t; +"1i1djacqbr88y7w18b4aa78zxnsyr4sz7yqdq2spi7gs0y6pzvjn")))) (build-system python-build-system) (arguments `(#:tests? #f ; tests are excluded in the manifest -- 2.10.0 >From 6a8ddd17d7f11e0687217c8649a452a1a56c596c Mon Sep 17 00:00:00 2001 From: Ma

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-20 Thread Ben Woodcroft
Hi again, On 20/09/16 16:34, Ben Woodcroft wrote: Oops, forgot to cc the list. On 20/09/16 16:02, Ben Woodcroft wrote: Hi Marius, Thanks for the patches. On 17/09/16 20:10, Marius Bakke wrote: [..] + (add-after 'unpack 'fix-tests + (lambda _ + ;; See https://

Re: [PATCH 2/3] gnu: Add python-pyxb.

2016-09-19 Thread Ben Woodcroft
Oops, forgot to cc the list. On 20/09/16 16:02, Ben Woodcroft wrote: Hi Marius, Thanks for the patches. On 17/09/16 20:10, Marius Bakke wrote: [..] + (add-after 'unpack 'fix-tests + (lambda _ + ;; See https://github.com/pabigot/pyxb/issues/26 ... + (

[PATCH 2/3] gnu: Add python-pyxb.

2016-09-17 Thread Marius Bakke
* gnu/packages/xml.scm (python-pyxb, python2-pyxb): New variables. --- gnu/packages/xml.scm | 37 + 1 file changed, 37 insertions(+) diff --git a/gnu/packages/xml.scm b/gnu/packages/xml.scm index 7befad5..793cd2f 100644 --- a/gnu/packages/xml.scm +++ b/gnu/pack