Merge 'trunk' into 'dbicadmin_pod'

diff --combined Makefile.PL

index d5f0f1c,a8c2065..f01351d
--- 1/Makefile.PL
--- 2/Makefile.PL
+++ b/Makefile.PL
@@@ -8,11 -8,6 +8,11 @@@ use 5.008001
  use FindBin;
  use lib "$FindBin::Bin/lib";
  
+ +# adjust ENV for $AUTHOR system() calls
+ +use Config;
+ +$ENV{PERL5LIB} = join ($Config{path_sep}, @INC);
+ +
+ +
  ###
  ### DO NOT ADD OPTIONAL DEPENDENCIES HERE, EVEN AS recommends()
  ### All of them should go to DBIx::Class::Optional::Dependencies
@@@ -30,19 -25,12 +30,12 @@@ my $build_requires = 
  my $test_requires = {
    'File::Temp'               => '0.22',
    'Test::Builder'            => '0.33',
-   'Test::Deep'               => '0',
    'Test::Exception'          => '0',
    'Test::More'               => '0.92',
    'Test::Warn'               => '0.21',
  };
  
  my $runtime_requires = {
-   # Core
-   'List::Util'               => '0',
-   'Scalar::Util'             => '0',
-   'Storable'                 => '0',
- 
-   # Dependencies
    'Carp::Clan'               => '6.0',
    'Class::Accessor::Grouped' => '0.09002',
    'Class::C3::Componentised' => '1.0005',
@@@ -52,7 -40,6 +45,6 @@@
    'MRO::Compat'              => '0.09',
    'Module::Find'             => '0.06',
    'Path::Class'              => '0.18',
-   'Scope::Guard'             => '0.03',
    'SQL::Abstract'            => '1.61',
    'SQL::Abstract::Limit'     => '0.13',
    'Sub::Name'                => '0.04',
@@@ -67,7 -54,7 +59,7 @@@ my $reqs = 
    test_requires => { %$test_requires },
  };
  
- -# re-build README and require extra modules for testing if we're in a checkout
+ +# autogenerate docs and require extra modules for testing if we're in a checkout
  if ($Module::Install::AUTHOR) {
  
    print "Regenerating README\n";
@@@ -78,9 -65,6 +70,9 @@@
      unlink 'MANIFEST';
    }
  
+ +  print "Regenerating dbicadmin.pod\n";
+ +  system('perl script/dbicadmin --pod > lib/dbicadmin.pod');
+ +
    print "Regenerating Optional/Dependencies.pod\n";
    require DBIx::Class::Optional::Dependencies;
    DBIx::Class::Optional::Dependencies->_gen_pod;
@@@ -146,7 -130,6 +138,6 @@@ no_index directory => $_ for (qw
    lib/DBIx/Class/PK/Auto
  |);
  no_index package => $_ for (qw/
-   DBIx::Class::Storage::DBI::AmbiguousGlob
    DBIx::Class::SQLAHacks DBIx::Class::Storage::DBIHacks
  /);
  
diff --combined t/03podcoverage.t

index fabdd0c,d3a5374..3115234
--- 1/t/03podcoverage.t
--- 2/t/03podcoverage.t
+++ b/t/03podcoverage.t
@@@ -6,24 -6,17 +6,17 @@@ use List::Util ()
  use lib qw(t/lib);
  use DBICTest;
  
- my @MODULES = (
-   'Test::Pod::Coverage 1.08',
-   'Pod::Coverage 0.20',
- );
- 
  # Don't run tests for installs
  unless ( DBICTest::AuthorCheck->is_author || $ENV{AUTOMATED_TESTING} || $ENV{RELEASE_TESTING} ) {
    plan( skip_all => "Author tests not required for installation" );
  }
  
- # Load the testing modules
- foreach my $MODULE ( @MODULES ) {
-   eval "use $MODULE";
-   if ( $@ ) {
-     $ENV{RELEASE_TESTING}
-     ? die( "Failed to load required release-testing module $MODULE" )
-     : plan( skip_all => "$MODULE not available for testing" );
-   }
+ require DBIx::Class;
+ unless ( DBIx::Class::Optional::Dependencies->req_ok_for ('test_podcoverage') ) {
+   my $missing = DBIx::Class::Optional::Dependencies->req_missing_for ('test_podcoverage');
+   $ENV{RELEASE_TESTING} || DBICTest::AuthorCheck->is_author
+     ? die ("Failed to load release-testing module requirements: $missing")
+     : plan skip_all => "Test needs: $missing"
  }
  
  # Since this is about checking documentation, a little documentation
@@@ -93,7 -86,7 +86,7 @@@ my $exceptions = 
          /]
      },
  
- -    'DBIx::Class::Admin::Types'                     => { skip => 1 },
+ +    'DBIx::Class::Admin::*'                         => { skip => 1 },
      'DBIx::Class::ClassResolver::PassThrough'       => { skip => 1 },
      'DBIx::Class::Componentised'                    => { skip => 1 },
      'DBIx::Class::Relationship::*'                  => { skip => 1 },
@@@ -148,7 -141,7 +141,7 @@@ foreach my $module (@modules) 
          if exists($ex->{ignore});
  
      # run the test with the potentially modified parm set
-     pod_coverage_ok($module, $parms, "$module POD coverage");
+     Test::Pod::Coverage::pod_coverage_ok($module, $parms, "$module POD coverage");
    }
  }
		1	2
Makefile.PL	patch \|	diff1 \|	diff2 \|	blob \| history
t/03podcoverage.t	patch \|	diff1 \|	diff2 \|	blob \| history