Skip Menu |
 

This queue is for tickets about the Config-Auto CPAN distribution.

Report information
The Basics
Id: 45863
Status: open
Priority: 0/
Queue: Config-Auto

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

Bug Information
Severity: Critical
Broken in: (no value)
Fixed in: 0.29_02



Subject: change to eval_perl implementation is not backcompat
Download (untitled) / with headers
text/plain 589b
I like the new change from Lyle Hopkins to only load modules if they are needed. Thanks for that. However, the current dev release 0.29_2 quits working for me because of the re-implementation of eval_perl. I had a config file like this: %CFG = ( a => 'b' ); \%CFG; ### The old implementation with "do" ran this in it's own scope, and all was good. The new implementation with eval "foo" runs this in the lexical scope of the subroutine in Config::Auto, and then dies because these are considered global variables. I suggest reverting the old behavior for backcompat. Mark
Download (untitled) / with headers
text/plain 704b
On Fri May 08 11:35:42 2009, MARKSTOS wrote: Show quoted text
> I like the new change from Lyle Hopkins to only load modules if they are > needed. Thanks for that. > > However, the current dev release 0.29_2 quits working for me because of > the re-implementation of eval_perl. I had a config file like this: > > %CFG = ( > a => 'b' > ); > > \%CFG; > > ### > > The old implementation with "do" ran this in it's own scope, and all was > good. > > The new implementation with eval "foo" runs this in the lexical scope of > the subroutine in Config::Auto, and then dies because these are > considered global variables. > > I suggest reverting the old behavior for backcompat.
Would you accept a patch for this?
CC: undisclosed-recipients: ;
Subject: Re: [rt.cpan.org #45863] change to eval_perl implementation is not backcompat
Date: Mon, 29 Mar 2010 16:24:25 +0100
To: MARKSTOS via RT <bug-Config-Auto [...] rt.cpan.org>
From: "Chris 'BinGOs' Williams" <chris [...] bingosnet.co.uk>
On Mon, Mar 22, 2010 at 11:15:21AM -0400, MARKSTOS via RT wrote: Show quoted text
> Queue: Config-Auto > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=45863 > > > On Fri May 08 11:35:42 2009, MARKSTOS wrote:
> > I like the new change from Lyle Hopkins to only load modules if they are > > needed. Thanks for that. > > > > However, the current dev release 0.29_2 quits working for me because of > > the re-implementation of eval_perl. I had a config file like this: > > > > %CFG = ( > > a => 'b' > > ); > > > > \%CFG; > > > > ### > > > > The old implementation with "do" ran this in it's own scope, and all was > > good. > > > > The new implementation with eval "foo" runs this in the lexical scope of > > the subroutine in Config::Auto, and then dies because these are > > considered global variables. > > > > I suggest reverting the old behavior for backcompat.
> > Would you accept a patch for this? >
Sure. -- Chris Williams aka BinGOs PGP ID 0x4658671F http://www.gumbynet.org.uk ==========================
Download (untitled)
application/pgp-signature 189b

Message body not shown because it is not plain text.



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.