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

NetBSD Home
NetBSD PR Database Search

(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.