X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=Makefile.PL;h=57b2be4eb8f8e15d0038466e03b7d176049fc0c7;hb=9901aad73ff9dc45b426534fe406c102fb9fb77c;hp=58c0c561dc694a636cce9f8ac0e69613066c793a;hpb=6882de2b334aac0a278b1984e82ddaca2ddff2b4;p=dbsrgits%2FDBIx-Class.git diff --git a/Makefile.PL b/Makefile.PL index 58c0c56..57b2be4 100644 --- a/Makefile.PL +++ b/Makefile.PL @@ -1,4 +1,4 @@ -use inc::Module::Install 0.67; +use inc::Module::Install 0.79; use strict; use warnings; use POSIX (); @@ -9,11 +9,16 @@ name 'DBIx-Class'; perl_version '5.006001'; all_from 'lib/DBIx/Class.pm'; +# configure_requires so _check_sqlite() below can run +# remove once test deprecated +configure_requires 'DBD::SQLite'; + +requires 'DBD::SQLite' => 1.23; requires 'Data::Page' => 2.00; requires 'Scalar::Util' => 0; -requires 'SQL::Abstract' => 1.51; +requires 'SQL::Abstract' => 1.53; requires 'SQL::Abstract::Limit' => 0.13; -requires 'Class::C3::Componentised' => 1.0004; +requires 'Class::C3::Componentised' => 1.0005; requires 'Storable' => 0; requires 'Carp::Clan' => 0; requires 'DBI' => 1.40; @@ -30,9 +35,6 @@ requires 'MRO::Compat' => 0.09; # Perl 5.8.0 doesn't have utf8::is_utf8() requires 'Encode' => 0 if ($] <= 5.008000); -# configure_requires so the sanity check below can run -configure_requires 'DBD::SQLite' => 1.21; - test_requires 'Test::Builder' => 0.33; test_requires 'Test::Warn' => 0.11; test_requires 'Test::Exception' => 0; @@ -40,9 +42,13 @@ test_requires 'Test::Deep' => 0; recommends 'SQL::Translator' => 0.09004; -install_script 'script/dbicadmin'; +install_script (qw| + script/dbicadmin +|); -tests_recursive 't'; +tests_recursive (qw| + t +|); # re-build README and require extra modules for testing if we're in a checkout @@ -54,8 +60,9 @@ my %force_requires_if_author = ( 'DBIx::ContextualFetch' => 0, 'Class::DBI::Plugin::DeepAbstractSearch' => 0, 'Class::Trigger' => 0, - 'Time::Piece' => 0, + 'Time::Piece::MySQL' => 0, 'Clone' => 0, + 'Date::Simple' => 0, # t/52cycle.t 'Test::Memory::Cycle' => 0, @@ -63,9 +70,17 @@ my %force_requires_if_author = ( # t/60core.t 'DateTime::Format::MySQL' => 0, + # t/72pg.t + $ENV{DBICTEST_PG_DSN} + ? ('Sys::SigAction'=> 0) + : () + , + # t/93storage_replication.t - 'Moose', => 0, - 'MooseX::AttributeHelpers' => 0.12, + 'Moose', => 0.54, + 'MooseX::AttributeHelpers' => 0.12, + 'MooseX::Types', => 0.10, + 'namespace::clean' => 0.11, # t/96_is_deteministic_value.t 'DateTime::Format::Strptime' => 0, @@ -97,59 +112,50 @@ EOW auto_install; # Have all prerequisites, check DBD::SQLite sanity -if (! $ENV{DBICTEST_NO_SQLITE_CHECK} ) { +_check_sqlite() if (! $ENV{DBICTEST_NO_SQLITE_CHECK} ); - my $pid = fork(); - if (not defined $pid) { - die "Unable to fork(): $!"; - } - elsif (! $pid) { +WriteAll(); - # Win32 does not have real fork()s so a segfault will bring - # everything down. Warn about it. - if ($^O eq 'MSWin32') { - print <<'EOW'; +if ($Module::Install::AUTHOR) { + # Need to do this _after_ WriteAll else it loses track of them + Meta->{values}{build_requires} = [ grep { + my $ok = 1; + foreach my $module (keys %force_requires_if_author) { + if ($_->[0] =~ /$module/) { + $ok = 0; + last; + } + } + $ok; + } @{Meta->{values}{build_requires}} ]; -###################################################################### -# # -# A short stress-testing of DBD::SQLite will follow. If you have a # -# buggy library this might very well be the last text you will see # -# before the installation silently terminates. If this happens it # -# would mean that you are running a buggy version of DBD::SQLite # -# known to randomly segfault on errors. Even if you have the latest # -# CPAN module version, the system sqlite3 dynamic library might have # -# been compiled against an older buggy sqlite3 dev library (oddly # -# DBD::SQLite will prefer the system library against the one bundled # -# with it). You are strongly advised to resolve this issue before # -# proceeding. # -# # -# If this happens to you (this text is the last thing you see), and # -# you just want to install this module without worrying about the # -# tests (which will almost certainly fail) - set the environment # -# variable DBICTEST_NO_SQLITE_CHECK to a true value and try again. # -# # -###################################################################### + Meta->{values}{resources} = [ + [ 'MailingList', 'http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/dbix-class' ], + [ 'IRC', 'irc://irc.perl.org/#dbix-class' ], + [ 'license', 'http://dev.perl.org/licenses/' ], + [ 'repository', 'http://dev.catalyst.perl.org/svnweb/bast/browse/DBIx-Class/' ], + ]; + Meta->write; +} -EOW - } - require DBI; - for (1 .. 100) { - my $dbh; - $dbh = DBI->connect ('dbi:SQLite::memory:', undef, undef, { - AutoCommit => 1, - RaiseError => 0, - PrintError => 0, - }) - or die "Unable to connect to database: $@"; - $dbh->do ('CREATE TABLE name_with_no_columns'); # a subtle syntax error - $dbh->do ('COMMIT'); # followed by commit - $dbh->disconnect; - } +# This is legacy code. Latest DBD::SQLite developments fixed all known bugs +# in this area. Remove before some arbitrary next version +sub _check_sqlite { + + # Win32 does not have real fork()s so a segfault will bring + # everything down. Warn about it below, and don't try fork() + if ($^O ne 'MSWin32') { + my $pid = fork(); + if (not defined $pid) { + die "Unable to fork(): $!"; + } + elsif (! $pid) { + _torture_sqlite(); exit 0; - } - else { + } + else { eval { local $SIG{ALRM} = sub { die "timeout\n" }; alarm 5; @@ -160,7 +166,7 @@ EOW my $sig = $? & 127; -# make sure process actually dies + # make sure process actually dies $exception && kill POSIX::SIGKILL(), $pid; if ($exception || $sig == POSIX::SIGSEGV() || $sig == POSIX::SIGABRT() @@ -186,40 +192,50 @@ EOE ); exit 0 unless ($ans =~ /^y(es)?$/i); } + } } -} + else { # the win32 version -WriteAll(); - + print <<'EOW'; +###################################################################### +# # +# A short stress-testing of DBD::SQLite will follow. If you have a # +# buggy library this might very well be the last text you will see # +# before the installation silently terminates. If this happens it # +# would mean that you are running a buggy version of DBD::SQLite # +# known to randomly segfault on errors. Even if you have the latest # +# CPAN module version, the system sqlite3 dynamic library might have # +# been compiled against an older buggy sqlite3 dev library (oddly # +# DBD::SQLite will prefer the system library against the one bundled # +# with it). You are strongly advised to resolve this issue before # +# proceeding. # +# # +# If this happens to you (this text is the last thing you see), and # +# you just want to install this module without worrying about the # +# tests (which will almost certainly fail) - set the environment # +# variable DBICTEST_NO_SQLITE_CHECK to a true value and try again. # +# # +###################################################################### -if ($Module::Install::AUTHOR) { - # Need to do this _after_ WriteAll else it loses track of them - Meta->{values}{build_requires} = [ grep { - my $ok = 1; - foreach my $module (keys %force_requires_if_author) { - if ($_->[0] =~ /$module/) { - $ok = 0; - last; - } - } - $ok; - } @{Meta->{values}{build_requires}} ]; +EOW - my @scalar_keys = Module::Install::Metadata::Meta_TupleKeys(); - my $cr = Module::Install::Metadata->can("Meta_TupleKeys"); - { - no warnings 'redefine'; - *Module::Install::Metadata::Meta_TupleKeys = sub { - return $cr->(@_), 'resources'; - }; + _torture_sqlite(); } - Meta->{values}{resources} = [ - [ 'MailingList', 'http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/dbix-class' ], - [ 'IRC', 'irc://irc.perl.org/#dbix-class' ], - [ 'license', 'http://dev.perl.org/licenses/' ], - [ 'repository', 'http://dev.catalyst.perl.org/svnweb/bast/browse/DBIx-Class/' ], - ]; - Meta->write; } +sub _torture_sqlite { + require DBI; + + for (1 .. 100) { + my $dbh = DBI->connect ('dbi:SQLite::memory:', undef, undef, { + AutoCommit => 1, + RaiseError => 0, + PrintError => 0, + }) or die "Unable to connect to database: $@"; + + $dbh->do ('CREATE TABLE name_with_no_columns'); # a subtle syntax error + $dbh->do ('COMMIT'); # followed by commit + $dbh->disconnect; + } +}