[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Antoine Pitrou added the comment: Committed. Thanks, Arfrever! -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Roundup Robot added the comment: New changeset 78c0d80a04f9 by Antoine Pitrou in branch 'default': Issue #20355: -W command line options now have higher priority than the PYTHONWARNINGS environment variable. Patch by Arfrever. http://hg.python.org/cpython/rev/78c0d80a04f9 New changeset 925c0b6

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Changes by Stefan Krah : -- nosy: -skrah ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.o

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Serhiy Storchaka added the comment: Final patch should remove a workaround in Tools/scripts/run_tests.py. -- nosy: +serhiy.storchaka ___ Python tracker ___ __

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Larry Hastings added the comment: Okay, then yes, let's save this for 3.5. Thanks! -- ___ Python tracker ___ ___ Python-bugs-list mai

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Brett Cannon added the comment: I agree with Antoine; this can wait for Python 3.5. -- ___ Python tracker ___ ___ Python-bugs-list mai

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Changes by Arfrever Frehtes Taifersar Arahesis : Removed file: http://bugs.python.org/file33645/python-warnings.patch ___ Python tracker ___ _

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Arfrever Frehtes Taifersar Arahesis added the comment: The new patch includes changes suggested by Antoine. (This patch should be applied after patch from issue #20373.) -- Added file: http://bugs.python.org/file33673/python-warnings.patch ___ Python

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Changes by Arfrever Frehtes Taifersar Arahesis : -- dependencies: +Use test.script_helper.assert_python_ok() instead of subprocess.* in test.test_warnings ___ Python tracker ___

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Antoine Pitrou added the comment: > Is this behavior causing problems? How bad is it? Not really. It's just a bit suboptimal and counter-intuitive (local settings should trump global settings). -- ___ Python tracker

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Larry Hastings added the comment: Is this behavior causing problems? How bad is it? Can I get more opinions about this than just Arfrever and Antoine? Not that I don't trust their opinions, I just want to see a larger sample size. -- ___ Python tr

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Antoine Pitrou added the comment: About the patch (note the code review tool hasn't accepted it, so I'm commenting here): - in the tests, I think comparing stderr as-is will fail under Windows, where the line separator is b"\r\n"; better call splitlines() first - in the tests, I think it would

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Antoine Pitrou added the comment: Well, 3.4 isn't supposed to receive behaviour changes anymore, unless Larry says it's ok. -- ___ Python tracker ___ ___

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Changes by Arfrever Frehtes Taifersar Arahesis : -- stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mai

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Arfrever Frehtes Taifersar Arahesis added the comment: The attached patch moves calls to PySys_AddWarnOption* for -W options to after parsing PYTHONWARNINGS. I think that this bug fix could be applied in Python 3.4. The patch does not introduce any failures in full test suite. -- keyw

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

Antoine Pitrou added the comment: Agreed. -- nosy: +brett.cannon stage: -> needs patch type: behavior -> enhancement versions: +Python 3.5 -Python 2.7, Python 3.3, Python 3.4 ___ Python tracker ___

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

New submission from Arfrever Frehtes Taifersar Arahesis: Currently Python has non-intuitive behavior such that PYTHONWARNINGS environmental variable has higher priority than -W command line options. $ python3.4 -c '__import__("warnings").warn("xxx", DeprecationWarning)' $ python3.4 -Wd -c '__im