Introduce basic optdeps group inclusion mechanism, simplify many declarations
[dbsrgits/DBIx-Class.git] / lib / DBIx / Class / Optional / Dependencies.pm
index 6870714..fcf1deb 100644 (file)
@@ -11,39 +11,44 @@ use Carp;
 # POD is generated automatically by calling _gen_pod from the
 # Makefile.PL in $AUTHOR mode
 
-my $moose_basic = {
-  'Moose'                      => '0.98',
-  'MooseX::Types'              => '0.21',
+# NOTE: the rationale for 2 JSON::Any versions is that
+# we need the newer only to work around JSON::XS, which
+# itself is an optional dep
+my $min_json_any = {
+  'JSON::Any'                     => '1.23',
 };
-
-my $admin_basic = {
-  %$moose_basic,
-  'MooseX::Types::Path::Class' => '0.05',
-  'MooseX::Types::JSON'        => '0.02',
-  'namespace::autoclean'       => '0.09',
-  'parent'                     => '0.223',
+my $test_and_dist_json_any = {
+  'JSON::Any'                     => '1.31',
 };
 
-my $reqs = {
-  dist => {
-    #'Module::Install::Pod::Inherit' => '0.01',
-  },
+my $moose_basic = {
+  'Moose'                         => '0.98',
+  'MooseX::Types'                 => '0.21',
+  'MooseX::Types::LoadableClass'  => '0.011',
+};
 
+my $dbic_reqs = {
   replicated => {
-    req => {
-      %$moose_basic,
-      'namespace::clean'          => '0.11',
-      'Hash::Merge'               => '0.11',
-    },
+    req => $moose_basic,
     pod => {
       title => 'Storage::Replicated',
       desc => 'Modules required for L<DBIx::Class::Storage::DBI::Replicated>',
     },
   },
 
+  test_replicated => {
+    include => 'replicated',
+    req => {
+      'Test::Moose' => '0',
+    },
+  },
+
   admin => {
     req => {
-      %$admin_basic,
+      %$moose_basic,
+      %$min_json_any,
+      'MooseX::Types::Path::Class' => '0.05',
+      'MooseX::Types::JSON' => '0.02',
     },
     pod => {
       title => 'DBIx::Class::Admin',
@@ -52,12 +57,10 @@ my $reqs = {
   },
 
   admin_script => {
+    include => 'admin',
     req => {
-      %$moose_basic,
-      %$admin_basic,
       'Getopt::Long::Descriptive' => '0.081',
-      'JSON::Any'                 => '1.22',
-      'Text::CSV'                 => '1.16',
+      'Text::CSV' => '1.16',
     },
     pod => {
       title => 'dbicadmin',
@@ -67,193 +70,792 @@ my $reqs = {
 
   deploy => {
     req => {
-      'SQL::Translator'           => '0.11002',
+      'SQL::Translator'           => '0.11018',
     },
     pod => {
       title => 'Storage::DBI::deploy()',
-      desc => 'Modules required for L<DBIx::Class::Storage::DBI/deploy> and L<DBIx::Class::Storage::DBI/deploymen_statements>',
+      desc => 'Modules required for L<DBIx::Class::Storage::DBI/deployment_statements> and L<DBIx::Class::Schema/deploy>',
+    },
+  },
+
+  id_shortener => {
+    req => {
+      'Math::BigInt' => '1.80',
+      'Math::Base36' => '0.07',
     },
   },
 
-  author => {
+  test_component_accessor => {
+    req => {
+      'Class::Unload'             => '0.07',
+    },
+  },
+
+  test_pod => {
+    req => {
+      'Test::Pod'                 => '1.42',
+    },
+  },
+
+  test_podcoverage => {
     req => {
-      'Test::Pod'                 => '1.26',
       'Test::Pod::Coverage'       => '1.08',
       'Pod::Coverage'             => '0.20',
-      #'Test::NoTabs'              => '0.9',
-      #'Test::EOL'                 => '0.6',
     },
   },
 
-  core => {
+  test_whitespace => {
+    req => {
+      'Test::EOL'                 => '1.0',
+      'Test::NoTabs'              => '0.9',
+    },
+  },
+
+  test_strictures => {
+    req => {
+      'Test::Strict'              => '0.20',
+    },
+  },
+
+  test_prettydebug => {
+    req => $min_json_any,
+  },
+
+  test_admin_script => {
+    include => 'admin_script',
+    req => {
+      %$test_and_dist_json_any,
+      'JSON' => 0,
+      'JSON::PP' => 0,
+      'Cpanel::JSON::XS' => 0,
+      'JSON::XS' => 0,
+      $^O eq 'MSWin32'
+        # for t/admin/10script.t
+        ? ('Win32::ShellQuote' => 0)
+        # DWIW does not compile (./configure even) on win32
+        : ('JSON::DWIW' => 0 )
+      ,
+    }
+  },
+
+  test_leaks_heavy => {
     req => {
-      # t/52cycle.t
-      'Test::Memory::Cycle'       => '0',
-      'Devel::Cycle'              => '1.10',
+      'Class::MethodCache' => '0.02',
+      'PadWalker' => '1.06',
+    },
+  },
+
+  test_dt => {
+    req => {
+      'DateTime'                    => '0.55',
+      'DateTime::Format::Strptime'  => '1.2',
+    },
+  },
 
+  test_dt_sqlite => {
+    include => 'test_dt',
+    req => {
       # t/36datetime.t
       # t/60core.t
       'DateTime::Format::SQLite'  => '0',
+    },
+  },
+
+  test_dt_mysql => {
+    include => 'test_dt',
+    req => {
+      # t/inflate/datetime_mysql.t
+      # (doesn't need Mysql itself)
+      'DateTime::Format::MySQL'   => '0',
+    },
+  },
 
-      # t/96_is_deteministic_value.t
-      'DateTime::Format::Strptime'=> '0',
+  test_dt_pg => {
+    include => 'test_dt',
+    req => {
+      # t/inflate/datetime_pg.t
+      # (doesn't need PG itself)
+      'DateTime::Format::Pg'      => '0.16004',
     },
   },
 
-  cdbicompat => {
+  test_cdbicompat => {
+    include => 'test_dt',
     req => {
-      'DBIx::ContextualFetch'     => '0',
       'Class::DBI::Plugin::DeepAbstractSearch' => '0',
-      'Class::Trigger'            => '0',
       'Time::Piece::MySQL'        => '0',
-      'Clone'                     => '0',
       'Date::Simple'              => '3.03',
     },
   },
 
+  rdbms_generic_odbc => {
+    req => {
+      'DBD::ODBC' => 0,
+    }
+  },
+
+  rdbms_generic_ado => {
+    req => {
+      'DBD::ADO' => 0,
+    }
+  },
+
+  # this is just for completeness as SQLite
+  # is a core dep of DBIC for testing
+  rdbms_sqlite => {
+    req => {
+      'DBD::SQLite' => 0,
+    },
+    pod => {
+      title => 'SQLite support',
+      desc => 'Modules required to connect to SQLite',
+    },
+  },
+
   rdbms_pg => {
     req => {
-      $ENV{DBICTEST_PG_DSN}
-        ? (
-          'Sys::SigAction'        => '0',
-          'DBD::Pg'               => '2.009002',
-          'DateTime::Format::Pg'  => '0',
-        ) : ()
+      # when changing this list make sure to adjust xt/optional_deps.t
+      'DBD::Pg' => 0,
+    },
+    pod => {
+      title => 'PostgreSQL support',
+      desc => 'Modules required to connect to PostgreSQL',
+    },
+  },
+
+  rdbms_mssql_odbc => {
+    include => 'rdbms_generic_odbc',
+    pod => {
+      title => 'MSSQL support via DBD::ODBC',
+      desc => 'Modules required to connect to MSSQL via DBD::ODBC',
+    },
+  },
+
+  rdbms_mssql_sybase => {
+    req => {
+      'DBD::Sybase' => 0,
+    },
+    pod => {
+      title => 'MSSQL support via DBD::Sybase',
+      desc => 'Modules required to connect to MSSQL via DBD::Sybase',
+    },
+  },
+
+  rdbms_mssql_ado => {
+    include => 'rdbms_generic_ado',
+    pod => {
+      title => 'MSSQL support via DBD::ADO (Windows only)',
+      desc => 'Modules required to connect to MSSQL via DBD::ADO. This particular DBD is available on Windows only',
+    },
+  },
+
+  rdbms_msaccess_odbc => {
+    include => 'rdbms_generic_odbc',
+    pod => {
+      title => 'MS Access support via DBD::ODBC',
+      desc => 'Modules required to connect to MS Access via DBD::ODBC',
+    },
+  },
+
+  rdbms_msaccess_ado => {
+    include => 'rdbms_generic_ado',
+    pod => {
+      title => 'MS Access support via DBD::ADO (Windows only)',
+      desc => 'Modules required to connect to MS Access via DBD::ADO. This particular DBD is available on Windows only',
     },
   },
 
   rdbms_mysql => {
     req => {
-      $ENV{DBICTEST_MYSQL_DSN}
-        ? (
-          'DateTime::Format::MySQL' => '0',
-          'DBD::mysql'              => '0',
-        ) : ()
+      'DBD::mysql' => 0,
+    },
+    pod => {
+      title => 'MySQL support',
+      desc => 'Modules required to connect to MySQL',
     },
   },
 
   rdbms_oracle => {
+    include => 'id_shortener',
     req => {
-      $ENV{DBICTEST_ORA_DSN}
-        ? (
-          'DateTime::Format::Oracle' => '0',
-        ) : ()
+      'DBD::Oracle' => 0,
+    },
+    pod => {
+      title => 'Oracle support',
+      desc => 'Modules required to connect to Oracle',
     },
   },
 
   rdbms_ase => {
     req => {
-      $ENV{DBICTEST_SYBASE_DSN}
-        ? (
-          'DateTime::Format::Sybase' => 0,
-        ) : ()
+      'DBD::Sybase' => 0,
+    },
+    pod => {
+      title => 'Sybase ASE support',
+      desc => 'Modules required to connect to Sybase ASE',
     },
   },
 
-  rdbms_asa => {
+  rdbms_db2 => {
     req => {
-      grep $_, @ENV{qw/DBICTEST_SYBASE_ASA_DSN DBICTEST_SYBASE_ASA_ODBC_DSN/}
-        ? (
-          'DateTime::Format::Strptime' => 0,
-        ) : ()
+      'DBD::DB2' => 0,
+    },
+    pod => {
+      title => 'DB2 support',
+      desc => 'Modules required to connect to DB2',
     },
   },
-};
 
+  rdbms_db2_400 => {
+    include => 'rdbms_generic_odbc',
+    pod => {
+      title => 'DB2 on AS/400 support',
+      desc => 'Modules required to connect to DB2 on AS/400',
+    },
+  },
 
-sub _all_optional_requirements {
-  return { map { %{ $reqs->{$_}{req} || {} } } (keys %$reqs) };
-}
+  rdbms_informix => {
+    req => {
+      'DBD::Informix' => 0,
+    },
+    pod => {
+      title => 'Informix support',
+      desc => 'Modules required to connect to Informix',
+    },
+  },
 
-sub req_list_for {
-  my ($class, $group) = @_;
+  rdbms_sqlanywhere => {
+    req => {
+      'DBD::SQLAnywhere' => 0,
+    },
+    pod => {
+      title => 'SQLAnywhere support',
+      desc => 'Modules required to connect to SQLAnywhere',
+    },
+  },
 
-  croak "req_list_for() expects a requirement group name"
-    unless $group;
+  rdbms_sqlanywhere_odbc => {
+    include => 'rdbms_generic_odbc',
+    pod => {
+      title => 'SQLAnywhere support via DBD::ODBC',
+      desc => 'Modules required to connect to SQLAnywhere via DBD::ODBC',
+    },
+  },
 
-  my $deps = $reqs->{$group}{req}
-    or croak "Requirement group '$group' does not exist";
+  rdbms_firebird => {
+    req => {
+      'DBD::Firebird' => 0,
+    },
+    pod => {
+      title => 'Firebird support',
+      desc => 'Modules required to connect to Firebird',
+    },
+  },
 
-  return { %$deps };
-}
+  rdbms_firebird_interbase => {
+    req => {
+      'DBD::InterBase' => 0,
+    },
+    pod => {
+      title => 'Firebird support via DBD::InterBase',
+      desc => 'Modules required to connect to Firebird via DBD::InterBase',
+    },
+  },
 
+  rdbms_firebird_odbc => {
+    include => 'rdbms_generic_odbc',
+    pod => {
+      title => 'Firebird support via DBD::ODBC',
+      desc => 'Modules required to connect to Firebird via DBD::ODBC',
+    },
+  },
+
+  test_rdbms_pg => {
+    include => 'rdbms_pg',
+    env => [
+      DBICTEST_PG_DSN => 1,
+      DBICTEST_PG_USER => 0,
+      DBICTEST_PG_PASS => 0,
+    ],
+    req => {
+      # the order does matter because the rdbms support group might require
+      # a different version that the test group
+      #
+      # when changing this list make sure to adjust xt/optional_deps.t
+      'DBD::Pg' => '2.009002',  # specific version to test bytea
+    },
+  },
+
+  test_rdbms_mssql_odbc => {
+    include => 'rdbms_mssql_odbc',
+    env => [
+      DBICTEST_MSSQL_ODBC_DSN => 1,
+      DBICTEST_MSSQL_ODBC_USER => 0,
+      DBICTEST_MSSQL_ODBC_PASS => 0,
+    ],
+  },
+
+  test_rdbms_mssql_ado => {
+    include => 'rdbms_mssql_ado',
+    env => [
+      DBICTEST_MSSQL_ADO_DSN => 1,
+      DBICTEST_MSSQL_ADO_USER => 0,
+      DBICTEST_MSSQL_ADO_PASS => 0,
+    ],
+  },
+
+  test_rdbms_mssql_sybase => {
+    include => 'rdbms_mssql_sybase',
+    env => [
+      DBICTEST_MSSQL_DSN => 1,
+      DBICTEST_MSSQL_USER => 0,
+      DBICTEST_MSSQL_PASS => 0,
+    ],
+  },
+
+  test_rdbms_msaccess_odbc => {
+    include => [qw(rdbms_msaccess_odbc test_dt)],
+    env => [
+      DBICTEST_MSACCESS_ODBC_DSN => 1,
+      DBICTEST_MSACCESS_ODBC_USER => 0,
+      DBICTEST_MSACCESS_ODBC_PASS => 0,
+    ],
+    req => {
+      'Data::GUID' => '0',
+    },
+  },
+
+  test_rdbms_msaccess_ado => {
+    include => [qw(rdbms_msaccess_ado test_dt)],
+    env => [
+      DBICTEST_MSACCESS_ADO_DSN => 1,
+      DBICTEST_MSACCESS_ADO_USER => 0,
+      DBICTEST_MSACCESS_ADO_PASS => 0,
+    ],
+    req => {
+      'Data::GUID' => 0,
+    },
+  },
+
+  test_rdbms_mysql => {
+    include => 'rdbms_mysql',
+    env => [
+      DBICTEST_MYSQL_DSN => 1,
+      DBICTEST_MYSQL_USER => 0,
+      DBICTEST_MYSQL_PASS => 0,
+    ],
+  },
+
+  test_rdbms_oracle => {
+    include => 'rdbms_oracle',
+    env => [
+      DBICTEST_ORA_DSN => 1,
+      DBICTEST_ORA_USER => 0,
+      DBICTEST_ORA_PASS => 0,
+    ],
+    req => {
+      'DateTime::Format::Oracle' => '0',
+      'DBD::Oracle'              => '1.24',
+    },
+  },
+
+  test_rdbms_ase => {
+    include => 'rdbms_ase',
+    env => [
+      DBICTEST_SYBASE_DSN => 1,
+      DBICTEST_SYBASE_USER => 0,
+      DBICTEST_SYBASE_PASS => 0,
+    ],
+  },
+
+  test_rdbms_db2 => {
+    include => 'rdbms_db2',
+    env => [
+      DBICTEST_DB2_DSN => 1,
+      DBICTEST_DB2_USER => 0,
+      DBICTEST_DB2_PASS => 0,
+    ],
+  },
+
+  test_rdbms_db2_400 => {
+    include => 'rdbms_db2_400',
+    env => [
+      DBICTEST_DB2_400_DSN => 1,
+      DBICTEST_DB2_400_USER => 0,
+      DBICTEST_DB2_400_PASS => 0,
+    ],
+  },
+
+  test_rdbms_informix => {
+    include => 'rdbms_informix',
+    env => [
+      DBICTEST_INFORMIX_DSN => 1,
+      DBICTEST_INFORMIX_USER => 0,
+      DBICTEST_INFORMIX_PASS => 0,
+    ],
+  },
+
+  test_rdbms_sqlanywhere => {
+    include => 'rdbms_sqlanywhere',
+    env => [
+      DBICTEST_SQLANYWHERE_DSN => 1,
+      DBICTEST_SQLANYWHERE_USER => 0,
+      DBICTEST_SQLANYWHERE_PASS => 0,
+    ],
+  },
+
+  test_rdbms_sqlanywhere_odbc => {
+    include => 'rdbms_sqlanywhere_odbc',
+    env => [
+      DBICTEST_SQLANYWHERE_ODBC_DSN => 1,
+      DBICTEST_SQLANYWHERE_ODBC_USER => 0,
+      DBICTEST_SQLANYWHERE_ODBC_PASS => 0,
+    ],
+  },
+
+  test_rdbms_firebird => {
+    include => 'rdbms_firebird',
+    env => [
+      DBICTEST_FIREBIRD_DSN => 1,
+      DBICTEST_FIREBIRD_USER => 0,
+      DBICTEST_FIREBIRD_PASS => 0,
+    ],
+  },
+
+  test_rdbms_firebird_interbase => {
+    include => 'rdbms_firebird_interbase',
+    env => [
+      DBICTEST_FIREBIRD_INTERBASE_DSN => 1,
+      DBICTEST_FIREBIRD_INTERBASE_USER => 0,
+      DBICTEST_FIREBIRD_INTERBASE_PASS => 0,
+    ],
+  },
+
+  test_rdbms_firebird_odbc => {
+    include => 'rdbms_firebird_odbc',
+    env => [
+      DBICTEST_FIREBIRD_ODBC_DSN => 1,
+      DBICTEST_FIREBIRD_ODBC_USER => 0,
+      DBICTEST_FIREBIRD_ODBC_PASS => 0,
+    ],
+  },
+
+  test_memcached => {
+    env => [
+      DBICTEST_MEMCACHED => 1,
+    ],
+    req => {
+      'Cache::Memcached' => 0,
+    },
+  },
+
+  dist_dir => {
+    req => {
+      %$test_and_dist_json_any,
+      'ExtUtils::MakeMaker' => '6.64',
+      'Pod::Inherit'        => '0.91',
+    },
+  },
+
+  dist_upload => {
+    req => {
+      'CPAN::Uploader' => '0.103001',
+    },
+  },
+};
 
-our %req_availability_cache;
-sub req_ok_for {
-  my ($class, $group) = @_;
 
-  croak "req_ok_for() expects a requirement group name"
-    unless $group;
 
-  $class->_check_deps ($group) unless $req_availability_cache{$group};
+### Public API
 
-  return $req_availability_cache{$group}{status};
+# OO for (mistakenly considered) ease of extensibility, not due to any need to
+# carry state of any sort. This API is currently used outside, so leave as-is.
+# FIXME - make sure to not propagate this further if module is extracted as a
+# standalone library - keep the stupidity to a DBIC-secific shim!
+#
+sub req_list_for {
+  shift->_groups_to_reqs(@_)->{effective_modreqs};
+}
+
+sub modreq_list_for {
+  shift->_groups_to_reqs(@_)->{modreqs};
+}
+
+sub req_group_list {
+  +{ map
+    { $_ => $_[0]->_groups_to_reqs($_) }
+    keys %$dbic_reqs
+  }
+}
+
+sub req_errorlist_for { shift->modreq_errorlist_for(@_) }  # deprecated
+sub modreq_errorlist_for {
+  my $self = shift;
+  $self->_errorlist_for_modreqs( $self->_groups_to_reqs(@_)->{modreqs} );
+}
+
+sub req_ok_for {
+  shift->req_missing_for(@_) ? 0 : 1;
 }
 
 sub req_missing_for {
-  my ($class, $group) = @_;
+  my $self = shift;
+
+  my $reqs = $self->_groups_to_reqs(@_);
+  my $mods_missing = $self->modreq_missing_for(@_);
+
+  return '' if
+    ! $mods_missing
+      and
+    ! $reqs->{missing_envvars}
+  ;
 
-  croak "req_missing_for() expects a requirement group name"
-    unless $group;
+  my @res = $mods_missing || ();
 
-  $class->_check_deps ($group) unless $req_availability_cache{$group};
+  push @res, 'the following group(s) of environment variables: ' . join ' and ', map
+    { __envvar_group_desc($_) }
+    @{$reqs->{missing_envvars}}
+  if $reqs->{missing_envvars};
 
-  return $req_availability_cache{$group}{missing};
+  return (
+    ( join ' as well as ', @res )
+      .
+    ( $reqs->{modreqs_fully_documented} ? " (see @{[ ref $self || $self ]} documentation for details)" : '' ),
+  );
 }
 
-sub req_errorlist_for {
-  my ($class, $group) = @_;
+sub modreq_missing_for {
+  my $self = shift;
 
-  croak "req_errorlist_for() expects a requirement group name"
-    unless $group;
+  my $reqs = $self->_groups_to_reqs(@_);
+  my $modreq_errors = $self->_errorlist_for_modreqs($reqs->{modreqs})
+    or return '';
 
-  $class->_check_deps ($group) unless $req_availability_cache{$group};
+  join ' ', map
+    { $reqs->{modreqs}{$_} ? qq("$_~>=$reqs->{modreqs}{$_}") : $_ }
+    sort { lc($a) cmp lc($b) } keys %$modreq_errors
+  ;
+}
 
-  return $req_availability_cache{$group}{errorlist};
+sub die_unless_req_ok_for {
+  if (my $err = shift->req_missing_for(@_) ) {
+    die "Unable to continue due to missing requirements: $err\n";
+  }
 }
 
-sub _check_deps {
-  my ($class, $group) = @_;
 
-  my $deps = $class->req_list_for ($group);
 
-  my %errors;
-  for my $mod (keys %$deps) {
-    if (my $ver = $deps->{$mod}) {
-      eval "use $mod $ver ()";
+### Private functions
+
+# potentially shorten group desc
+sub __envvar_group_desc {
+  my @envs = @{$_[0]};
+
+  my (@res, $last_prefix);
+  while (my $ev = shift @envs) {
+    my ($pref, $sep, $suff) = split / ([\_\-]) (?= [^\_\-]+ \z )/x, $ev;
+
+    if ( defined $sep and ($last_prefix||'') eq $pref ) {
+        push @res, "...${sep}${suff}"
     }
     else {
-      eval "require $mod";
+      push @res, $ev;
     }
 
-    $errors{$mod} = $@ if $@;
+    $last_prefix = $pref if $sep;
   }
 
-  if (keys %errors) {
-    my $missing = join (', ', map { $deps->{$_} ? "$_ >= $deps->{$_}" : $_ } (sort keys %errors) );
-    $missing .= " (see $class for details)" if $reqs->{$group}{pod};
-    $req_availability_cache{$group} = {
-      status => 0,
-      errorlist => { %errors },
-      missing => $missing,
-    };
+  join '/', @res;
+}
+
+
+
+### Private OO API
+our %req_unavailability_cache;
+
+# this method is just a lister and envvar/metadata checker - it does not try to load anything
+my $processed_groups = {};
+sub _groups_to_reqs {
+  my ($self, $groups) = @_;
+
+  $groups = [ $groups || () ]
+    unless ref $groups eq 'ARRAY';
+
+  croak "@{[ (caller(1))[3] ]}() expects a requirement group name or arrayref of group names"
+    unless @$groups;
+
+  my $ret = {
+    modreqs => {},
+    modreqs_fully_documented => 1,
+  };
+
+  for my $group ( grep { ! $processed_groups->{$_} } @$groups ) {
+
+    $group =~ /\A [A-Za-z][0-9A-Z_a-z]* \z/x
+      or croak "Invalid requirement group name '$group': only ascii alphanumerics and _ are allowed";
+
+    croak "Requirement group '$group' is not defined" unless defined $dbic_reqs->{$group};
+
+    my $group_reqs = $dbic_reqs->{$group}{req};
+
+    # sanity-check
+    for (keys %$group_reqs) {
+
+      $_ =~ /\A [A-Z_a-z][0-9A-Z_a-z]* (?:::[0-9A-Z_a-z]+)* \z /x
+        or croak "Requirement '$_' in group '$group' is not a valid module name";
+
+      # !!!DO NOT CHANGE!!!
+      # remember - version.pm may not be available on the system
+      croak "Requirement '$_' in group '$group' specifies an invalid version '$group_reqs->{$_}' (only plain non-underscored floating point decimals are supported)"
+        if ( ($group_reqs->{$_}||0) !~ / \A [0-9]+ (?: \. [0-9]+ )? \z /x );
+    }
+
+    # check if we have all required envvars if such names are defined
+    my ($some_envs_required, $some_envs_missing);
+    if (my @e = @{$dbic_reqs->{$group}{env} || [] }) {
+
+      croak "Unexpected 'env' attribute under group '$group' (only allowed in test_* groups)"
+        unless $group =~ /^test_/;
+
+      croak "Unexpected *odd* list in 'env' under group '$group'"
+        if @e % 2;
+
+      my @group_envnames_list;
+
+      # deconstruct the whole thing
+      while (@e) {
+        push @group_envnames_list, my $envname = shift @e;
+
+        # env required or not
+        next unless shift @e;
+
+        $some_envs_required ||= 1;
+
+        $some_envs_missing ||= (
+          ! defined $ENV{$envname}
+            or
+          ! length $ENV{$envname}
+        );
+      }
+
+      croak "None of the envvars in group '$group' declared as required, making the requirement moot"
+        unless $some_envs_required;
+
+      push @{$ret->{missing_envvars}}, \@group_envnames_list if $some_envs_missing;
+    }
+
+    # get the reqs for includes if any
+    my $inc_reqs;
+    if (my $incs = $dbic_reqs->{$group}{include}) {
+      $incs = [ $incs ] unless ref $incs eq 'ARRAY';
+
+      croak "Malformed 'include' for group '$group': must be another existing group name or arrayref of existing group names"
+        unless @$incs;
+
+      local $processed_groups->{$group} = 1;
+
+      my $subreqs = $self->_groups_to_reqs($incs);
+
+      croak "Includes with variable effective_modreqs not yet supported"
+        if $subreqs->{effective_modreqs_differ};
+
+      $inc_reqs = $subreqs->{modreqs};
+
+    }
+
+    # assemble into the final ret
+    for my $type (
+      'modreqs',
+      $some_envs_missing ? () : 'effective_modreqs'
+    ) {
+      for my $req_bag ($group_reqs, $inc_reqs||()) {
+        for my $mod (keys %$req_bag) {
+
+          $ret->{$type}{$mod} = $req_bag->{$mod}||0 if (
+
+            ! exists $ret->{$type}{$mod}
+              or
+            # we sanitized the version to be numeric above - we can just -gt it
+            ($req_bag->{$mod}||0) > $ret->{$type}{$mod}
+
+          );
+        }
+      }
+    }
+
+    $ret->{effective_modreqs_differ} ||= !!$some_envs_missing;
+
+    $ret->{modreqs_fully_documented} &&= !!$dbic_reqs->{$group}{pod};
   }
-  else {
-    $req_availability_cache{$group} = {
-      status => 1,
-      errorlist => {},
-      missing => '',
-    };
+
+  return $ret;
+}
+
+
+# this method tries to load specified modreqs and returns a hashref of
+# module/loaderror pairs for anything that failed
+sub _errorlist_for_modreqs {
+  # args supposedly already went through _groups_to_reqs and are therefore sanitized
+  # safe to eval at will
+  my ($self, $reqs) = @_;
+
+  my $ret;
+
+  for my $m ( keys %$reqs ) {
+    my $v = $reqs->{$m};
+
+    if (! exists $req_unavailability_cache{$m}{$v} ) {
+      local $@;
+      eval( "require $m;" . ( $v ? "$m->VERSION(q($v))" : '' ) );
+      $req_unavailability_cache{$m}{$v} = $@;
+    }
+
+    $ret->{$m} = $req_unavailability_cache{$m}{$v}
+      if $req_unavailability_cache{$m}{$v};
   }
+
+  $ret;
 }
 
+
+# This is to be called by the author only (automatically in Makefile.PL)
 sub _gen_pod {
-  my $class = shift;
-  my $modfn = __PACKAGE__ . '.pm';
-  $modfn =~ s/\:\:/\//g;
+  my ($class, $distver, $pod_dir) = @_;
+
+  die "No POD root dir supplied" unless $pod_dir;
+
+  $distver ||=
+    eval { require DBIx::Class; DBIx::Class->VERSION; }
+      ||
+    die
+"\n\n---------------------------------------------------------------------\n" .
+'Unable to load core DBIx::Class module to determine current version, '.
+'possibly due to missing dependencies. Author-mode autodocumentation ' .
+"halted\n\n" . $@ .
+"\n\n---------------------------------------------------------------------\n"
+  ;
+
+  # do not ask for a recent version, use 1.x API calls
+  # this *may* execute on a smoker with old perl or whatnot
+  require File::Path;
+
+  (my $modfn = __PACKAGE__ . '.pm') =~ s|::|/|g;
+
+  (my $podfn = "$pod_dir/$modfn") =~ s/\.pm$/\.pod/;
+  (my $dir = $podfn) =~ s|/[^/]+$||;
+
+  File::Path::mkpath([$dir]);
+
+  my $sqltver = $class->req_list_for('deploy')->{'SQL::Translator'}
+    or die "Hrmm? No sqlt dep?";
+
 
-  my @chunks = (
-    <<"EOC",
+  my @chunks;
+
+#@@
+#@@ HEADER
+#@@
+  push @chunks, <<"EOC";
 #########################################################################
 #####################  A U T O G E N E R A T E D ########################
 #########################################################################
@@ -262,34 +864,96 @@ sub _gen_pod {
 # will be lost. If you need to change the generated text edit _gen_pod()
 # at the end of $modfn
 #
+
+=head1 NAME
+
+$class - Optional module dependency specifications (for module authors)
 EOC
-    '=head1 NAME',
-    "$class - Optional module dependency specifications",
-    '=head1 DESCRIPTION',
-    <<'EOD',
+
+
+#@@
+#@@ SYNOPSIS HEADING
+#@@
+  push @chunks, <<"EOC";
+=head1 SYNOPSIS
+
+Somewhere in your build-file (e.g. L<ExtUtils::MakeMaker>'s F<Makefile.PL>):
+
+  ...
+
+  \$EUMM_ARGS{CONFIGURE_REQUIRES} = {
+    \%{ \$EUMM_ARGS{CONFIGURE_REQUIRES} || {} },
+    'DBIx::Class' => '$distver',
+  };
+
+  ...
+
+  my %DBIC_DEPLOY_DEPS = %{ eval {
+    require $class;
+    $class->req_list_for('deploy');
+  } || {} };
+
+  \$EUMM_ARGS{PREREQ_PM} = {
+    \%DBIC_DEPLOY_DEPS,
+    \%{ \$EUMM_ARGS{PREREQ_PM} || {} },
+  };
+
+  ...
+
+  ExtUtils::MakeMaker::WriteMakefile(\%EUMM_ARGS);
+
+B<Note>: The C<eval> protection within the example is due to support for
+requirements during L<the C<configure> build phase|CPAN::Meta::Spec/Phases>
+not being available on a sufficient portion of production installations of
+Perl. Robust support for such dependency requirements is available in the
+L<CPAN> installer only since version C<1.94_56> first made available for
+production with perl version C<5.12>. It is the belief of the current
+maintainer that support for requirements during the C<configure> build phase
+will not be sufficiently ubiquitous until the B<year 2020> at the earliest,
+hence the extra care demonstrated above. It should also be noted that some
+3rd party installers (e.g. L<cpanminus|App::cpanminus>) do the right thing
+with configure requirements independent from the versions of perl and CPAN
+available.
+EOC
+
+
+#@@
+#@@ DESCRIPTION HEADING
+#@@
+  push @chunks, <<'EOC';
+=head1 DESCRIPTION
+
 Some of the less-frequently used features of L<DBIx::Class> have external
 module dependencies on their own. In order not to burden the average user
-with modules he will never use, these optional dependencies are not included
+with modules they will never use, these optional dependencies are not included
 in the base Makefile.PL. Instead an exception with a descriptive message is
-thrown when a specific feature is missing one or several modules required for
-its operation. This module is the central holding place for  the current list
-of such dependencies.
-EOD
-    '=head1 CURRENT REQUIREMENT GROUPS',
-    <<'EOD',
-Dependencies are organized in C<groups> and each group can list one or more
-required modules, with an optional minimum version (or 0 for any version).
-The group name can be used in the 
-EOD
-  );
+thrown when a specific feature can't find one or several modules required for
+its operation. This module is the central holding place for the current list
+of such dependencies, for DBIx::Class core authors, and DBIx::Class extension
+authors alike.
+
+Dependencies are organized in L<groups|/CURRENT REQUIREMENT GROUPS> where each
+group can list one or more required modules, with an optional minimum version
+(or 0 for any version). In addition groups prefixed with C<test_> can specify
+a set of environment variables, some (or all) of which are marked as required
+for the group to be considered by L</req_list_for>
+
+Each group name (or a combination thereof) can be used in the
+L<public methods|/METHODS> as described below.
+EOC
 
-  for my $group (sort keys %$reqs) {
-    my $p = $reqs->{$group}{pod}
-      or next;
 
-    my $modlist = $reqs->{$group}{req}
+#@@
+#@@ REQUIREMENT GROUPLIST HEADING
+#@@
+  push @chunks, '=head1 CURRENT REQUIREMENT GROUPS';
+
+  for my $group (sort keys %$dbic_reqs) {
+    my $p = $dbic_reqs->{$group}{pod}
       or next;
 
+    my $modlist = $class->modreq_list_for($group);
+
     next unless keys %$modlist;
 
     push @chunks, (
@@ -302,76 +966,180 @@ EOD
     );
   }
 
-  push @chunks, (
-    '=head1 METHODS',
-    '=head2 req_list_for',
-    '=over',
-    '=item Arguments: $group_name',
-    '=item Returns: \%list_of_module_version_pairs',
-    '=back',
-    <<EOD,
+
+#@@
+#@@ API DOCUMENTATION HEADING
+#@@
+  push @chunks, <<'EOC';
+
+=head1 METHODS
+
+=head2 req_group_list
+
+=over
+
+=item Arguments: none
+
+=item Return Value: \%list_of_requirement_groups
+
+=back
+
+This method should be used by DBIx::Class packagers, to get a hashref of all
+dependencies B<keyed> by dependency group. Each key (group name), or a combination
+thereof (as an arrayref) can be supplied to the methods below.
+The B<values> of the returned hash are currently a set of options B<without a
+well defined structure>. If you have use for any of the contents - contact the
+maintainers, instead of treating this as public (left alone stable) API.
+
+=head2 req_list_for
+
+=over
+
+=item Arguments: $group_name | \@group_names
+
+=item Return Value: \%set_of_module_version_pairs
+
+=back
+
 This method should be used by DBIx::Class extension authors, to determine the
-version of modules which a specific feature requires in the current version of
-DBIx::Class. For example if you write a module/extension that requires
-DBIx::Class and also requires the availability of
-L<DBIx::Class::Storage::DBI/deploy>, you can do the following in your
-C<Makefile.PL> or C<Build.PL>
-
- require $class;
- my \$dep_list = $class->req_list_for ('deploy');
-
-Which will give you a list of module/version pairs necessary for the particular
-feature to function with this version of DBIx::Class.
-EOD
-
-    '=head2 req_ok_for',
-    '=over',
-    '=item Arguments: $group_name',
-    '=item Returns: 1|0',
-    '=back',
-    'Returns true or false depending on whether all modules required by C<$group_name> are present on the system and loadable',
-
-    '=head2 req_missing_for',
-    '=over',
-    '=item Arguments: $group_name',
-    '=item Returns: $error_message_string',
-    '=back',
-    <<EOD,
-Returns a single line string suitable for inclusion in larger error messages.
-This method would normally be used by DBIx::Class core-module author, to
-indicate to the user that he needs to install specific modules before he will
-be able to use a specific feature.
-
-For example if the requirements for C<replicated> are not available, the
-returned string would look like:
-
- Moose >= 0.98, MooseX::Types >= 0.21, namespace::clean (see $class for details)
+version of modules a specific set of features requires for this version of
+DBIx::Class (regardless of their availability on the system).
+See the L</SYNOPSIS> for a real-world example.
+
+When handling C<test_*> groups this method behaves B<differently> from
+L</modreq_list_for> below (and is the only such inconsistency among the
+C<req_*> methods). If a particular group declares as requirements some
+C<environment variables> and these requirements are not satisfied (the envvars
+are unset) - then the C<module requirements> of this group are not included in
+the returned list.
+
+=head2 modreq_list_for
+
+=over
+
+=item Arguments: $group_name | \@group_names
+
+=item Return Value: \%set_of_module_version_pairs
+
+=back
 
+Same as L</req_list_for> but does not take into consideration any
+C<environment variable requirements> - returns just the list of required
+modules.
+
+=head2 req_ok_for
+
+=over
+
+=item Arguments: $group_name | \@group_names
+
+=item Return Value: 1|0
+
+=back
+
+Returns true or false depending on whether all modules/envvars required by
+the group(s) are loadable/set on the system.
+
+=head2 req_missing_for
+
+=over
+
+=item Arguments: $group_name | \@group_names
+
+=item Return Value: $error_message_string
+
+=back
+
+Returns a single-line string suitable for inclusion in larger error messages.
+This method would normally be used by DBIx::Class core features, to indicate to
+the user that they need to install specific modules and/or set specific
+environment variables before being able to use a specific feature set.
+
+For example if some of the requirements for C<deploy> are not available,
+the returned string could look like:
+EOC
+
+  push @chunks, qq{ "SQL::Translator~>=$sqltver" (see $class documentation for details)};
+
+  push @chunks, <<'EOC';
 The author is expected to prepend the necessary text to this message before
-returning the actual error seen by the user.
-EOD
-
-    '=head2 req_errorlist_for',
-    '=over',
-    '=item Arguments: $group_name',
-    '=item Returns: \%list_of_loaderrors_per_module',
-    '=back',
-    <<'EOD',
-Returns a hashref containing the actual errors that occured while attempting
-to load each module in the requirement group.
-EOD
-    '=head1 AUTHOR',
-    'See L<DBIx::Class/CONTRIBUTORS>.',
-    '=head1 LICENSE',
-    'You may distribute this code under the same terms as Perl itself',
-  );
+returning the actual error seen by the user. See also L</modreq_missing_for>
+
+=head2 modreq_missing_for
+
+=over
+
+=item Arguments: $group_name | \@group_names
+
+=item Return Value: $error_message_string
+
+=back
+
+Same as L</req_missing_for> except that the error string is guaranteed to be
+either empty, or contain a set of module requirement specifications suitable
+for piping to e.g. L<cpanminus|App::cpanminus>. The method explicitly does not
+attempt to validate the state of required environment variables (if any).
 
-  my $fn = __FILE__;
-  $fn =~ s/\.pm$/\.pod/;
+For instance if some of the requirements for C<deploy> are not available,
+the returned string could look like:
+EOC
+
+  push @chunks, qq{ "SQL::Translator~>=$sqltver"};
+
+  push @chunks, <<'EOC';
+=head2 die_unless_req_ok_for
+
+=over
+
+=item Arguments: $group_name | \@group_names
+
+=back
+
+Checks if L</req_ok_for> passes for the supplied group(s), and
+in case of failure throws an exception including the information
+from L</req_missing_for>.
+
+=head2 modreq_errorlist_for
+
+=over
+
+=item Arguments: $group_name | \@group_names
+
+=item Return Value: \%set_of_loaderrors_per_module
+
+=back
+
+Returns a hashref containing the actual errors that occurred while attempting
+to load each module in the requirement group(s).
+
+=head2 req_errorlist_for
+
+Deprecated method name, equivalent (via proxy) to L</modreq_errorlist_for>.
+
+EOC
+
+#@@
+#@@ FOOTER
+#@@
+  push @chunks, <<'EOC';
+=head1 FURTHER QUESTIONS?
+
+Check the list of L<additional DBIC resources|DBIx::Class/GETTING HELP/SUPPORT>.
+
+=head1 COPYRIGHT AND LICENSE
+
+This module is free software L<copyright|DBIx::Class/COPYRIGHT AND LICENSE>
+by the L<DBIx::Class (DBIC) authors|DBIx::Class/AUTHORS>. You can
+redistribute it and/or modify it under the same terms as the
+L<DBIx::Class library|DBIx::Class/COPYRIGHT AND LICENSE>.
+EOC
 
-  open (my $fh, '>', $fn) or croak "Unable to write to $fn: $!";
-  print $fh join ("\n\n", @chunks);
-  close ($fh);
+  eval {
+    open (my $fh, '>', $podfn) or die;
+    print $fh join ("\n\n", @chunks) or die;
+    print $fh "\n" or die;
+    close ($fh) or die;
+  } or croak( "Unable to write $podfn: " . ( $! || $@ || 'unknown error') );
 }
 
 1;