[issue34624] -W option does not accept module regexes

2018-09-17 Thread Thomas Gläßle
Thomas Gläßle added the comment: Thanks for your response. I have opened a PR at [1] that would remove the re.escape such that the implementation matches the documentation if you decide that this is fine. [1]: https://github.com/python/cpython/pull/9358 Personally, I would go even further

[issue34624] -W option does not accept module regexes

2018-09-17 Thread Thomas Gläßle
Change by Thomas Gläßle : -- keywords: +patch pull_requests: +8781 stage: -> patch review ___ Python tracker ___ ___

[issue34624] -W option does not accept module regexes

2018-09-11 Thread Karthikeyan Singaravelan
Karthikeyan Singaravelan added the comment: The original escape was done with 2a862c614d6d3ff50dc644150670651fdc9f3a99 which was in 2000. The doc example you are referring to at [1] was added with https://bugs.python.org/issue31975 and there doesn't seem to be a test involving regular

[issue34624] -W option does not accept module regexes

2018-09-11 Thread Karthikeyan Singaravelan
Change by Karthikeyan Singaravelan : -- nosy: +xtreak ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34624] -W option does not accept module regexes

2018-09-10 Thread Thomas Gläßle
Thomas Gläßle added the comment: Very sorry, the example command above should read: python -Wi -W'default:::.*' -c 'import warnings; warnings.warn("This should show up")' -- ___ Python tracker

[issue34624] -W option does not accept module regexes

2018-09-10 Thread Thomas Gläßle
New submission from Thomas Gläßle : Hi, This command does not report a warning, while it should: python -c 'import warnings; warnings.warn("This should show up")' -Wi -W'default:::.*' If the regex `.*` is replaced by `__main__` it works as expected. Same applies for regexes in