X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FObject%2FRemote%2FModuleLoader.pm;h=79ae8b187d699a4f25b2d417de30c59a4baa3305;hb=f1c7aaa88c44996c7e15256021c07aaea1c6dd0e;hp=8b2e6c27156bcf50d357545d8b31b1d9b526dbb5;hpb=37efeb68748dd589455d8ae58d39e74700abdd67;p=scpubgit%2FObject-Remote.git

diff --git a/lib/Object/Remote/ModuleLoader.pm b/lib/Object/Remote/ModuleLoader.pm
index 8b2e6c2..79ae8b1 100644
--- a/lib/Object/Remote/ModuleLoader.pm
+++ b/lib/Object/Remote/ModuleLoader.pm
@@ -3,18 +3,49 @@ package Object::Remote::ModuleLoader;
 BEGIN {
   package Object::Remote::ModuleLoader::Hook;
   use Moo;
-  use Object::Remote::Logging qw(:log);
+  use Object::Remote::Logging qw( :log :dlog );
   has sender => (is => 'ro', required => 1);
 
   # unqualified INC forced into package main
   sub Object::Remote::ModuleLoader::Hook::INC {
     my ($self, $module) = @_;
     log_debug { "Loading $module via " . ref($self) };
-    if (my $code = $self->sender->source_for($module)) {
-      open my $fh, '<', \$code;
-      return $fh;
+    my $ret = eval
+    {
+      if (my $code = $self->sender->source_for($module)) {
+        open my $fh, '<', \$code;
+        Dlog_trace { "Module sender successfully sent code for '$module': $code" } $code;
+        return $fh;
+      }
+      log_trace { "Module sender did not return code for '$module'" };
+      return;
+    };
+    if($@)
+    {
+      log_trace { "Module sender blew up - $@" };
+      if($@ =~ /Can't locate/)
+      {
+        # Fudge the error messge to make it work with
+        # Module::Runtime use_package_optimistically
+        # Module::Runtime wants - /\ACan't locate \Q$fn\E .+ at \Q@{[__FILE__]}\E line/
+        # the callstack range is a bit of a guess, we want
+        # to make a reasonable effort, but not spend forever.
+        for(my $i = 4; $i < 20; $i++)
+        {
+            my ($package, $file, $line) = caller($i);
+            last unless $package;
+            if($package eq 'Module::Runtime')
+            {
+                # we want to fill in the error message with the 
+                # module runtime module call info.
+                $@ =~ s/(in \@INC.)/$1 at $file line $line/;
+                last;
+            }
+        }
+      }
+      die $@;
     }
-    return;
+    return $ret;
   }
 }
 
@@ -31,7 +62,7 @@ sub _build_inc_hook {
   log_debug { "Constructing module builder hook" };
   my $hook = Object::Remote::ModuleLoader::Hook->new(sender => $self->module_sender);
   log_trace { "Done constructing module builder hook" };
-  return $hook;   
+  return $hook;
 }
 
 sub BUILD { shift->enable }