Skip Menu |
 
rt.cpan.org will be shut down on March 1st, 2021.

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Locale-Maketext-Lexicon CPAN distribution.

Report information
The Basics
Id: 54881
Status: resolved
Priority: 0/
Queue: Locale-Maketext-Lexicon

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

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



Subject: Test suite fails on perl 5.11.1 and newer
Download (untitled) / with headers
text/plain 197b
It seems that the Locale::Maketext::Lexicon test suite was broken at some point after perl 5.11.1 was released. See http://matrix.cpantesters.org/?dist=Locale-Maketext-Lexicon Regards, Slaven
Download (untitled) / with headers
text/plain 308b
On Mon Feb 22 11:30:18 2010, SREZIC wrote: Show quoted text
> It seems that the Locale::Maketext::Lexicon test suite was broken at > some point after perl 5.11.1 was released. See > http://matrix.cpantesters.org/?dist=Locale-Maketext-Lexicon > > Regards, > Slaven
Hiya Should be fixed in v 0.78 - just uploaded clint
Subject: Re: [rt.cpan.org #54881] Test suite fails on perl 5.11.1 and newer
Date: Wed, 24 Feb 2010 12:42:19 -0500
To: Clinton Gormley via RT <bug-Locale-Maketext-Lexicon [...] rt.cpan.org>
From: Jesse Vincent <jesse [...] fsck.com>
Download (untitled) / with headers
text/plain 747b
C, What was the actual issue? If blead broke L::M::L, it's important to make sure that 5.12 isn't breaking backward compatibility in some way we don't know about. (And we're really close to 5.12's release ;) Thanks! -Jesse On Tue 23.Feb'10 at 5:37:14 -0500, Clinton Gormley via RT wrote: Show quoted text
> Queue: Locale-Maketext-Lexicon > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=54881 > > > On Mon Feb 22 11:30:18 2010, SREZIC wrote:
> > It seems that the Locale::Maketext::Lexicon test suite was broken at > > some point after perl 5.11.1 was released. See > > http://matrix.cpantesters.org/?dist=Locale-Maketext-Lexicon > > > > Regards, > > Slaven
> > > Hiya > > Should be fixed in v 0.78 - just uploaded > > clint >
Download (untitled) / with headers
text/plain 468b
Hi Jesse Show quoted text
> What was the actual issue? If blead broke L::M::L, it's important to > make sure that 5.12 isn't breaking backward compatibility in some way we > don't know about.
The issue was that the code was using if (defined %hash), which is deprecated. I fixed that, but I've seen some more fail reports which I haven't had time to investigate yet, eg: http://www.cpantesters.org/cpan/report/6862269 I'm away this weekend, but will look at it next week. clint
RT-Send-CC: jesse [...] fsck.com
Download (untitled) / with headers
text/plain 461b
I made a bisect against 0.78 and it came to the conclusion: d47e1c27dd43d7b79216c58864a5e539f612e974 is the first bad commit commit d47e1c27dd43d7b79216c58864a5e539f612e974 Author: Nicholas Clark <nick@ccl4.org> Date: Sat Oct 24 10:25:28 2009 +0100 Make defined %hash on a non-lexical (also) generate a deprecated warning. Tk has been fixed, and no longer uses defined %stash:: to determine if a package has been loaded. HTH,
RT-Send-CC: jesse [...] fsck.com
Removed another recurrence of "defined %hash" This now tests fine on perl 5.11.5


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.