R. David Murray added the comment: Issue 12890 fix is now committed. Perhaps I should have worked with yours instead...but I didn't. The test solution in that patch, calling python in a separate process, is conceptually simpler but will take more time to run. I'm on the fence as to which one I prefer, having looked at your patch now...but I'm not sure that your patch cleans up properly, and that fact that that is difficult to figure out argues a bit for the cleaner subprocess solution. I much prefer tests that run fast, though...
---------- nosy: +r.david.murray _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue15749> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
