NetBSD Problem Report #46322
From khorben@defora.org Wed Apr 11 00:12:22 2012
Return-Path: <khorben@defora.org>
Received: from mail.netbsd.org (mail.netbsd.org [149.20.53.66])
by www.NetBSD.org (Postfix) with ESMTP id 08FA563B9FE
for <gnats-bugs@gnats.NetBSD.org>; Wed, 11 Apr 2012 00:12:22 +0000 (UTC)
Message-Id: <20120410225450.474B71A474@syn.defora.rom>
Date: Wed, 11 Apr 2012 00:54:50 +0200 (CEST)
From: Pierre Pronchery <khorben@defora.org>
Reply-To:
To: gnats-bugs@gnats.NetBSD.org
Subject: SQLite from base does not include useful functions
X-Send-Pr-Version: 3.95
>Number: 46322
>Category: lib
>Synopsis: SQLite from base is compiled without SQLITE_ENABLE_COLUMN_METADATA
>Confidential: no
>Severity: non-critical
>Priority: medium
>Responsible: lib-bug-people
>State: closed
>Class: sw-bug
>Submitter-Id: net
>Arrival-Date: Wed Apr 11 00:15:00 +0000 2012
>Closed-Date: Mon Oct 07 06:15:32 +0000 2013
>Last-Modified: Thu Oct 10 16:25:00 +0000 2013
>Originator: Pierre Pronchery <khorben@defora.org>
>Release: NetBSD 6.0_BETA
>Organization:
>Environment:
System: NetBSD syn.defora.rom 6.0_BETA NetBSD 6.0_BETA (GENERIC) #7: Tue Apr 10 02:09:40 CEST 2012 khorben@syn.defora.rom:/home/amd64/obj/sys/arch/amd64/compile/GENERIC amd64
Architecture: x86_64
Machine: amd64
>Description:
A number of functions from SQLite 3 are only available when the
libsqlite3 library is built with particular values defined. One such
setting is SQLITE_ENABLE_COLUMN_METADATA, which includes the following
functions when enabled:
- sqlite3_column_database_name()
- sqlite3_column_database_name16()
- sqlite3_column_table_name()
- sqlite3_column_table_name16()
- sqlite3_column_origin_name()
- sqlite3_column_origin_name16()
- sqlite3_table_column_metadata()
These functions may be essential for completeness (including matching
the documentation), as well as for programmers relying on SQLite 3 as
available from base to be fully functional. At least to me :)
FWIW, I am currently relying on the sqlite3_column_origin_name()
function while analyzing results from prepared statements, passing the
column names along with the resulting rows of data to my respective
callbacks. The other functions are returning the respective databases
and tables associated to each column.
If this change is accepted, a pull-up to the netbsd-6 branch would also
be much appreciated.
>How-To-Repeat:
Use any of the functions listed above in a C/C++ program linked with
libsqlite3 from base.
>Fix:
As follows:
Index: external/public-domain/sqlite/Makefile.inc
===================================================================
RCS file: /cvsroot/src/external/public-domain/sqlite/Makefile.inc,v
retrieving revision 1.3
diff -p -u -r1.3 Makefile.inc
--- external/public-domain/sqlite/Makefile.inc 11 Nov 2011 01:34:18 -0000 1.3
+++ external/public-domain/sqlite/Makefile.inc 10 Apr 2012 22:16:11 -0000
@@ -10,6 +10,7 @@ CPPFLAGS+= -DHAVE_READLINE=1 -DHAVE_LOCA
-DSQLITE_HAVE_ISNAN -DSQLITE_ENABLE_FTS4 \
-DSQLITE_ENABLE_FTS3_PARENTHESIS \
-DSQLITE_ENABLE_UNLOCK_NOTIFY \
- -DSQLITE_ENABLE_LOAD_EXTENSION
+ -DSQLITE_ENABLE_LOAD_EXTENSION \
+ -DSQLITE_ENABLE_COLUMN_METADATA
CWARNFLAGS+= -Wno-shadow -Wno-unused
>Release-Note:
>Audit-Trail:
From: "Christos Zoulas" <christos@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc:
Subject: PR/46322 CVS commit: src/external/public-domain/sqlite
Date: Tue, 10 Apr 2012 20:41:41 -0400
Module Name: src
Committed By: christos
Date: Wed Apr 11 00:41:41 UTC 2012
Modified Files:
src/external/public-domain/sqlite: Makefile.inc
Log Message:
PR/46322: Pierre Pronchery: SQLite from base is compiled without
SQLITE_ENABLE_COLUMN_METADATA
XXX: Pullup for 6
To generate a diff of this commit:
cvs rdiff -u -r1.3 -r1.4 src/external/public-domain/sqlite/Makefile.inc
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
From: "Jeff Rizzo" <riz@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc:
Subject: PR/46322 CVS commit: [netbsd-6] src/external/public-domain/sqlite
Date: Mon, 11 Jun 2012 17:51:42 +0000
Module Name: src
Committed By: riz
Date: Mon Jun 11 17:51:42 UTC 2012
Modified Files:
src/external/public-domain/sqlite [netbsd-6]: Makefile.inc
Log Message:
Pull up following revision(s) (requested by khorben in ticket #294):
external/public-domain/sqlite/Makefile.inc: revision 1.4
PR/46322: Pierre Pronchery: SQLite from base is compiled without
SQLITE_ENABLE_COLUMN_METADATA
XXX: Pullup for 6
To generate a diff of this commit:
cvs rdiff -u -r1.3 -r1.3.2.1 src/external/public-domain/sqlite/Makefile.inc
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
State-Changed-From-To: open->closed
State-Changed-By: dholland@NetBSD.org
State-Changed-When: Mon, 07 Oct 2013 06:15:32 +0000
State-Changed-Why:
pullups completed in june 2012
From: Pierre Pronchery <khorben@defora.org>
To: gnats-bugs@NetBSD.org
Cc: dholland@NetBSD.org, lib-bug-people@netbsd.org,
netbsd-bugs@netbsd.org, gnats-admin@netbsd.org
Subject: Re: lib/46322 (SQLite from base is compiled without SQLITE_ENABLE_COLUMN_METADATA)
Date: Wed, 09 Oct 2013 21:25:15 +0200
On 07/10/2013 08:15, dholland@NetBSD.org wrote:
> Synopsis: SQLite from base is compiled without SQLITE_ENABLE_COLUMN_METADATA
>
> State-Changed-From-To: open->closed
> State-Changed-By: dholland@NetBSD.org
> State-Changed-When: Mon, 07 Oct 2013 06:15:32 +0000
> State-Changed-Why:
> pullups completed in june 2012
Actually, the more I think about it, the more I think it should be
reverted. Not a big deal, this only brought more functionality, but then
it's also no longer the standard SQLite.
Should I revert it in -current, or is it fine as it is?
Sorry,
--
khorben
From: Joerg Sonnenberger <joerg@britannica.bec.de>
To: gnats-bugs@NetBSD.org
Cc:
Subject: Re: lib/46322 (SQLite from base is compiled without
SQLITE_ENABLE_COLUMN_METADATA)
Date: Wed, 9 Oct 2013 22:21:55 +0200
On Wed, Oct 09, 2013 at 07:30:01PM +0000, Pierre Pronchery wrote:
> The following reply was made to PR lib/46322; it has been noted by GNATS.
>
> From: Pierre Pronchery <khorben@defora.org>
> To: gnats-bugs@NetBSD.org
> Cc: dholland@NetBSD.org, lib-bug-people@netbsd.org,
> netbsd-bugs@netbsd.org, gnats-admin@netbsd.org
> Subject: Re: lib/46322 (SQLite from base is compiled without SQLITE_ENABLE_COLUMN_METADATA)
> Date: Wed, 09 Oct 2013 21:25:15 +0200
>
> On 07/10/2013 08:15, dholland@NetBSD.org wrote:
> > Synopsis: SQLite from base is compiled without SQLITE_ENABLE_COLUMN_METADATA
> >
> > State-Changed-From-To: open->closed
> > State-Changed-By: dholland@NetBSD.org
> > State-Changed-When: Mon, 07 Oct 2013 06:15:32 +0000
> > State-Changed-Why:
> > pullups completed in june 2012
>
> Actually, the more I think about it, the more I think it should be
> reverted. Not a big deal, this only brought more functionality, but then
> it's also no longer the standard SQLite.
>
> Should I revert it in -current, or is it fine as it is?
It is fine as is.
Joerg
From: David Holland <dholland-bugs@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc:
Subject: Re: lib/46322 (SQLite from base is compiled without
SQLITE_ENABLE_COLUMN_METADATA)
Date: Thu, 10 Oct 2013 16:22:41 +0000
On Wed, Oct 09, 2013 at 07:30:01PM +0000, Pierre Pronchery wrote:
> > pullups completed in june 2012
>
> Actually, the more I think about it, the more I think it should be
> reverted. Not a big deal, this only brought more functionality, but then
> it's also no longer the standard SQLite.
>
> Should I revert it in -current, or is it fine as it is?
I would say there's no need to revert it unless it's causing a
problem, and I've not run across any evidence of that.
--
David A. Holland
dholland@netbsd.org
>Unformatted:
Wed Apr 11 00:14:47 CEST 2012
(Contact us)
$NetBSD: query-full-pr,v 1.39 2013/11/01 18:47:49 spz Exp $
$NetBSD: gnats_config.sh,v 1.8 2006/05/07 09:23:38 tsutsui Exp $
Copyright © 1994-2007
The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.