[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Roundup Robot added the comment: New changeset e35eafc872cd by Vinay Sajip in branch '2.7': Issue #12718: Set importer on instance if Python function, to avoid bad interaction with winpdb. http://hg.python.org/cpython/rev/e35eafc872cd -- ___ Python t

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

R. David Murray added the comment: See also issue 11470. I admit that this topic confuses me; I'm not sure where the bug is, but it seems to me from reading that issue that assuming the behavior of a builtin is not correct. -- ___ Python tracker <

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Vinay Sajip added the comment: As suggested in the documentation, you could assign the importer as an attribute of the configurator *instance*, in which case it shouldn't be treated as a bound method. What happens if you use the suggestion in the documentation change referred to here? ---

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Bluebird75 added the comment: This issue is closed, but the problems remain that the current version of winpdb is incompatible with django + python 2.7 . I don't really understsand why the issue is closed without any change to python logging library itself. To sum-up the issue: 1. django uses

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Changes by R. David Murray : -- nosy: +ncoghlan ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Roundup Robot added the comment: New changeset 0fbd44e3f342 by Vinay Sajip in branch '2.7': Issue #12718: Add documentation on using custom importers. http://hg.python.org/cpython/rev/0fbd44e3f342 New changeset 1e96a4406565 by Vinay Sajip in branch '3.2': Issue #12718: Add documentation on usin

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Vinay Sajip added the comment: Okay, the problem occurs because builtin functions and Python functions are handled differently. This is how you need to set a Python importer: vinay@eta-natty:~$ python Python 2.7.1+ (r271:86832, Apr 11 2011, 18:05:24) [GCC 4.5.2] on linux2 Type "help", "copyri

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Александр added the comment: As I understand, you created importer as a class attribute, so we can opportunity set import function to importer when inheritance new class from BaseConfigurator(# We might want to use a different one, e.g. importlib)? As example importer = import_module # from i

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Changes by Vinay Sajip : -- status: open -> pending ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Vinay Sajip added the comment: BTW I just tested on PyPy. The following program: import logging.config import sys d = { 'version': 1, 'disable_existing_loggers': False, 'handlers': { 'console': { 'level': 'DEBUG', 'class': 'logging.StreamHandler',

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Vinay Sajip added the comment: @Александр: It still works for me without winpdb/rpdb2, so you should probably report this there. Can you confirm that it works as expected when you do "python manage.py help"? vinay@eta-natty:/tmp$ virtualenv -p python2.7 bbbenv Running virtualenv with interpre

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

R. David Murray added the comment: Vinay: you want to mark importer as a static method anyway, otherwise it won't work with pypy. I don't know why this would show up on CPython, though. -- nosy: +r.david.murray ___ Python tracker

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Александр added the comment: Install rpdb2 or winpdb for you python. And install django to you python path. Create empty djnago-project: django-admin createproject bbbtest Add line LOGGING_CONFIG = 'logging.config.dictConfig' to setting.py for use python logging.config for logging. Execute r

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Vinay Sajip added the comment: It works as expected for me: vinay@eta-jaunty:~/projects/python/2.7$ python2.7 Python 2.7.2+ (2.7:7d5a37ce42d5, Aug 10 2011, 09:20:30) [GCC 4.3.3] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> import logging.config [62163 re

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

Changes by Benjamin Peterson : -- assignee: -> vinay.sajip nosy: +vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing lis

[issue12718] Logical mistake of importer method in logging.config.BaseConfigurator

New submission from Александр : At line 377 __import__ function assing to class variable importer. __import__ get module name as first argument, but when we use self.importer at line 391 (http://hg.python.org/cpython/file/10f20ad2fbb6/Lib/logging/config.py#l391) we give class instance(self) a