[OpenSER-Devel] [ openser-Patches-1927301 ] add SQL support to dbtext

[SourceForge.net]

Patches item #1927301, was opened at 2008-03-27 12:32
Message generated for change (Comment added) made by has207
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1927301&group_id=139143

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tools
Group: ver 1.3.x
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Herman Sheremetyev (has207)
Assigned to: Nobody/Anonymous (nobody)
Summary: add SQL support to dbtext

Initial Comment:
Add SQL support to dbtext using a python module and integrate it into openserctl getting rid of the special case handling of dbtext.  The dbtextdb python module is written in a generic enough way that it can be used by other tools as  well or called from the command line directly (the way openserctl does it now).

----------------------------------------------------------------------

>Comment By: Herman Sheremetyev (has207)
Date: 2008-03-27 14:25

Message:
Logged In: YES 
user_id=808795
Originator: YES

It only adds a python dependency for openserctl when used with dbtext, not
openser itself.  You can still run openser, with dbtext even, without
python.  However the current implementation of openserctl with dbtext is
awful to put it mildly.  There are special dbtext cases all over the
script, and the way the dbtext tables are updated is hardcoded so that if
the table structure changes you have to modify the script.  dbtextdb makes
the dbtext tables accessible using the same SQL queries currently used for
MySQL and Postgres.  IMO, installing python is a small price to pay to make
your dbtext tables manageable, and it's not like there aren't other modules
that have other external dependencies already.  

----------------------------------------------------------------------

Comment By: Ovidiu Sas (osas)
Date: 2008-03-27 14:17

Message:
Logged In: YES 
user_id=1395524
Originator: NO

This patch will create an extra dependency on python.
Right now python is not required to run openser.

----------------------------------------------------------------------

Comment By: Herman Sheremetyev (has207)
Date: 2008-03-27 12:38

Message:
Logged In: YES 
user_id=808795
Originator: YES

File Added: openserctl-dbtext-openserctl-sqlbase.patch

----------------------------------------------------------------------

Comment By: Herman Sheremetyev (has207)
Date: 2008-03-27 12:37

Message:
Logged In: YES 
user_id=808795
Originator: YES

File Added: openserctl-dbtext-openserctl-dbtext.patch

----------------------------------------------------------------------

Comment By: Herman Sheremetyev (has207)
Date: 2008-03-27 12:36

Message:
Logged In: YES 
user_id=808795
Originator: YES

File Added: openserctl-dbtext-openserctl-base.patch

----------------------------------------------------------------------

Comment By: Herman Sheremetyev (has207)
Date: 2008-03-27 12:35

Message:
Logged In: YES 
user_id=808795
Originator: YES

File Added: openserctl-dbtext-openserctl.patch

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1927301&group_id=139143