Remove Chromium code from Tools/BuildSlaveSupport
Created attachment 199963 [details] Patch
Comment on attachment 199963 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=199963&action=review Gosh, we keep cleaning and you keep finding crap everywhere :( > Tools/BuildSlaveSupport/built-product-archive:95 > - > - # Chromium bots may not have this directory > if not os.path.isdir(archiveDir): > os.mkdir(archiveDir) Shouldn't the if go away? If the comment is correct, it has become useless.
Comment on attachment 199963 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=199963&action=review >> Tools/BuildSlaveSupport/built-product-archive:95 >> os.mkdir(archiveDir) > > Shouldn't the if go away? > If the comment is correct, it has become useless. Yup, we should get rid of it. > Tools/BuildSlaveSupport/test-result-archive:-101 > - zipper = zipfile.ZipFile(archiveFile, 'w', zipfile.ZIP_DEFLATED) You can also remove zipfile from the import list.
Committed r149270: <http://trac.webkit.org/changeset/149270>
(In reply to comment #4) > Committed r149270: <http://trac.webkit.org/changeset/149270> ... with all the comments addressed, thanks for the reviews.