Summary:	ASTERISK-06576: [branch] SQLite 3 support for asterisk CDR
Reporter:	Moises Silva (moy)	Labels:
Date Opened:	2006-03-20 11:11:49.000-0600	Date Closed:	2011-06-07 14:02:44
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	CDR/cdr_sqlite
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) 46432.sqlite3.patch ( 1) asterisk-svn-13620-sqlite3.patch ( 2) cdrsqlite-Makefile-svn-17823 ( 3) sqlite3.patch
Description:	New file, cdr/cdr_sqlite3.c to support SQLite 3 databases as well. Just need a new entry in the Makefile linked against sqlite3 library
Comments:	By: Moises Silva (moy) 2006-03-20 11:12:22.000-0600 disclaimer sign/fax 30 minutes ago By: Moises Silva (moy) 2006-03-20 12:25:22.000-0600 tested with SVN trunk revision number: 13620 By: Olle Johansson (oej) 2006-04-05 11:34:32 Can we get the patch for the makefile too, so I don't have to figure it out all by myself? Thanks! By: Moises Silva (moy) 2006-04-06 09:18:26 sorry about that oej, here is the Makefile patch for svn-17823. The same sqlite3.c  will be fine. Please let me know if you need anything else. By: Olle Johansson (oej) 2006-04-06 09:34:54 Added to test-this-branch By: Olle Johansson (oej) 2006-06-09 03:40:01 We need an update for the new module loader, thanks. By: Denis Smirnov (mithraen) 2006-06-09 08:26:29 Patch for update to new loader (not tested!) By: Olle Johansson (oej) 2006-06-09 08:28:42 Mithraen: Thanks. Please confirm that you have a disclaimer on file. What about looking into autoconf/menuselect? :-) By: Denis Smirnov (mithraen) 2006-06-09 14:28:14 I confirm disclaimer. Ok, I try to add menuselect/autoconf stuff. By: Denis Smirnov (mithraen) 2006-06-11 09:14:30 sqlite3.patch contain loader.update.patch data and patches for porting to autoconf/menuselect (not tested). By: Denis Smirnov (mithraen) 2006-08-10 17:57:19 What about integrating this to test-this-branch? By: Serge Vecher (serge-v) 2006-08-11 08:43:14 Mithraen: test-this-branch is not actively maintained at this point, as all of the things that needed to make it into trunk out of that branch for 1.4 have been mostly merged. I do agree that this would be better off living in it's own branch and evolve with trunk, rather than sit in the patch... Have you considered emailing kpf regarding setting up a developer branch for you? I believe you have enough karma to qualify ;) By: jmls (jmls) 2006-11-01 05:29:39.000-0600 Mithraen, do you have any comments to serge-v's questions and comments ? By: Denis Smirnov (mithraen) 2006-11-01 06:21:09.000-0600 I don't have svn access now. I asks many time for this, but today I don't have it. When someonegive me this access, I try to complete this work. By: Anthony LaMantia (alamantia) 2006-11-02 14:42:15.000-0600 Hi, What do you think about overhauling the sqlite support in trunk to require sqlite3 ... as far as i know, sqlite3 can read sqlite and sqlite2 files.. just requiring the latest version over adding support for each individual one seems like the more logical route to go with this. anyone have thoughts commends on this issue? By: Denis Smirnov (mithraen) 2006-11-03 10:47:17.000-0600 sqlite3 not compatible with database created by 2.* versions By: Denis Smirnov (mithraen) 2006-11-03 12:16:19.000-0600 Try to update for last trunk (not tested) By: Anthony LaMantia (alamantia) 2006-11-06 12:12:00.000-0600 i have commited code into a branch of mine that adds support for sqlite1/2/3, detection for each with configure and modfications to menuselect. it is located here, if you have any comments/thoughts/improvements please post them here. the branch can be found here: view: http://svn.digium.com/view/asterisk/team/anthonyl/sqlite-support/ checkout: http://svn.digium.com/svn/asterisk/team/anthonyl/sqlite-support/ By: Russell Bryant (russell) 2007-03-13 16:25:23 Unfortunately, this module has not been merged because another module, cdr_sqlite3_custom has been merged instead.  I would encourage you to try out that module to make sure that it fits your needs, as well. Thank you very much for your contribution!