Bug #78478 for Regexp-Debugger: REPL needs evalbytes under 5.16

Thu Jul 19 23:21:39 2012 http://alierak.dreamwidth.org/ - Ticket created

Subject:

REPL needs evalbytes under 5.16

Pressing m in the REPL does nothing under perl 5.16.0. Further examination shows $regex is undef because the eval that sets it is silently failing. Adding use feature "evalbytes" in Debugger.pm seems to make it work.

Fri Jul 20 20:15:46 2012 damian [...] conway.org - Correspondence added

Subject:	Re: [rt.cpan.org #78478] REPL needs evalbytes under 5.16
Date:	Fri, 20 Jul 2012 17:14:52 -0700
To:	bug-Regexp-Debugger [...] rt.cpan.org
From:	Damian Conway <damian [...] conway.org>

Thanks for the bug report, alierak. My perlbrewed 5.16 is currently misbehaving, so it will be a little while before I can sort this out. I'm glad you found a workaround in the meantime. Can I just clarify though. You said: Show quoted text

> Adding use feature "evalbytes" in Debugger.pm seems to > make it work.

You didn't get it work to by just loading the module, did you? You had to s/eval/evalbytes/ in the actual code, right? Damian

Fri Jul 20 20:15:47 2012 The RT System itself - Status changed from 'new' to 'open'

Fri Jul 20 20:34:12 2012 http://alierak.dreamwidth.org/ - Correspondence added

From:

alierak [...] gmail.com

On Fri Jul 20 20:15:46 2012, damian@conway.org wrote: Show quoted text

> You didn't get it work to by just loading the module, did you? > You had to s/eval/evalbytes/ in the actual code, right?

Surprisingly, yes, I got it to work by just loading the module, without changing any other code. I haven't studied evalbytes enough to tell whether that should have worked. I'm using ActivePerl 5.16.0 build 1600 on MacOS X 10.6.8, but I doubt that has anything to do with evalbytes behavior.

Fri Jul 20 20:45:33 2012 damian [...] conway.org - Correspondence added

Subject:	Re: [rt.cpan.org #78478] REPL needs evalbytes under 5.16
Date:	Fri, 20 Jul 2012 17:44:44 -0700
To:	bug-Regexp-Debugger [...] rt.cpan.org
From:	Damian Conway <damian [...] conway.org>

Show quoted text

> Surprisingly, yes, I got it to work by just loading the module, without > changing any other code.

Thanks for the quick feedback. I certainly not doubting you, but I find that extremely surprising. I think I must not fully understand the behaviour of 'evalbytes' (it's documentation definitely doesn't hint at that weird effect). On the other hand, over the years I have become deeply suspicious of ActivePerl, and that might potentially be a factor here. Once I get 5.16 happy on my current machine, I'll look into this further and see it I can replicate it under perlbrew under the same O/S version. Thanks again, Damian

Fri Jul 20 23:35:52 2012 http://alierak.dreamwidth.org/ - Correspondence added

From:

alierak [...] gmail.com

On Fri Jul 20 20:45:33 2012, damian@conway.org wrote: Show quoted text

> Once I get 5.16 happy on my current machine, I'll look into this > further and see it I can replicate it under perlbrew under the same > O/S version.

No worries. I just tested this with a perlbrew 5.16.0 install on CentOS 5.5 ("Linux *** 2.6.18-194.el5 #1 SMP Fri Apr 2 14:58:14 EDT 2010 x86_64 x86_64 x86_64 GNU/Linux") and saw the same behavior.

Sat Jul 21 13:33:00 2012 damian [...] conway.org - Correspondence added

Subject:	Re: [rt.cpan.org #78478] REPL needs evalbytes under 5.16
Date:	Sat, 21 Jul 2012 10:32:09 -0700
To:	bug-Regexp-Debugger [...] rt.cpan.org
From:	Damian Conway <damian [...] conway.org>

I've now uploaded 0.001001 with a patch for 5.16 behaviour. Give it a try when you get the chance and let me know if it still doesn't solve the problem. Thanks again for the report! Damian

Sat Jul 21 19:43:01 2012 DCONWAY [...] cpan.org - Status changed from 'open' to 'patched'

Sun Aug 19 06:17:50 2012 DCONWAY [...] cpan.org - Correspondence added

Appears to have been resolved. Please report if otherwise.

Sun Aug 19 06:17:50 2012 The RT System itself - Status changed from 'patched' to 'open'

Sun Aug 19 06:17:50 2012 DCONWAY [...] cpan.org - Status changed from 'open' to 'resolved'