I am using a modified version of subprocess.py,

where I have removed the _active list and all
references to it.

I have tested it (under Windows 2000) and there were
no errors.

So what is the reason for managing the _active list
at all? Why not drop it?

Logged In: YES
user_id=1488657

I agree.

The use of _active makes subprocess.py thread-UNsafe.

See also: Bug bpo-1199282

In order to have a thread-safe subprocess.py, I commented
out the call to _cleanup() in Popen.__init__(). As a side
effect, _active becomes useless.

Logged In: YES
user_id=21627

The purpose of the _active list is to wait(2) for open
processes. It needs to stay.

Logged In: YES
user_id=33168

If you always called wait() the _active list isn't
beneficial to you. However, many people do not call wait
and the _active list provides a mechanism to cleanup zombied
children. This is important for many users.

If you need thread safely, you can handle the locking
yourself before calling poll()/wait().

Logged In: YES
user_id=1276121

what happens if you are doing a _cleanup (iterating over a
copy of _active) in multiple threads?
can it not happen then that you clean up a process 2 times?

thread 1 starts a _cleanup: makes a copy of _active[:] and
starts polling
thread 2 starts a _cleanup: makes a copy of _active[:] and
starts polling

thread 1 encounters a finished process and removes it from
_active[]
thread 2 does not know the process is removed, finds the
same process finished and tries to remove it from _active
but this fails, because thread 1 removed it already

so the action of cleaning up should maybe be serialized
if 1 thread is doing it, the other one should block

everyone who needs this can of course patch the
subprocess.py file, but shouldn't this be fixed in the library?

Logged In: YES
user_id=1276121

the same problem probably exists in popen2.py
there _active is also used
so if something is fixed in subprocess.py, it should
probably also be fixed in popen2.py

Logged In: YES
user_id=1276121

also bpo-1214859 is interesting, has a patch

Logged In: YES
user_id=21627

attila-cheops, please read the 2.5 version of popen2.

popen2 now only adds processes to _active in __del__, not in
__init__, so the problem with the application still wanting
to wait/poll is solved.

Multiple threads simultaneously isn't a problem, since it is
(or should be) harmless to invoke poll on a process that has
already been waited for. For only one of the poll calls, the
wait will actually succeed, and that should be the one that
removes it from the _active list.

The same strategy should be applied to subprocess.

Logged In: YES
user_id=1276121

the implementation in 2.5 of popen2 seems good
if you or astrand could patch subprocess.py accordingly,
that would be great.

Logged In: YES
user_id=1276121

see patch bpo-1467770

Logged In: YES
user_id=21627

This was fixed with said patch.