[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/3] gnu: Add python-pyxb.
From: |
Marius Bakke |
Subject: |
Re: [PATCH 2/3] gnu: Add python-pyxb. |
Date: |
Fri, 23 Sep 2016 17:06:18 +0100 |
Marius Bakke <address@hidden> writes:
> Ben Woodcroft <address@hidden> 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 mind taking a look?
>>>> http://pysam.readthedocs.io/en/latest/installation.html#installation
>>> I had a go at this, and also enabled tests since I was reading the build
>>> system anyway. Samtools and bcftools does not seem possible to un-bundle
>>> at this time, but htslib was straightforward.
>> OK. I don't think it needs to be propagated though, right? Also, would
>> you mind separating the change to modify-phases syntax and unbundling of
>> htslib into two patches please? Other than that this whole series LGTM.
>
> Modify-phases is now a separate patch. 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.
>
> I added some comments and other cosmetic changes. Updated patch below.
Oops, found a spelling error in the previous patch. Fixed.
pysam-and-pbcore-update.patch
Description: Text Data
- [PATCH 0/3] gnu: python2-pbcore: Update to 1.2.10., Marius Bakke, 2016/09/17
- [PATCH 1/3] gnu: python-pysam: Update to 0.9.1.4., Marius Bakke, 2016/09/17
- [PATCH 2/3] gnu: Add python-pyxb., Marius Bakke, 2016/09/17
- Message not available
- Re: [PATCH 2/3] gnu: Add python-pyxb., Ben Woodcroft, 2016/09/20
- Re: [PATCH 2/3] gnu: Add python-pyxb., Ben Woodcroft, 2016/09/20
- Re: [PATCH 2/3] gnu: Add python-pyxb., Marius Bakke, 2016/09/21
- Re: [PATCH 2/3] gnu: Add python-pyxb., Ben Woodcroft, 2016/09/21
- Re: [PATCH 2/3] gnu: Add python-pyxb., Marius Bakke, 2016/09/22
- Re: [PATCH 2/3] gnu: Add python-pyxb., Ben Woodcroft, 2016/09/22
- Re: [PATCH 2/3] gnu: Add python-pyxb., Marius Bakke, 2016/09/23
- Re: [PATCH 2/3] gnu: Add python-pyxb.,
Marius Bakke <=
- Re: [PATCH 2/3] gnu: Add python-pyxb., Ben Woodcroft, 2016/09/23
- Re: [PATCH 2/3] gnu: Add python-pyxb., Marius Bakke, 2016/09/23
- Re: [PATCH 2/3] gnu: Add python-pyxb., Ben Woodcroft, 2016/09/26
- Re: [PATCH 2/3] gnu: Add python-pyxb., Marius Bakke, 2016/09/26
- Re: [PATCH 2/3] gnu: Add python-pyxb., Ben Woodcroft, 2016/09/27
- Re: [PATCH 2/3] gnu: Add python-pyxb., Ludovic Courtès, 2016/09/30
[PATCH 3/3] gnu: python2-pbcore: Update to 1.2.10., Marius Bakke, 2016/09/17