logilab-mtconverter #5068 lynx transform leaves zombies around [validation pending]
Somehow there are cases where zombies of lynx are still around. The library calls os.popen4 which uses popen2.Popen4 internally and the __del__ method is probably not called. The code should use popen2.Popen4 explicitly or even better use subprocess. btw there's a potential for deadlock as the code is writing a whole file to a pipe before reading the results. With subprocess one can use a tmpfile for stdout and avoir that. | |
priority | minor |
---|---|
type | bug |
done in | 0.8.4 |
load left | 0.000 |
closed by | #9a74f342146f Fix Popen convensforms] use Popen.communicate (closes #5068, #92676) |
Comments
-
2008/06/30 09:18, written by adimascio
-
2009/04/09 10:56, written by acampeas
add commentwhile the bug still remains, it is not as blocking as before since the text2html transformation is not by default done with lynx anymore.
is there some good reason to keep lynx transform ?