Commit 6434bc02 authored by Arthur Eubanks's avatar Arthur Eubanks Committed by Commit Bot

Point TODOs at newer bug

Bug: 1067752
Change-Id: Ib24d6e6542fc9a21e8105f8bcd9776ab34968ab7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2137542Reviewed-by: default avatarNico Weber <thakis@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Auto-Submit: Arthur Eubanks <aeubanks@google.com>
Cr-Commit-Position: refs/heads/master@{#758285}
parent a56c5993
......@@ -129,11 +129,7 @@ def CheckoutLLVM(commit, dir):
def UrlOpen(url):
# Normally we'd use urllib, but on our bots it can't connect to the GitHub API
# due to using too old TLS (see crbug.com/897796#c56). As a horrible
# workaround, shell out to curl instead. It seems curl is recent enough on all
# our machines that it can connect. On Windows it's in our gnuwin package.
# TODO(crbug.com/965937): Use urllib once our Python is recent enough.
# TODO(crbug.com/1067752): Use urllib once certificates are fixed.
return subprocess.check_output(['curl', '--silent', url])
......@@ -396,7 +392,7 @@ def main():
# The gnuwin package also includes curl, which is needed to interact with the
# github API below.
# TODO(crbug.com/965937): Use urllib once our Python is recent enough, and
# TODO(crbug.com/1067752): Use urllib once certificates are fixed, and
# move this down to where we fetch other build tools.
AddGnuWinToPath()
......
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