Skip Menu |

This queue is for tickets about the Template-Declare CPAN distribution.

Report information
The Basics
Id: 100709
Status: resolved
Priority: 0/
Queue: Template-Declare

People
Owner: Nobody in particular
Requestors: syber [...] crazypanda.ru
Cc:
AdminCc:

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



Subject: Bug in test t/namespace.t
On the line 58: ::like $@, qr/Can't locate object method "div" via package "htm"/, 'package htm is intact'; You should not check perl's diagnostic messages, as they are subject to change. In particular, there was a bug in perl which caused code like htm::div {}; produce message "Can't locate object method "div" via package "htm"" when there is no method htm::div. Since perl 5.21.7 this was fixed and now message is "Can't call method "div" without a package or object reference" because you can't call method without passing first arg. Nevertheless i'd suggest you NOT to check perl's diagnostic messages. Therefore the best solution would be just deleting line 58. Because you've already checked that that method call croaked.
Subject: [rt.cpan.org #100709] Proposed patch to support Perl 5.21.7+
Date: Tue, 16 Dec 2014 15:54:32 +0000
To: bug-Template-Declare [...] rt.cpan.org
From: Aaron Crane <arc [...] cpan.org>
The attached patch fixes the test failure under Perl 5.21.7, while retaining what I think is the intent of the failing test. -- Aaron Crane ** http://aaroncrane.co.uk/

Message body is not shown because sender requested not to inline it.

Subject: Re: [rt.cpan.org #100709] Proposed patch to support Perl 5.21.7+
Date: Tue, 16 Dec 2014 23:30:10 -0500
To: bug-Template-Declare [...] rt.cpan.org
From: Alex Vandiver <alex [...] chmrr.net>
On 12/16/2014 10:54 AM, arc@cpan.org via RT wrote: Show quoted text
> The attached patch fixes the test failure under Perl 5.21.7, while > retaining what I think is the intent of the failing test.
Thanks, applied! - Alex