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

Issue 197283006: Update common.gypi to set clang=1 for all iOS builds. (Closed)

Created:
6 years, 9 months ago by rohitrao (ping after 24h)
Modified:
6 years, 9 months ago
Reviewers:
Mark Mentovai, Nico
CC:
chromium-reviews, stuartmorgan, justincohen, sdefresne
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Update common.gypi to set clang=1 for all iOS builds. All iOS builds now use clang, regardless of which gyp generator is used, so change common.gypi to reflect that. NOTRY=True BUG=352167 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=256923

Patch Set 1 #

Patch Set 2 : Retry. #

Patch Set 3 : Rebased. #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -47 lines) Patch
M build/common.gypi View 1 2 7 chunks +18 lines, -47 lines 1 comment Download

Messages

Total messages: 15 (0 generated)
rohitrao (ping after 24h)
This turned out to be much smaller than I was expecting. There are "clang==1" conditions ...
6 years, 9 months ago (2014-03-13 15:36:22 UTC) #1
Nico
lgtm as far as I can tell. Thanks! https://codereview.chromium.org/197283006/diff/40001/build/common.gypi File build/common.gypi (right): https://codereview.chromium.org/197283006/diff/40001/build/common.gypi#newcode1534 build/common.gypi:1534: # ...
6 years, 9 months ago (2014-03-13 15:43:37 UTC) #2
Mark Mentovai
LGTM
6 years, 9 months ago (2014-03-13 15:56:04 UTC) #3
rohitrao (ping after 24h)
The CQ bit was checked by rohitrao@chromium.org
6 years, 9 months ago (2014-03-13 16:30:49 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rohitrao@chromium.org/197283006/40001
6 years, 9 months ago (2014-03-13 16:32:04 UTC) #5
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-13 19:47:29 UTC) #6
commit-bot: I haz the power
Retried try job too often on win_rel for step(s) browser_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&number=281493
6 years, 9 months ago (2014-03-13 19:47:30 UTC) #7
rohitrao (ping after 24h)
The CQ bit was checked by rohitrao@chromium.org
6 years, 9 months ago (2014-03-13 20:31:56 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rohitrao@chromium.org/197283006/40001
6 years, 9 months ago (2014-03-13 20:33:17 UTC) #9
rohitrao (ping after 24h)
The CQ bit was unchecked by rohitrao@chromium.org
6 years, 9 months ago (2014-03-13 20:37:24 UTC) #10
rohitrao (ping after 24h)
Adding NOTRY=True because I've hit four broken win_rel bots in a row, two of those ...
6 years, 9 months ago (2014-03-13 20:39:27 UTC) #11
rohitrao (ping after 24h)
The CQ bit was checked by rohitrao@chromium.org
6 years, 9 months ago (2014-03-13 20:39:31 UTC) #12
Nico
(Doesn't that invalidate your argument for ignoring my comment?)
6 years, 9 months ago (2014-03-13 20:40:46 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rohitrao@chromium.org/197283006/40001
6 years, 9 months ago (2014-03-13 20:41:16 UTC) #14
commit-bot: I haz the power
6 years, 9 months ago (2014-03-13 20:46:03 UTC) #15
Message was sent while issue was closed.
Change committed as 256923

Powered by Google App Engine
This is Rietveld 408576698