Skip Menu |

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

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

People
Owner: Nobody in particular
Requestors: GAAS [...] cpan.org
Cc:
AdminCc:

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



Subject: Name "DBD::SQLite::sqlite_version" used only once
This warning shows up in an app I have: $ perl -w -MDBD::SQLite -le 'print $DBD::SQLite::VERSION' Name "DBD::SQLite::sqlite_version" used only once: possible typo at /usr/local/ActivePerl-5.10/lib/DynaLoader.pm line 229. 1.14
This is a suggested patch: --- DBD-SQLite/1.13/SQLite.xs.~1~ Thu Jun 5 10:34:55 2008 +++ DBD-SQLite/1.13/SQLite.xs Thu Jun 5 10:34:55 2008 @@ -9,7 +9,7 @@ PROTOTYPES: DISABLE BOOT: - sv_setpv(get_sv("DBD::SQLite::sqlite_version", TRUE), SQLITE_VERSION); + sv_setpv(get_sv("DBD::SQLite::sqlite_version", TRUE|GV_ADDMULTI), SQLITE_VERSION); AV * list_tables(dbh)
Index: DBD-SQLite/1.13/SQLite.xs --- DBD-SQLite/1.13/SQLite.xs.~1~ Thu Jun 5 10:34:55 2008 +++ DBD-SQLite/1.13/SQLite.xs Thu Jun 5 10:34:55 2008 @@ -9,7 +9,7 @@ PROTOTYPES: DISABLE BOOT: - sv_setpv(get_sv("DBD::SQLite::sqlite_version", TRUE), SQLITE_VERSION); + sv_setpv(get_sv("DBD::SQLite::sqlite_version", TRUE|GV_ADDMULTI), SQLITE_VERSION); AV * list_tables(dbh) End of Patch.
On Thu Jun 05 04:37:54 2008, GAAS wrote: Show quoted text
> + sv_setpv(get_sv("DBD::SQLite::sqlite_version", TRUE|GV_ADDMULTI),
SQLITE_VERSION); Looks better if you then also drop the "TRUE|" part.
From: msergeant [...] cpan.org
On Thu Jun 05 04:39:57 2008, GAAS wrote: Show quoted text
> On Thu Jun 05 04:37:54 2008, GAAS wrote:
> > + sv_setpv(get_sv("DBD::SQLite::sqlite_version", TRUE|GV_ADDMULTI),
> SQLITE_VERSION); > > Looks better if you then also drop the "TRUE|" part.
Applied in revision 191.
From: kasal [...] ucw.cz
On Fri Jun 06 13:57:08 2008, MSERGEANT wrote: Show quoted text
> Applied in revision 191.
But where does the VCS live? (I have tried svn.perl.org, but have not found it there.) Stepan Kasal
Setting status to resolved again, just a more or less unrelated question reopened it.
Actually setting this to resolved