changeset:   90321:2d33cbf02522
user:        Vinay Sajip <vinay_sajip@yahoo.co.uk>
date:        Tue Apr 15 14:24:53 2014 +0100
files:       Lib/logging/config.py Misc/NEWS
description:
Closes #21203: Updated fileConfig and dictConfig to remove inconsistencies. Thanks to Jure Koren for the patch.


diff -r 947515cc8957 -r 2d33cbf02522 Lib/logging/config.py
--- a/Lib/logging/config.py	Tue Apr 15 13:56:36 2014 +0100
+++ b/Lib/logging/config.py	Tue Apr 15 14:24:53 2014 +0100
@@ -116,11 +116,12 @@
         sectname = "formatter_%s" % form
         fs = cp.get(sectname, "format", raw=True, fallback=None)
         dfs = cp.get(sectname, "datefmt", raw=True, fallback=None)
+        stl = cp.get(sectname, "style", raw=True, fallback='%')
         c = logging.Formatter
         class_name = cp[sectname].get("class")
         if class_name:
             c = _resolve(class_name)
-        f = c(fs, dfs)
+        f = c(fs, dfs, stl)
         formatters[form] = f
     return formatters
 
@@ -660,7 +661,12 @@
             fmt = config.get('format', None)
             dfmt = config.get('datefmt', None)
             style = config.get('style', '%')
-            result = logging.Formatter(fmt, dfmt, style)
+            cname = config.get('class', None)
+            if not cname:
+                c = logging.Formatter
+            else:
+                c = _resolve(cname)
+            result = c(fmt, dfmt, style)
         return result
 
     def configure_filter(self, config):
diff -r 947515cc8957 -r 2d33cbf02522 Misc/NEWS
--- a/Misc/NEWS	Tue Apr 15 13:56:36 2014 +0100
+++ b/Misc/NEWS	Tue Apr 15 14:24:53 2014 +0100
@@ -46,6 +46,9 @@
 Library
 -------
 
+- Issue #21203: Updated fileConfig and dictConfig to remove inconsistencies.
+  Thanks to Jure Koren for the patch.
+
 - Issue #21197: Add lib64 -> lib symlink in venvs on 64-bit non-OS X POSIX.
 
 - Issue #17498: Some SMTP servers disconnect after certain errors, violating