ConfigData->feature() confirms that modules actually load

Closes RT#43557 git-svn-id: http://svn.perl.org/modules/Module-Build/trunk@12865 50811bd7-b8ce-0310-adc1-d9db26280581
FROGGS · Jun 19, 2009 · c3d3dbe · c3d3dbe
1 parent 832d81f
commit c3d3dbe
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/Changes b/Changes
@@ -4,6 +4,8 @@ Revision history for Perl extension Module::Build.
 
  Bug-fixes:
  - Removes Module::Build from its own configure/build_requires
+ - ConfigData->feature() confirms that modules actually load successfully,
+   not just that they are present. (RT#43557)
 
  Other
  - On MSWin32, bumped File::Spec prereq to 3.30 for a variety of fixes

diff --git a/lib/Module/Build/Notes.pm b/lib/Module/Build/Notes.pm
@@ -186,6 +186,7 @@ sub feature {
     while (my ($modname, $spec) = each %%p) {
       my $status = Module::Build->check_installed_status($modname, $spec);
       if ((!$status->{ok}) xor ($type =~ /conflicts$/)) { return 0; }
+      if ( ! eval "require $modname; 1" ) { return 0; }
     }
   }
   return 1;