Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(11)

Issue 1494883004: Revert of Fix downloading LLVM Gold plugin in the case, when Clang is up to date. (Closed)

Created:
5 years ago by Adrian Kuegel
Modified:
5 years ago
Reviewers:
Nico, krasin
CC:
chromium-reviews, pcc, kcc
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Fix downloading LLVM Gold plugin in the case, when Clang is up to date. (patchset #1 id:1 of https://codereview.chromium.org/1491203003/ ) Reason for revert: This breaks Codesearch bots. Original issue's description: > Fix downloading LLVM Gold plugin in the case, when Clang is up to date. > > The script should download LLVM Gold plugin, if it's missing, even if the > main Clang toolchain is up to date. This change is to match update.py > functionality with the gone update.sh. > > BUG=464797, 494442 > > Committed: https://crrev.com/7ff79f9569429444dabdb6d5024918bc548916b3 > Cr-Commit-Position: refs/heads/master@{#362803} TBR=thakis@chromium.org,krasin@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=464797, 494442 Committed: https://crrev.com/9e37ffbe43c0b100bcc02f9faf648f11d86316c1 Cr-Commit-Position: refs/heads/master@{#362938}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -11 lines) Patch
M tools/clang/scripts/update.py View 2 chunks +6 lines, -11 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
Adrian Kuegel
Created Revert of Fix downloading LLVM Gold plugin in the case, when Clang is up ...
5 years ago (2015-12-03 09:29:40 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1494883004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1494883004/1
5 years ago (2015-12-03 09:29:49 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years ago (2015-12-03 09:30:32 UTC) #3
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/9e37ffbe43c0b100bcc02f9faf648f11d86316c1 Cr-Commit-Position: refs/heads/master@{#362938}
5 years ago (2015-12-03 09:31:37 UTC) #5
Adrian Kuegel
5 years ago (2015-12-03 10:12:43 UTC) #6
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1493283002/ by akuegel@chromium.org.

The reason for reverting is: Reverting didn't help..

Powered by Google App Engine
This is Rietveld 408576698