diff --git a/bin/autojump b/bin/autojump index 96e3fbc..4ea2ced 100755 --- a/bin/autojump +++ b/bin/autojump @@ -29,7 +29,7 @@ import shutil import sys from tempfile import NamedTemporaryFile -VERSION = 'release-v20.9.4' +VERSION = 'release-v20.9.5' MAX_KEYWEIGHT = 1000 MAX_STORED_PATHS = 1000 COMPLETION_SEPARATOR = '__' @@ -129,8 +129,6 @@ class Database: trying to recover from backup...', file=sys.stderr) shutil.copy(self.filename + '.bak', self.filename) return self.load(True) - else: - self.migrate() def maintenance(self): """ @@ -142,33 +140,6 @@ class Database: self.trim() self.save() - def migrate(self): - """ - Migrates database from old format autojump_py, last used in release-v17. - """ - self.filename = get_db_file('autojump_py') - if os.path.exists(self.filename): - try: # fix to get optimised pickle in python < 3 - import cPickle as pickle - except ImportError: - import pickle - - try: - with open(self.filename, 'rb') as f: - # encoding is only specified for Python 2 compatibility - if sys.version_info[0] > 2: - self.data = pickle.load(f, encoding="utf-8") - else: - self.data = pickle.load(f) - unicode_dict = {} - for k, v in self.data.items(): - print(k) - unicode_dict[decode(k, errors="replace")] = v - return unicode_dict - except (IOError, EOFError, pickle.UnpicklingError): - pass - return {} # if everything fails, return an empty dictionary - def purge(self): """ Deletes all entries that no longer exist on system. @@ -228,16 +199,6 @@ class Database: del self.data[path] -def get_db_file(filename = "autojump.txt"): - """ - Retrieve full database path. - """ - # TODO: Remove when migration code is removed. - if CONFIG_DIR == os.path.expanduser("~"): - return CONFIG_DIR + "/." + filename - else: - return CONFIG_DIR + "/" + filename - def options(): """ Parse command line options. diff --git a/tests/runtests.py b/tests/runtests.py index 6c65272..94f592c 100755 --- a/tests/runtests.py +++ b/tests/runtests.py @@ -95,34 +95,6 @@ class TestAutojump(unittest.TestCase): os.remove(fname) os.remove(fname + '.bak') - def test_db_migration(self): - ORIG_CONFIG_DIR = autojump.CONFIG_DIR - try: - # setup - CONFIG_DIR = '/tmp/autojump_test_db_load_migrate_' + str(random.randint(0,32678)) - os.mkdir(CONFIG_DIR) - autojump.CONFIG_DIR = CONFIG_DIR - fname = CONFIG_DIR + '/autojump_py' - db = autojump.Database(fname) - db.add('/1') - db.add('/2') - shutil.copy(fname, fname + '.bak') - - # test - missing_fname = '/tmp/autojump_test_db_load_missing_' + str(random.randint(0,32678)) + '.txt' - db = autojump.Database(missing_fname) - - db.add('/3') - self.assertEquals(len(db.data), 3) - - finally: - # teardown - shutil.rmtree(CONFIG_DIR) - os.remove(missing_fname) - os.remove(missing_fname + '.bak') - - autojump.CONFIG_DIR = ORIG_CONFIG_DIR - def test_db_purge(self): self.db.add('/1') self.db.purge()