Remove Old Style Tracker and db/backend_name support (was: Is anybody using config.py rather ...)
"John P. Rouillard" <[email protected]>
| Newsgroups | gmane.comp.bug-tracking.roundup.devel,gmane.comp.bug-tracking.roundup.user |
|---|---|
| Message-ID | <[email protected]> |
Hi all:
Earlier I wrote:
>Way back in the depths of time (before 2004-12-08: 0.8.0b1) roundup's
>configuration used to be done using config.py. After this point
>support for config.ini was provided. However it looks like config.py
>support is still present in the code as a backwards compatible method
>if config.ini is not loaded.
It turns out that the support for old style trackers which had the files:
config.py - replaced by config.ini in new style trackers
dbinit.py - replaced by schema.py in new style trackers (if present
the tracker is an old style tracker).
select_db.py - replaced by db/backend_name then RDBMS_BACKEND in config.ini
interfaces.py - still around
is still in the code base in instance.py. To go along with removal of
support for config.py from configure.py, I propose removing the
OldStyleTracker class from instance.py. This also removes another:
'import imp' from the code.
Also the obsolete db/backend_name file was replaced by RDBMS_BACKEND in
config.ini in release 1.6. The code that supports it has a comment to
delete support in revision 1.7 of roundup. That would be the minor rev
after 1.6 which turned out to be rev 2.0. So I would like to nuke that
support as well.
Does anybody have a problem with removing this dead code? (Hopefully
with the removal of this old untested code I can get to 70% code
coverage 8-).)
Have a great week.
--
-- rouilj
John Rouillard
===========================================================================
My employers don't acknowledge my existence much less my opinions.