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 _______________________________________________ Devel mailing list Devel@lists.openser.org http://lists.openser.org/cgi-bin/mailman/listinfo/devel
