Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/utils.py

Issue 5744805892063232: Issue 1374 - Don`t hardcode web handlers for multiplexer (Closed)
Patch Set: Created Sept. 15, 2014, 9:58 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « .sitescripts.example ('k') | sitescripts/web.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/utils.py
===================================================================
--- a/sitescripts/utils.py
+++ b/sitescripts/utils.py
@@ -68,16 +68,17 @@ def get_config():
# Server-wide configuration if no custom found
paths.append('/etc/sitescripts')
paths.append('/etc/sitescripts.ini')
for path in paths:
path = os.path.abspath(path)
if os.path.exists(path):
config = SafeConfigParser()
+ config.optionxform = str
Sebastian Noack 2014/09/18 16:05:37 Why is this necessary? Won't that break for option
Wladimir Palant 2014/09/18 18:04:14 By default, SafeConfigParser will lower-case all o
Sebastian Noack 2014/09/18 19:01:07 How about a simple no-op? config.optionxform = la
Wladimir Palant 2014/09/18 19:12:22 Sounds like a good compromise. This will allow usi
config.read(path)
return config
raise Exception('No config file found. Please put sitescripts.ini into your home directory or /etc')
def setupStderr(stream=sys.stderr):
"""
Sets up sys.stderr to accept Unicode characters, redirects error output to
« no previous file with comments | « .sitescripts.example ('k') | sitescripts/web.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld