Skip Menu |
 

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

Report information
The Basics
Id: 40489
Status: open
Priority: 0/
Queue: Config-Simple

People
Owner: Nobody in particular
Requestors: FANY [...] cpan.org
Cc: 460423 [...] rt.noris.net
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 4.58
  • 4.59
Fixed in: (no value)



CC: 460423 [...] rt.noris.net
Subject: ->guess_syntax modifies $_
Download (untitled) / with headers
text/plain 630b
fany@fany:~> perl -MConfig::Simple -e '$_="Test";print"$_\n"' Test fany@fany:~> perl -MConfig::Simple -e '$_="Test";Config::Simple->new("foo");print"$_\n"' This es even more of a problem if $_ is a read-only value... fany@fany:~> perl -MConfig::Simple -e 'Config::Simple->new("foo") for undef' Modification of a read-only value attempted at /usr/lib/perl5/site_perl/5.10.0/Config/Simple.pm line 220. ... which just happened when trying to use your module in conjunction with CGI::Application::Plugin::Config::Simple. A simple solution would be to add a "local $_;" in front of the "while ( <$fh> )" at line 220. Regards, fany
From: ohiocore [...] gmail.com
Download (untitled) / with headers
text/plain 379b
I confirm this is still a problem with version 4.58. All it takes to avoid this problem is the insertion at line 219: local $_; Without that, in my application I get the same error: Modification of a read-only value attempted at /usr/local/share/perl5/Config/Simple.pm line 220 I would be happy to undertake maintenance of this module if you are no longer interested.
From: ohiocore [...] gmail.com
Download (untitled) / with headers
text/plain 114b
I posted a repo incorporating patches for this and another ticket on github: https://github.com/atz/Config-Simple


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.