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:58 Message: Logged In: YES user_id=808795 Originator: YES If python is not available then openserctl won't work for updating the tables. However, it's already fairly broken with the current setup (with auto-increment added to dbtext in 1.3 openserctl/dbtext no longer works so well). And since it requires special handling in openserctl it's in a constant state of brokeness as the various patches are eventually added to support this and that new feature. As aresult I'd wager most people using dbtext just don't rely on openserctl at all. I've had to apply local patches to it myself for the last year or so, finally resulting in getting so frustrated with the massive changes 1.3 brought that I wrote dbtextdb. For what it's worth I use dbtext on normal (non-embedded) machines in a fairly large corporate setup. Perhaps if it's better supported by the tools more people would opt to use dbtext over a 'real' db -- in the end python dependency is still better than full-blown db dependency. -Herman ---------------------------------------------------------------------- Comment By: Ovidiu Sas (osas) Date: 2008-03-27 14:39 Message: Logged In: YES user_id=1395524 Originator: NO I know about the openserctl db_text mess :) db_text is mostly used on small or embedded systems when a real db is not available or it is to heavy for the system. On this systems, python may or may not be available ... this is my only concern ... Regards, Ovidiu Sas ---------------------------------------------------------------------- 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
