Commit 46bbd542 authored by Corentin Wallez's avatar Corentin Wallez Committed by Chromium LUCI CQ

tool_wrapper.py: Use bytes in ExecLinkWrapper

While compiling Dawn with MSVC python3 complained that startswith's
arguments needed to be bytes.

Bug: None
Change-Id: Id6d1a1d536e4e235eacf1818d31e59a6b5b7e122
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643277
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Auto-Submit: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: default avatarDirk Pranke <dpranke@google.com>
Cr-Commit-Position: refs/heads/master@{#846017}
parent 89d93205
......@@ -141,9 +141,9 @@ class WinTool(object):
# Read output one line at a time as it shows up to avoid OOM failures when
# GBs of output is produced.
for line in link.stdout:
if (not line.startswith(' Creating library ') and
not line.startswith('Generating code') and
not line.startswith('Finished generating code')):
if (not line.startswith(b' Creating library ')
and not line.startswith(b'Generating code')
and not line.startswith(b'Finished generating code')):
print(line)
return link.wait()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment