Skip to content

Commit 923de3f

Browse files
committed
Handle configparser.InterpolationSyntaxError
Under Python 3, an option value with a single % (e.g. this % is a test) throws configparser.InterpolationSyntaxError: '%' must be followed by '%' or '(', found: '%s))' Added code to capture this, raise a different exception. roundup-admin handles the error and exits cleanly. Other code shows the traceback. The new error message reports the file, section and option causing the problem to allow easier repair. Also updated roundup translations and added tests.
1 parent 3b9e87c commit 923de3f

File tree

5 files changed

+308
-206
lines changed

5 files changed

+308
-206
lines changed

CHANGES.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,9 @@ Fixed:
5656
config_ini.ini overrides. Needed for jinja to set template lang etc.
5757
Recognize minimal template when presented with a full
5858
path. (John Kristensen (jerrykan) and John Rouillard)
59+
- handle configparser.InterpolationSyntaxError raised if value
60+
has a single %. Seems to afect python 3 only. Reported by
61+
nomicon on IRC. (John Rouillard)
5962

6063
Features:
6164

0 commit comments

Comments
 (0)