Bug #67022 for Devel-Declare: Debugger broken in 5.13.[7..11]

Mon Mar 28 09:37:09 2011 mschwern [...] cpan.org - Ticket created

Subject:

Debugger broken in 5.13.[7..11]

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.

Mon Apr 04 23:23:14 2011 mschwern [...] cpan.org - Correspondence added

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.

Mon Apr 04 23:23:15 2011 mschwern [...] cpan.org - Status changed from 'new' to 'open'

Tue Apr 05 09:26:39 2011 rafl [...] debian.org - Correspondence added

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.

Tue Apr 05 18:22:08 2011 schwern [...] pobox.com - Correspondence added

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>

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

Tue Apr 05 20:01:03 2011 rafl [...] debian.org - Correspondence added

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>

"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.

Sun Sep 11 14:06:18 2011 ZEFRAM [...] cpan.org - Correspondence added

Was fixed in Devel-Declare 0.006002.

Sun Sep 11 14:06:19 2011 ZEFRAM [...] cpan.org - Status changed from 'open' to 'resolved'