Skip Menu |
 

This queue is for tickets about the DBD-SQLite CPAN distribution.

Report information
The Basics
Id: 8750
Status: resolved
Priority: 0/
Queue: DBD-SQLite

People
Owner: Nobody in particular
Requestors: nick [...] ccl4.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)

Attachments
DBD-SQLite_prereq_DBI.patch



Subject: DBD::SQLite Makefile.PL fails if no DBD
Download (untitled) / with headers
text/plain 695b
Trying to build DBD::SQLite (1.07) against a fresh perl fails: CPAN.pm: Going to build M/MS/MSERGEANT/DBD-SQLite-1.07.tar.gz Checking if your kit is complete... Looks good Warning: prerequisite DBI 1.21 not found. Undefined subroutine &DBI::DBD::dbd_postamble called at Makefile.PL line 58. Running make test Make had some problems, maybe interrupted? Won't test Running make install Make had some problems, maybe interrupted? Won't install The problem seems to be this section of Makefile.PL, which naïvely assumes that DBI is already installed. I think that you need to check that the sub is defined before calling it. package MY; sub postamble { DBI::DBD::dbd_postamble(@_); }
From: david_dick [...] iprimus.com.au
This patch should fix your issue.
diff -Naur DBD-SQLite-1.07/Makefile.PL new/Makefile.PL --- DBD-SQLite-1.07/Makefile.PL 2004-10-12 19:03:50.000000000 +1000 +++ new/Makefile.PL 2004-12-09 11:28:14.553552208 +1100 @@ -1,9 +1,23 @@ eval { require DBI; +}; +if ($@) { + print "DBI must be installed\n"; + print "$@"; + exit 1; +}; +eval { require DBI::DBD; - die "Your DBI Version is too old - require at least 1.03" - unless $DBI::VERSION >= 1.03; }; +if ($@) { + print "DBI::DBD must be installed\n"; + print "$@"; + exit 1; +} +unless ($DBI::VERSION >= 1.03) { + print "Your DBI Version is too old - require at least 1.03\n"; + exit 1; +} use ExtUtils::MakeMaker; use Config; use strict;


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.