Make PYTHONINSPECT check consistent

StanFromIreland · StanFromIreland · commit 89db20038666 · 2025-12-24T11:44:29.000Z
diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2025-12-24-11-39-59.gh-issue-143135.3d5ovx.rst b/Misc/NEWS.d/next/Core_and_Builtins/2025-12-24-11-39-59.gh-issue-143135.3d5ovx.rst
@@ -0,0 +1 @@
+Fix :envvar:`PYTHONINSPECT` to correctly treat ``0`` as false.
diff --git a/Modules/main.c b/Modules/main.c
@@ -576,8 +576,12 @@ pymain_repl(PyConfig *config, int *exitcode)
 {
     /* Check this environment variable at the end, to give programs the
        opportunity to set it from Python. */
-    if (!config->inspect && _Py_GetEnv(config->use_environment, "PYTHONINSPECT")) {
-        pymain_set_inspect(config, 1);
+    if (!config->inspect) {
+        int inspect_env = 0;
+        _Py_get_env_flag(config->use_environment, &inspect_env, "PYTHONINSPECT");
+        if (inspect_env) {
+            pymain_set_inspect(config, 1);
+        }
     }
 
     if (!(config->inspect && stdin_is_interactive(config) && config_run_code(config))) {

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+Fix :envvar:`PYTHONINSPECT` to correctly treat ``0`` as false.
Original file line number	Diff line number	Diff line change
`@@ -576,8 +576,12 @@ pymain_repl(PyConfig config, int exitcode)`
`576`	`576`	`{`
`577`	`577`	`/* Check this environment variable at the end, to give programs the`
`578`	`578`	`opportunity to set it from Python. */`
`579`		`- if (!config->inspect && _Py_GetEnv(config->use_environment, "PYTHONINSPECT")) {`
`580`		`- pymain_set_inspect(config, 1);`
	`579`	`+ if (!config->inspect) {`
	`580`	`+ int inspect_env = 0;`
	`581`	`+ _Py_get_env_flag(config->use_environment, &inspect_env, "PYTHONINSPECT");`
	`582`	`+ if (inspect_env) {`
	`583`	`+ pymain_set_inspect(config, 1);`
	`584`	`+ }`
`581`	`585`	`}`
`582`	`586`
`583`	`587`	`if (!(config->inspect && stdin_is_interactive(config) && config_run_code(config))) {`