Neal Norwitz <[EMAIL PROTECTED]> added the comment: Thanks for the patch. It would be nice to get more instrumentation like coverage, performance, etc.
Here are some things I noticed while reviewing the patch: * This won't work on unix other than OSX. Can you change ../../python.exe to sys.executable? * How big is figleaf? Should you try to read/write the file in chunks? * optparse doesn't seem to be used. * Can you change `cat file` to read the file and pass those as arguments so this could work on windows? BTW, does figleaf work on Windows? ---------- nosy: +nnorwitz __________________________________ Tracker <[EMAIL PROTECTED]> <http://bugs.python.org/issue2403> __________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
