Commit 59206c22 authored by Nico Weber's avatar Nico Weber

mac: Make do-I-need-to-download-lld check actually work.

Bug: 774209
Change-Id: Ia95ff7b9bb74d4b69067c121d0a95af96331199e
Reviewed-on: https://chromium-review.googlesource.com/723941
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: default avatarHans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509549}
parent 5d54a0c0
...@@ -24,7 +24,7 @@ def AlreadyUpToDate(): ...@@ -24,7 +24,7 @@ def AlreadyUpToDate():
# lld-link seems to have no flag to query the version; go to ld.lld instead. # lld-link seems to have no flag to query the version; go to ld.lld instead.
# TODO(thakis): Remove ld.lld workaround if https://llvm.org/PR34955 is fixed. # TODO(thakis): Remove ld.lld workaround if https://llvm.org/PR34955 is fixed.
lld_rev = subprocess.check_output([LD_LLD_PATH, '--version']) lld_rev = subprocess.check_output([LD_LLD_PATH, '--version'])
return (re.match(r'LLD.*\(trunk (\d+)\)', lld_rev).group(1) != return (re.match(r'LLD.*\(trunk (\d+)\)', lld_rev).group(1) ==
update.CLANG_REVISION) update.CLANG_REVISION)
......
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