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

Issue 2576553002: Roll clang 288545:288545. (Closed)

Created:
4 years ago by krasin1
Modified:
4 years ago
Reviewers:
Nico
CC:
chromium-reviews, eugenis+clang_chromium.org, vmpstr+watch_chromium.org, Lei Zhang, dsinclair, yunlian, glider+clang_chromium.org, Nico, ukai+watch_chromium.org, Reid Kleckner, dmikurube+clang_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Roll clang 288545:288545. Ran `./tools/clang/scripts/upload_revision.py 288545 2016121301`. BUG=

Patch Set 1 #

Patch Set 2 : Checkout LLD #

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

Messages

Total messages: 5 (1 generated)
Nico
https://codereview.chromium.org/2576553002/diff/20001/tools/clang/scripts/update.py File tools/clang/scripts/update.py (right): https://codereview.chromium.org/2576553002/diff/20001/tools/clang/scripts/update.py#newcode460 tools/clang/scripts/update.py:460: if sys.platform == 'win32' or sys.platform.startswith('linux') or use_head_revision: Sending ...
4 years ago (2016-12-14 23:39:13 UTC) #2
hans
On 2016/12/14 23:39:13, Nico wrote: > https://codereview.chromium.org/2576553002/diff/20001/tools/clang/scripts/update.py > File tools/clang/scripts/update.py (right): > > https://codereview.chromium.org/2576553002/diff/20001/tools/clang/scripts/update.py#newcode460 > ...
4 years ago (2016-12-14 23:42:56 UTC) #3
krasin1
On 2016/12/14 23:39:13, Nico wrote: > https://codereview.chromium.org/2576553002/diff/20001/tools/clang/scripts/update.py > File tools/clang/scripts/update.py (right): > > https://codereview.chromium.org/2576553002/diff/20001/tools/clang/scripts/update.py#newcode460 > ...
4 years ago (2016-12-14 23:46:44 UTC) #4
krasin1
4 years ago (2016-12-14 23:47:38 UTC) #5
But yes, we shall make the bot less fragile, as this particular try was to test
that everything works on the bot. Locally, it worked even without the change.

Powered by Google App Engine
This is Rietveld 408576698