Fix [perl #42163] "use base" masks $SIG{__DIE__}
authorRafael Garcia-Suarez <rgarciasuarez@gmail.com>
Mon, 7 May 2007 13:38:24 +0000 (13:38 +0000)
committerRafael Garcia-Suarez <rgarciasuarez@gmail.com>
Mon, 7 May 2007 13:38:24 +0000 (13:38 +0000)
adapted from a patch by Michael G Schwern

p4raw-id: //depot/perl@31163

lib/base.pm

index d8baa95..8bcbb5f 100644 (file)
@@ -2,7 +2,7 @@ package base;
 
 use strict 'vars';
 use vars qw($VERSION);
-$VERSION = '2.08';
+$VERSION = '2.09';
 
 # constant.pm is slow
 sub SUCCESS () { 1 }
@@ -82,19 +82,24 @@ sub import {
              unless defined ${$base.'::VERSION'};
         }
         else {
-            local $SIG{__DIE__};
-            eval "require $base";
-            # Only ignore "Can't locate" errors from our eval require.
-            # Other fatal errors (syntax etc) must be reported.
-            die if $@ && $@ !~ /^Can't locate .*? at \(eval /;
-            unless (%{"$base\::"}) {
-                require Carp;
-                Carp::croak(<<ERROR);
+           my $sigdie;
+           {
+               local $SIG{__DIE__};
+               eval "require $base";
+               # Only ignore "Can't locate" errors from our eval require.
+               # Other fatal errors (syntax etc) must be reported.
+               die if $@ && $@ !~ /^Can't locate .*? at \(eval /;
+               unless (%{"$base\::"}) {
+                   require Carp;
+                   Carp::croak(<<ERROR);
 Base class package "$base" is empty.
     (Perhaps you need to 'use' the module which defines that package first.)
 ERROR
-
-            }
+               }
+               $sigdie = $SIG{__DIE__};
+           }
+           # Make sure a global $SIG{__DIE__} makes it out of the localization.
+           $SIG{__DIE__} = $sigdie if defined $sigdie;
             ${$base.'::VERSION'} = "-1, set by base.pm"
               unless defined ${$base.'::VERSION'};
         }