Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the YAML-LibYAML CPAN distribution.

Report information
The Basics
Id: 43115
Status: resolved
Priority: 0/
Queue: YAML-LibYAML

People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.32
Fixed in: (no value)

Attachments
0004-Add-support-for-SVt_REGEXP-RT-43115.patch



Subject: Two binary searches
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Type: text/plain
Charset: utf8
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 644
Download (untitled) / with headers
text/plain 644b
cpantesters expose there is only a range of perls that pass all tests. Current blead does not PASS because of the new regexes. These started with patch 32740. From then on the tests fail similar to this: http://www.nntp.perl.org/group/perl.cpan.testers/2009/02/msg3220752.html http://www.nntp.perl.org/group/perl.cpan.testers/2009/02/msg3219948.html At the lower end there was patch 22862 which fixed Scalar::Util::refaddr. Before this fix the tests complained about bad quoting as you can see here: http://www.nntp.perl.org/group/perl.cpan.testers/2009/02/msg3225989.html For the latter you could get away with 'require 5.008005'. Best,
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Charset: utf8
Content-Type: text/plain
Message-ID: <rt-3.6.HEAD-9877-1260427288-1070.43115-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 791
Download (untitled) / with headers
text/plain 791b
I have the same issues as http://www.nntp.perl.org/group/perl.cpan.testers/2009/02/msg3220615.html when I try to install YAML::LibYAML 0.32 I use Perl 5.11.2, any solution by now? Show quoted text
> Current blead does not PASS because of the new regexes. These started > with patch 32740. From then on the tests fail similar to this: > > http://www.nntp.perl.org/group/perl.cpan.testers/2009/02/msg3220752.html > http://www.nntp.perl.org/group/perl.cpan.testers/2009/02/msg3219948.html > > At the lower end there was patch 22862 which fixed > Scalar::Util::refaddr. Before this fix the tests complained about bad > quoting as you can see here: > > http://www.nntp.perl.org/group/perl.cpan.testers/2009/02/msg3225989.html > > For the latter you could get away with 'require 5.008005'. > > Best,
MIME-Version: 1.0
X-Spam-Status: No, hits=0.0 required=8.0 tests=
In-Reply-To: <rt-3.6.HEAD-9877-1260427288-1070.43115-5-0 [...] rt.cpan.org>
References: <RT-Ticket-43115 [...] rt.cpan.org> <rt-3.6.HEAD-9877-1260427288-1070.43115-5-0 [...] rt.cpan.org>
Message-ID: <a891e1bd0912100140y2cde581dxcdcd83121cac5ab1 [...] mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by diesel.bestpractical.com (Postfix) with SMTP id 27AD14D8025 for <bug-YAML-LibYAML [...] rt.cpan.org>; Thu, 10 Dec 2009 04:40:25 -0500 (EST)
Received: (qmail 29811 invoked by uid 103); 10 Dec 2009 09:40:25 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 10 Dec 2009 09:40:25 -0000
Received: from mail-fx0-f219.google.com (HELO mail-fx0-f219.google.com) (209.85.220.219) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Thu, 10 Dec 2009 01:40:23 -0800
Received: by fxm19 with SMTP id 19so7011279fxm.37 for <bug-YAML-LibYAML [...] rt.cpan.org>; Thu, 10 Dec 2009 01:40:20 -0800 (PST)
Received: by 10.223.13.214 with SMTP id d22mr2122815faa.53.1260438020153; Thu, 10 Dec 2009 01:40:20 -0800 (PST)
Delivered-To: cpan-bug+YAML-LibYAML [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43115] Broken with bleadperl and with 5.8.4 (was: Two binary searches)
Return-Path: <nothingmuch [...] woobling.org>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-YAML-LibYAML [...] rt.cpan.org
X-Google-Sender-Auth: 7a8ac36a371cbec3
Date: Thu, 10 Dec 2009 11:40:00 +0200
Sender: nothingmuch [...] woobling.org
X-Spam-Level: *
To: bug-YAML-LibYAML [...] rt.cpan.org
From: Yuval Kogman <nuffin [...] cpan.org>
RT-Message-ID: <rt-3.6.HEAD-9877-1260438036-1682.43115-0-0 [...] rt.cpan.org>
Content-Length: 138
Download (untitled) / with headers
text/plain 138b
Looks like the new SvTYPE for regexes is not yet supported. It shouldn't be hard to patch. i might have time to try doing this next week
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Type: multipart/mixed; boundary="----------=_1266142272-14820-138"
Message-ID: <rt-3.8.HEAD-14820-1266142272-126.43115-0-0 [...] rt.cpan.org>
X-RT-Original-Encoding: utf-8
RT-Send-CC: nuffin [...] cpan.org
Content-Length: 0
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 33
This patch seems to do the trick.
MIME-Version: 1.0
Subject: 0004-Add-support-for-SVt_REGEXP-RT-43115.patch
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Type: application/octet-stream; name="0004-Add-support-for-SVt_REGEXP-RT-43115.patch"
Content-Disposition: inline; filename="0004-Add-support-for-SVt_REGEXP-RT-43115.patch"
Content-Transfer-Encoding: base64
Content-Length: 1134
From 01e53918fab954e8c52bba258068599807fe6367 Mon Sep 17 00:00:00 2001 From: Gisle Aas <gisle@activestate.com> Date: Sun, 14 Feb 2010 02:06:16 -0800 Subject: [PATCH 4/4] Add support for SVt_REGEXP [RT#43115] perl-5.12 make regular expressions first class. --- LibYAML/perl_libyaml.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/LibYAML/perl_libyaml.c b/LibYAML/perl_libyaml.c index 811e3a5..29e0294 100644 --- a/LibYAML/perl_libyaml.c +++ b/LibYAML/perl_libyaml.c @@ -652,6 +652,15 @@ dump_node(perl_yaml_dumper_t *dumper, SV *node) } dump_scalar(dumper, node, tag); } +#if PERL_REVISION > 5 || (PERL_REVISION == 5 && PERL_VERSION >= 11) + else if (ref_type == SVt_REGEXP) { + yaml_char_t *tag = (yaml_char_t *)form(TAG_PERL_PREFIX "regexp"); + class = sv_reftype(rnode, TRUE); + if (!strEQ(class, "Regexp")) + tag = (yaml_char_t *)form("%s:%s", tag, class); + dump_scalar(dumper, node, tag); + } +#endif else { printf( "YAML::XS dump unhandled ref. type == '%d'!\n", -- 1.6.2.rc2
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-14820-1266142272-126.43115-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <rt-3.8.HEAD-14820-1266142272-126.43115-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-26543-1271069396-1505.43115-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 266
Download (untitled) / with headers
text/plain 266b
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-26544-1271294620-1936.43115-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
RT-Send-CC: nuffin [...] cpan.org
Content-Length: 56
Applied Gisle's patch to 0.32 and released 0.33 to CPAN.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.