classification
Title: PYTHONFAULTHANDLER enables faulthandler when the variable is empty
Type: Stage: resolved
Components: Versions: Python 3.3, Python 3.4
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: python-dev, vajrasky, vstinner
Priority: normal Keywords: patch

Created on 2013-09-07 08:34 by vstinner, last changed 2013-09-08 09:37 by vstinner. This issue is now closed.

Files
File name Uploaded Description Edit
pythonfaulthandler_env_var.patch vajrasky, 2013-09-07 10:12 review
Messages (5)
msg197142 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2013-09-07 08:34
Most Python environment variables (PYTHON*) only enable something when their value is non-empty. The PYTHONFAULTHANDLER enables the faulthandler even if the value is empty. As the author of the module, I think that it is a bug and should be fixed (even in Python 3.3).
msg197145 - (view) Author: Vajrasky Kok (vajrasky) * Date: 2013-09-07 10:12
Do you mean something like *this*?

this -> the patch, pythonfaulthandler_env_var.patch.
msg197198 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2013-09-07 23:24
Yes. Maybe also with a test. The doc of all env vars must be checked.
msg197235 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013-09-08 09:36
New changeset 5c2cf4349adc by Victor Stinner in branch 'default':
Close #18957: The PYTHONFAULTHANDLER environment variable now only enables the
http://hg.python.org/cpython/rev/5c2cf4349adc
msg197236 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2013-09-08 09:37
> Do you mean something like *this*?
> this -> the patch, pythonfaulthandler_env_var.patch.

Thanks for the patch, I combined it with other changes.
History
Date User Action Args
2013-09-08 09:37:17vstinnersetmessages: + msg197236
2013-09-08 09:36:47python-devsetstatus: open -> closed

nosy: + python-dev
messages: + msg197235

resolution: fixed
stage: resolved
2013-09-07 23:24:03vstinnersetmessages: + msg197198
2013-09-07 10:12:07vajraskysetfiles: + pythonfaulthandler_env_var.patch

nosy: + vajrasky
messages: + msg197145

keywords: + patch
2013-09-07 08:34:01vstinnercreate