➜ 
This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on 2014-01-23 09:19 by Arfrever, last changed 2022-04-11 14:57 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| python-bytes_warnings.patch | Arfrever, 2014-01-23 09:19 | |||
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 4734 | closed | ncoghlan, 2017-12-07 07:24 | |
| PR 4820 | merged | vstinner, 2017-12-12 21:40 | |
| PR 5283 | merged | vstinner, 2018-01-23 11:24 | |
| Messages (8) | |||
|---|---|---|---|
| msg208894 - (view) | Author: Arfrever Frehtes Taifersar Arahesis (Arfrever) *  | Date: 2014-01-23 09:19 | |
Currently -W command line options and PYTHONWARNINGS environmental variable non-intuitively override settings specified by -b / -bb command line options. $ python3.4 -c 'print("" == b"")' False $ python3.4 -b -c 'print("" == b"")' -c:1: BytesWarning: Comparison between bytes and string False $ python3.4 -bb -c 'print("" == b"")' Traceback (most recent call last): File "<string>", line 1, in <module> BytesWarning: Comparison between bytes and string $ python3.4 -We -b -c 'print("" == b"")' Traceback (most recent call last): File "<string>", line 1, in <module> BytesWarning: Comparison between bytes and string $ python3.4 -Wd -bb -c 'print("" == b"")' -c:1: BytesWarning: Comparison between bytes and string False $ python3.4 -Wi -b -c 'print("" == b"")' False $ python3.4 -Wi -bb -c 'print("" == b"")' False $ I suggest to increase priority of -b / -bb command line options. The attached patch should be applied after patch from issue #20355. (Test suite of Python triggers some BytesWarnings, so firstly they will have to be fixed.) | |||
| msg208898 - (view) | Author: Arfrever Frehtes Taifersar Arahesis (Arfrever) * | Date: 2014-01-23 09:34 | |
By the way, Tools/scripts/run_tests.py (used by 'test' target in Makefile) passes '-W default -bb', which currently fails work as intended (i.e. treating BytesWarnings as errors). | |||
| msg212095 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) *  | Date: 2014-02-24 13:33 | |
Final patch should remove a workaround in Tools/scripts/run_tests.py. | |||
| msg217439 - (view) | Author: Antoine Pitrou (pitrou) * | Date: 2014-04-28 22:59 | |
The patch is a bit problematic, because Py_BytesWarningFlag may also be set by e.g. an application embedding Python, but then Python's main.c won't be executed. | |||
| msg307792 - (view) | Author: Alyssa Coghlan (ncoghlan) * | Date: 2017-12-07 07:37 | |
The approach I've taken in my PR is similar to the one that Arfrever proposed (albeit different in the details due to other changes in the startup code since that patch was written). For the embedding case, I've simply noted in the What's New section of the porting guide that embedding applications need to configure a `error::BytesWarning` filter explicitly if they want to treat BytesWarning as errors (in addition to setting the flag to request that the warnings be emitted in the first place). | |||
| msg308157 - (view) | Author: STINNER Victor (vstinner) * | Date: 2017-12-12 21:59 | |
New changeset 747f48e2e92390c44c72f52a1239959601cde157 by Victor Stinner in branch 'master': bpo-32230: Set sys.warnoptions with -X dev (#4820) https://github.com/python/cpython/commit/747f48e2e92390c44c72f52a1239959601cde157 | |||
| msg308158 - (view) | Author: STINNER Victor (vstinner) * | Date: 2017-12-12 22:01 | |
Ok, I merged Nick's PR: -b and -bb options now have the highest priority. I close the issue. | |||
| msg310500 - (view) | Author: STINNER Victor (vstinner) * | Date: 2018-01-23 12:30 | |
New changeset dc6b9462c00873c8404a7966b7ca210717718af5 by Victor Stinner in branch 'master': bpo-20361: Remove workaround for a now fixed bug (#5283) https://github.com/python/cpython/commit/dc6b9462c00873c8404a7966b7ca210717718af5 | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:57:57 | admin | set | github: 64560 |
| 2018-01-23 12:30:55 | vstinner | set | messages: + msg310500 |
| 2018-01-23 11:24:45 | vstinner | set | pull_requests: + pull_request5129 |
| 2018-01-23 11:24:37 | vstinner | set | versions: + Python 3.7, - Python 3.5 |
| 2017-12-12 22:01:13 | vstinner | set | status: open -> closed resolution: fixed messages: + msg308158 stage: patch review -> resolved |
| 2017-12-12 21:59:50 | vstinner | set | messages: + msg308157 |
| 2017-12-12 21:40:02 | vstinner | set | pull_requests: + pull_request4711 |
| 2017-12-07 07:37:39 | ncoghlan | set | messages: + msg307792 |
| 2017-12-07 07:24:18 | ncoghlan | set | stage: needs patch -> patch review pull_requests: + pull_request4647 |
| 2017-12-06 11:58:33 | vstinner | set | nosy: + ncoghlan, vstinner |
| 2017-12-06 11:47:47 | serhiy.storchaka | link | issue32231 superseder |
| 2016-10-19 21:57:18 | jdufresne | set | nosy: + jdufresne |
| 2014-06-01 15:28:24 | serhiy.storchaka | set | stage: patch review -> needs patch |
| 2014-04-28 22:59:58 | pitrou | set | messages: + msg217439 |
| 2014-04-03 20:14:00 | skrah | set | nosy: - skrah |
| 2014-02-24 13:33:13 | serhiy.storchaka | set | nosy: + serhiy.storchaka messages: + msg212095 |
| 2014-02-15 15:42:07 | ezio.melotti | set | nosy: + ezio.melotti |
| 2014-02-10 23:54:58 | jwilk | set | nosy: + jwilk |
| 2014-02-06 21:14:26 | Arfrever | set | versions: + Python 3.5, - Python 3.4 |
| 2014-01-23 09:48:19 | Arfrever | set | dependencies: + BytesWarnings triggerred by test suite |
| 2014-01-23 09:34:42 | Arfrever | set | messages: + msg208898 |
| 2014-01-23 09:20:26 | Arfrever | set | stage: patch review |
| 2014-01-23 09:19:49 | Arfrever | create | |