[svn:Module-Build] r12865 - in Module-Build/trunk: . lib/Module/Build

[email protected]
Newsgroups perl.module.build.checkins
Message-ID <[email protected]>
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;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.