[issue20242] logging style parameter does not work correctly

2014-01-13 Thread Roundup Robot
Roundup Robot added the comment: New changeset e7fcf0d8008f by Vinay Sajip in branch '3.3': Issue #20242: Fixed basicConfig() format strings for the alternative formatting styles. http://hg.python.org/cpython/rev/e7fcf0d8008f New changeset c1605d24fb35 by Vinay Sajip in branch 'default': Closes

[issue20242] logging style parameter does not work correctly

2014-01-13 Thread Vinay Sajip
Vinay Sajip added the comment: On further reflection, I will implement this slightly differently. The tests I will commit as is, except for stripping newlines before the comparison (to avoid worrying about differences in line separators across different platforms). However, the correct fix IMO

[issue20242] logging style parameter does not work correctly

2014-01-13 Thread Vinay Sajip
Vinay Sajip added the comment: Thanks for this. The patch looks good, and I expect to implement it shortly. -- assignee: -> vinay.sajip ___ Python tracker ___ __

[issue20242] logging style parameter does not work correctly

2014-01-13 Thread Ned Deily
Changes by Ned Deily : -- nosy: +vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue20242] logging style parameter does not work correctly

2014-01-13 Thread kespindler
kespindler added the comment: I created a patch that fixes this behavior for both { and $ style formats. Previously, default format strings were not being set correctly for those styles. This patch ensures that the correct default_format string is used. There is a minor "weirdness" issue that,

[issue20242] logging style parameter does not work correctly

2014-01-13 Thread kespindler
New submission from kespindler: In Python 3.3, 3.4beta2, and bleeding edge (88411:e7d922d8ee03), the following occurs >>> import logging >>> logging.basicConfig(style="{") >>> logging.error("hello") %(levelname)s:%(name)s:%(message)s Of course, I would expect ERROR:root:hello to be output