Stop removing entries from %INC to load code

[toddr]

We rejected #79 and took #80 because #79 is a behavior change. We still think it might be worth doing but it should be addressed as its own issue.

[toddr]

@xsawyerx ^^

[xsawyerx]

I don't have a preference here. As noted in one of the aforementioned tickets, the entirety of the work was done by @kentfredric.

[toddr]

The fix in question would be something like this:

diff --git a/lib/Template/Provider.pm b/lib/Template/Provider.pm
index 6ecb2453..3a75a29b 100644
--- a/lib/Template/Provider.pm
+++ b/lib/Template/Provider.pm
@@ -562,13 +562,9 @@ sub _compiled_filename {
 
 sub _load_compiled {
     my ($self, $file) = @_;
-    my $compiled;
 
-    # load compiled template via require();  we zap any
-    # %INC entry to ensure it is reloaded (we don't
-    # want 1 returned by require() to say it's in memory)
-    delete $INC{ $file };
-    eval { $compiled = require $file; };
+    my $fpath = File::Spec->rel2abs($file);
+    my $compiled = do $fpath;
     return $@
         ? $self->error("compiled template $compiled: $@")
         : $compiled;