LGTM with two comments.
https://chromiumcodereview.appspot.com/10383128/diff/1/DEPS File DEPS (right): https://chromiumcodereview.appspot.com/10383128/diff/1/DEPS#newcode27 DEPS:27: ] Make sure the commit doesn't include the following lines. https://chromiumcodereview.appspot.com/10383128/diff/1/test/test262/testcfg.py File test/test262/testcfg.py (right): https://chromiumcodereview.appspot.com/10383128/diff/1/test/test262/testcfg.py#newcode125 test/test262/testcfg.py:125: archive.extractall(u'\\\\?\\%s' % self.root) Maybe add a comment here. https://chromiumcodereview.appspot.com/10383128/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
