Commit 7dcfef03 authored by Kai Willadsen's avatar Kai Willadsen

ui.historyentry: Don't use SafeConfigParser's interpolation (bgo#757659)

SafeConfigParser and ConfigParser both support magical interpolation
syntax that we don't use. Better yet, these classes will happily write
out values that they can't read back in, and traceback when you try.

Specifically, in this case a commit message containing "%PARAM1%" broke
the parsers because they considered this to be an invalid interpolation
string, but only errored the *next* time you tried to change the config
file.

This commit just moves us to using RawConfigParser because we don't
want the magical behaviour.
parent 1c95f056
......@@ -88,7 +88,7 @@ class HistoryCombo(Gtk.ComboBox):
os.makedirs(pref_dir)
self.history_file = os.path.join(pref_dir, "history.ini")
self.config = configparser.SafeConfigParser()
self.config = configparser.RawConfigParser()
if os.path.exists(self.history_file):
self.config.read(self.history_file)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment