Skip Menu |
 

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

Report information
The Basics
Id: 60064
Status: resolved
Priority: 0/
Queue: YAML-Syck

People
Owner: TODDR [...] cpan.org
Requestors: paul [...] city-fan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 1.11
Fixed in: (no value)

Attachments


Subject: perl 5.8.8 has problems with yaml-tie.t, just like 5.8.[67]
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
X-RT-Original-Encoding: utf-8
Content-Type: multipart/mixed; boundary="----------=_1280915976-2217-243"
Content-Length: 0
Content-Type: text/plain; charset="UTF-8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
Content-Length: 530
Download (untitled) / with headers
text/plain 530b
Part of yaml-tie.t is skipped with perl 5.8.[67] but this needs to be extended to 5.8.8 too. Example: http://www.cpantesters.org/cpan/report/07646655-b19f-3f77-b713-d32bba55d77f I see the same issue building Fedora Linux releases 5, 6, 7 and 8, which are the four releases that shipped with 5.8.8. Attached patch resolves the issue for me, and resulting module builds successfully on Red Hat Linux 9, all Fedora releases ever plus current Fedora 14 and Rawhide trees, plus Red Hat Enterprise Linux releases 3, 4, 5 and 6beta2.
Subject: YAML-Syck-1.11-yaml-tie-5.8.8.patch
MIME-Version: 1.0
Content-Type: application/octet-stream; name="YAML-Syck-1.11-yaml-tie-5.8.8.patch"
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline; filename="YAML-Syck-1.11-yaml-tie-5.8.8.patch"
Content-Transfer-Encoding: base64
Content-Length: 528
--- YAML-Syck-1.11/t/yaml-tie.t 2010-08-03 15:56:52.000000000 +0100 +++ YAML-Syck-1.11/t/yaml-tie.t 2010-08-04 10:31:11.238393145 +0100 @@ -18,7 +18,7 @@ %h = (a=>1, b=>'2', c=>3.1415, d=>4); SKIP: { - skip "Perl 5.8.[67] seem to have issues with large ints as ints", 1 unless($] >= '5.008008' || $] < '5.007'); + skip "Perl 5.8.[678] seem to have issues with large ints as ints", 1 unless($] > '5.008008' || $] < '5.007'); is(Dump($th), "--- !!perl/hash:Tie::StdHash \na: 1\nb: 2\nc: '3.1415'\nd: 4\n"); }
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-2141-1280916969-1772.60064-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 294
Download (untitled) / with headers
text/plain 294b
Thanks and confirmed, but I'll let toddr apply it since he wrote it. I've tested on 5.8.8 which had this bug, but 5.8.9 doesn't, however I didn't go down to 5.8.5, do all version of 5.8 below 5.8.6 really not have this? Todd? Maybe we should just skip this test on 5.8 && != 5.8.9 entirely.
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-2141-1280916969-1772.60064-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <rt-3.8.HEAD-2141-1280916969-1772.60064-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2141-1280919327-1404.60064-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
From: paul [...] city-fan.org
X-RT-Original-Encoding: utf-8
Content-Length: 688
Download (untitled) / with headers
text/plain 688b
On Wed Aug 04 06:16:09 2010, AVAR wrote: Show quoted text
> Thanks and confirmed, but I'll let toddr apply it since he wrote it. > > I've tested on 5.8.8 which had this bug, but 5.8.9 doesn't, however I > didn't go down to 5.8.5, do all version of 5.8 below 5.8.6 really not > have this? Todd? > > Maybe we should just skip this test on 5.8 && != 5.8.9 entirely.
I believe so. If I patch out the skip, the test fails with: RHEL-3 (perl 5.8.0) RHEL-4 (perl 5.8.5) RHEL-5 (perl 5.8.8) FC-1 (perl 5.8.3) FC-2 (perl 5.8.3) FC-3 (perl 5.8.5) FC-4 (perl 5.8.6) FC-5 (perl 5.8.8) FC-6 (perl 5.8.8) F-7 (perl 5.8.8) F-8 (perl 5.8.8) It passes on Fedora 9 (perl 5.10.0) and later releases.
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-2141-1280919327-1404.60064-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <rt-3.8.HEAD-2141-1280916969-1772.60064-0-0 [...] rt.cpan.org> <rt-3.8.HEAD-2141-1280919327-1404.60064-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-3226-1280943215-1085.60064-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 152
Download (untitled) / with headers
text/plain 152b
I've moved it into a TODO test for all of 5.8 the test is actually not so relevant to tie anyways as far as I can tell. 1.12 should be on CPAN shortly.


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.