Skip Menu |
 

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

Report information
The Basics
Id: 67022
Status: resolved
Priority: 0/
Queue: Devel-Declare

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

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



Subject: Debugger broken in 5.13.[7..11]
Download (untitled) / with headers
text/plain 523b
Devel::Declare has been broken when used in the debugger since at least 5.13.7. You can see it reflected in the Method::Signatures tests which check it works in the debugger. http://www.cpantesters.org/distro/M/Method-Signatures.html#Method-Signatures-20110324.1600_001?grade=1&perlmat=3&patches=2&oncpan=2&distmat=1&perlver=ALL&osname=ALL&version=20110324.1600_001 For example: http://www.cpantesters.org/cpan/report/1ee76bd0-5612-11e0-8a39-a371061ad747 I tried it with MooseX::Method::Signatures as well, same problem.
Download (untitled) / with headers
text/plain 684b
I've knocked it down to something repeatable with just Devel::Declare::MethodInstaller::Simple and 5.13.11. Unpack the attached tarball, cd into dd_test and run "perl5.13 -Ilib test.plx". You should get "missing argument $this" but instead I get "PL_linestr not long enough, was Devel::Declare loaded soon enough in lib/Bar.pm" It is *extremely finky*. If you put a comment line after "use DD" it will work. If you change the call to Bar->foo() in Bar.pm to Bar->foo(42, 23) or Bar->foo("foo") or Bar->foo(1234) it will work. Bar->foo(123) will NOT work. It is definitely related to how much stuff is injected. If you change DD::required_arg() to have a shorter name it works.
Subject: dd_test.tgz
Download dd_test.tgz
application/x-gzip 696b

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #67022] Debugger broken in 5.13.[7..11]
Date: Tue, 05 Apr 2011 15:26:14 +0200
To: bug-Devel-Declare [...] rt.cpan.org
From: Florian Ragwitz <rafl [...] debian.org>
Does it start working again with http://gist.github.com/903574 applied?
Download (untitled)
application/pgp-signature 197b

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #67022] Debugger broken in 5.13.[7..11]
Date: Wed, 06 Apr 2011 08:21:50 +1000
To: bug-Devel-Declare [...] rt.cpan.org
From: Michael G Schwern <schwern [...] pobox.com>
Download (untitled) / with headers
text/plain 420b
On 2011.4.5 11:26 PM, Florian Ragwitz via RT wrote: Show quoted text
> Does it start working again with http://gist.github.com/903574 applied?
Yes! Both Method::Signatures and my test code work with that. Thank you! PS 0.006001 is not in git://github.com/rafl/devel-declare.git -- 'All anyone gets in a mirror is themselves,' she said. 'But what you gets in a good gumbo is everything.' -- "Witches Abroad" by Terry Prachett
Subject: Re: [rt.cpan.org #67022] Debugger broken in 5.13.[7..11]
Date: Wed, 06 Apr 2011 02:00:42 +0200
To: bug-Devel-Declare [...] rt.cpan.org
From: Florian Ragwitz <rafl [...] debian.org>
Download (untitled) / with headers
text/plain 778b
"Michael G Schwern via RT" <bug-Devel-Declare@rt.cpan.org> writes: Show quoted text
> Queue: Devel-Declare > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=67022 > > > On 2011.4.5 11:26 PM, Florian Ragwitz via RT wrote:
>> Does it start working again with http://gist.github.com/903574 applied?
> > Yes! Both Method::Signatures and my test code work with that. Thank > you!
Awesome, thanks for testing! This was actually a fix for test failures of MXD without the debugger on 5.13 that Zefram provided a patch for. I just hadn't gotten around to testing it yet. I'll ship it to CPAN, along with some more tests, including yours, during this week. Show quoted text
> PS 0.006001 is not in git://github.com/rafl/devel-declare.git
Thanks, I wasn't aware. I'll fix that too, while I'm at it.
Download (untitled)
application/pgp-signature 197b

Message body not shown because it is not plain text.

Was fixed in Devel-Declare 0.006002.


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.