Yay! This is great! LGTM with comments.
http://codereview.chromium.org/9192010/diff/1/tools/presubmit.py File tools/presubmit.py (left): http://codereview.chromium.org/9192010/diff/1/tools/presubmit.py#oldcode231 tools/presubmit.py:231: return process.returncode == 0 I think we still want a return value here. IIUC, it should be: return sum(results) == 0 http://codereview.chromium.org/9192010/diff/1/tools/presubmit.py File tools/presubmit.py (right): http://codereview.chromium.org/9192010/diff/1/tools/presubmit.py#newcode125 tools/presubmit.py:125: child.kill() I think you meant s/child/process/ http://codereview.chromium.org/9192010/diff/1/tools/presubmit.py#newcode129 tools/presubmit.py:129: child.kill() same here. http://codereview.chromium.org/9192010/diff/1/tools/presubmit.py#newcode246 tools/presubmit.py:246: results = pool.map(CppLintWorker, commands) I need the following patch to make this react properly to Ctrl+C: - results = pool.map(CppLintWorker, commands) + # workaround for http://bugs.python.org/issue8296 + results = pool.map_async(CppLintWorker, commands).get(999999) http://codereview.chromium.org/9192010/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
