Skip Menu |

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

History Show all quoted text
  Mon Feb 22 11:30:18 2010 SREZIC [...] cpan.org - Ticket created
Subject: Test suite fails on perl 5.11.1 and newer
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
  Tue Feb 23 05:37:10 2010 develop [...] traveljury.com - Correspondence added
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
  Tue Feb 23 05:37:15 2010 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 23 05:37:17 2010 develop [...] traveljury.com - Status changed from 'open' to 'resolved'
  Wed Feb 24 19:58:31 2010 jesse [...] fsck.com - Correspondence added
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>
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 >
  Wed Feb 24 19:58:33 2010 The RT System itself - Status changed from 'resolved' to 'open'
  Thu Feb 25 12:43:57 2010 develop [...] traveljury.com - Correspondence added
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
  Sun Feb 28 06:53:46 2010 ANDK [...] cpan.org - Correspondence added
RT-Send-CC: jesse [...] fsck.com
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,
  Tue Mar 02 10:50:04 2010 develop [...] traveljury.com - Correspondence added
RT-Send-CC: jesse [...] fsck.com
Removed another recurrence of "defined %hash" This now tests fine on perl 5.11.5
  Tue Mar 02 10:50:10 2010 develop [...] traveljury.com - Status changed from 'open' to 'resolved'