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.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.