# from dagol...@cvs.perl.org
# on Tuesday 25 August 2009 13:30:

>Log:
>Fix t/destinations.t segfault on 5.6.2

>-    local $mb->install_sets->{site} = \%test_config;
>+    my $old =  $mb->install_sets->{site} = \%test_config;
...
>+    $mb->install_sets->{site} = $old;

Hi David,

Is that really what you intended?  The log sounds like you wanted to 
workaround a local() bug, but this code changes the value of site 
rather than restoring it.

--Eric
-- 
"Everything should be made as simple as possible, but no simpler."
--Albert Einstein
---------------------------------------------------
    http://scratchcomputing.com
---------------------------------------------------

Reply via email to