On Sep 30, 2:15 pm, Christian Boos <[email protected]> wrote: > On 9/27/2010 10:31 AM, Christian Boos wrote: > > > > > I've been busy the past weeks on finishing the rework of our db API, > > > ... > > > Full details on the new API are available at > >http://trac.edgewall.org/wiki/TracDev/DatabaseApi?version=16#Accessin... > > I've done some more tests and all seems fine, so I'll proceed and commit > this to trunk in the coming days. > > > ... > > Comments welcome, in particular concerning how much we should > > deprecate the "old" ways, in particular if we should remove the good > > old `env.get_db_cnx()` completely, and when. > > So I'll keep the status quo for now, but we may want to remove it at > some point later (or at least, log a warning whenever it get used). > > -- Christian
I think the API looks very good, and makes sense when aiming for Python 2.5 as minimum. Great work! As for env.get_db_cnx() there is a LOT of code out there that use it. Removing it will be quite troublesome to say the least... :::simon -- You received this message because you are subscribed to the Google Groups "Trac Development" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/trac-dev?hl=en.
