
I have a need for a timeout feature in some of the subprocess.Popen methods. I looked around and found this bug http://bugs.python.org/issue5673 As it seems it the activity on the issue just stopped. I would like to take the reins on the issue and tie the loose ends. The current state is "test needed". If it is updated, what tests are missing for it to qualify? What other comments do you have on the subject?

Saggi Mizrahi <smizrahi@...> writes:
current
state is "test needed".
This looks misleading, since there /are/ unit tests in the proposed patch. I've switched the state to "patch review". It merely needs someone to review the patch for errors/issues/etc., and then a committer to ok it (but the reviewer needn't be a committer, though). Don't hesitate to chime in on the issue. Regards Antoine.

Saggi Mizrahi <smizrahi@...> writes:
current
state is "test needed".
This looks misleading, since there /are/ unit tests in the proposed patch. I've switched the state to "patch review". It merely needs someone to review the patch for errors/issues/etc., and then a committer to ok it (but the reviewer needn't be a committer, though). Don't hesitate to chime in on the issue. Regards Antoine.
participants (2)
-
Antoine Pitrou
-
Saggi Mizrahi