Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:3077 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 70189 invoked from network); 1 Jul 2003 10:27:10 -0000 Received: from unknown (HELO jdi.jdimedia.nl) (212.204.192.51) by pb1.pair.com with SMTP; 1 Jul 2003 10:27:10 -0000 Received: from jdi.jdimedia.nl (jdi.jdimedia.nl [212.204.192.51]) by jdi.jdimedia.nl (8.12.4/8.12.4) with ESMTP id h61AR592015214; Tue, 1 Jul 2003 12:27:05 +0200 Date: Tue, 1 Jul 2003 12:27:05 +0200 (CEST) X-X-Sender: derick@jdi.jdimedia.nl To: Sascha Schumann cc: John Coggeshall , internals@lists.php.net In-Reply-To: Message-ID: References: <1056790680.5070.13.camel@coogle.localdomain> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Subject: Re: [PHP-DEV] [PATCH] SQLite session storage From: derick@php.net (Derick Rethans) On Tue, 1 Jul 2003, Sascha Schumann wrote: > On Sat, 28 Jun 2003, John Coggeshall wrote: > > > Since SQLite will be bundled in the future, I sat down and got > > acquainted with it by writing a module for the session extension which > > allows users to store their session data in a SQLite database*. > > > > To use, just apply the patch to session.c (which adds the reference to > > the new module and adds the mod_sqlite.c to config.m4) and compile.. To > > use it set session.save_handler = sqlite. > > Sounds fine in principle. Where is the patch? Wouldn't it be much nicer to put this in the sqlite extension instead because people may turn sqlite support off? php_register_session_module should be used here IMO. Derick -- "Interpreting what the GPL actually means is a job best left to those that read the future by examining animal entrails." ------------------------------------------------------------------------- Derick Rethans http://derickrethans.nl/ International PHP Magazine http://php-mag.net/ -------------------------------------------------------------------------