[PATCH] V3 Add pyyaml to -devel
richard.maw at codethink.co.uk
Mon Jan 28 11:23:54 GMT 2013
On Mon, Jan 28, 2013 at 11:13:16AM +0000, Sam Thursfield wrote:
> On 01/24/2013 06:26 PM, Javier Jardón wrote:
> >Repo: git://git.baserock.org/baserock/baserock/morphs
> >Ref: jjardon/yaml
> >Sha1: ce470314bb28e8a0595cf66b4d30404d5efc0d2a
> >This patch only adds pyyaml to the morpholy.
> >As others commented this avoid pull on more dependencies for now
> >I tested this with the morph branch to use YAML and all the test passes.
> >Ill resubmit the patches for morph with some fixes in a posterior email.
> >Javier Jardón (1):
> > devel: Add pyyaml
> > devel.morph | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> I have just realised that we approached this problem the wrong way.
> If we want to minimise disruption to Baserock users, but also want
> to use libyaml and a C YAML parser if possible, we should merge
> libyaml to baserock:baserock/morphs now, so that we can use it Morph
> after the next release -- right?
The problem with that is that pyyaml with libyaml currently doesn't
build because we haven't included Cython.
I agree that it would be nice to have it now, but the branch of morph to
add yaml doesn't currently use the C Loaders anyway, and we can
conditionally use them if they are available if we have to provide
backwards compatibility after we've added libyaml.
More information about the baserock-dev