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

History Show all quoted text
  Fri May 08 11:35:42 2009 MARKSTOS [...] cpan.org - Ticket created
Subject: change to eval_perl implementation is not backcompat
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
  Mon Mar 22 11:15:21 2010 MARKSTOS [...] cpan.org - Correspondence added
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?
  Mon Mar 22 11:15:25 2010 MARKSTOS [...] cpan.org - Status changed from 'new' to 'open'
  Mon Mar 29 11:24:43 2010 chris [...] bingosnet.co.uk - Correspondence added
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.