make DBI completely optional

Previously, the 00_compile test would fail if DBI was missing. Now, it's checked for at runtime, not compile time. Fixes #77
preaction · Jan 10, 2015 · cb42b49 · cb42b49
1 parent aff2829
commit cb42b49
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/lib/ETL/Yertl/Command/ysql.pm b/lib/ETL/Yertl/Command/ysql.pm
@@ -5,15 +5,13 @@ use Getopt::Long qw( GetOptionsFromArray );
 use ETL::Yertl::Format::yaml;
 use File::HomeDir;
 
-BEGIN {
-    eval { use DBI; };
+sub main {
+    my $class = shift;
+
+    eval { require DBI; };
     if ( $@ ) {
         die "Can't load ysql: Can't load DBI. Make sure the DBI module is installed.\n";
     }
-}
-
-sub main {
-    my $class = shift;
 
     my %opt;
     if ( ref $_[-1] eq 'HASH' ) {

diff --git a/t/bin/ysql.t b/t/bin/ysql.t
@@ -5,7 +5,7 @@ use ETL::Yertl::Format::yaml;
 my $SHARE_DIR = path( __DIR__, '..', 'share' );
 
 BEGIN {
-    eval { use DBI; use DBD::SQLite; };
+    eval { require DBI; require DBD::SQLite; };
     if ( $@ ) {
         plan skip_all => 'missing DBI and/or DBD::SQLite';
     }