Author: dagolden
Date: Fri Jun 19 09:31:20 2009
New Revision: 12865

Modified:
   Module-Build/trunk/Changes
   Module-Build/trunk/lib/Module/Build/Notes.pm

Log:
ConfigData->feature() confirms that modules actually load

Closes RT#43557


Modified: Module-Build/trunk/Changes
==============================================================================
--- Module-Build/trunk/Changes  (original)
+++ Module-Build/trunk/Changes  Fri Jun 19 09:31:20 2009
@@ -4,6 +4,8 @@
 
  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

Modified: Module-Build/trunk/lib/Module/Build/Notes.pm
==============================================================================
--- Module-Build/trunk/lib/Module/Build/Notes.pm        (original)
+++ Module-Build/trunk/lib/Module/Build/Notes.pm        Fri Jun 19 09:31:20 2009
@@ -186,6 +186,7 @@
     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;

Reply via email to