Merge lp:~shevonar/widelands-website/bug1202301 into lp:widelands-website

Proposed by Shevonar
Status: Merged
Merged at revision: 363
Proposed branch: lp:~shevonar/widelands-website/bug1202301
Merge into: lp:widelands-website
Diff against target: 12 lines (+2/-0)
1 file modified
widelandslib/conf.py (+2/-0)
To merge this branch: bzr merge lp:~shevonar/widelands-website/bug1202301
Reviewer Review Type Date Requested Status
Widelands Developers Pending
Review via email: mp+175470@code.launchpad.net

Description of the change

This should fix bug 1202301. However, I could not test it because I messed up my widelands-website installation when updating to current trunk. I tested the WidelandsConfigParser and it can now handle the % symbol in config files properly.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'widelandslib/conf.py'
2--- widelandslib/conf.py 2010-11-01 17:52:47 +0000
3+++ widelandslib/conf.py 2013-07-18 08:25:30 +0000
4@@ -30,6 +30,8 @@
5 except AttributeError:
6 string = open(fn, "r").read()
7
8+ string = string.replace('%', "%%")
9+
10 try:
11 self.readfp(cStringIO.StringIO(string))
12 except MissingSectionHeaderError:

Subscribers

People subscribed via source and target branches