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

Issue 3390027: Add a clang/mac bot to the FYI waterfall. (Closed)

Created:
10 years, 3 months ago by Nico
Modified:
9 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Add a clang/mac bot to the FYI waterfall. It only builds Chromium.app for now, no unit test binaries. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=60697

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Total comments: 2

Patch Set 5 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+65 lines, -12 lines) Patch
M buildbot/master.chromium.fyi/master.cfg View 1 2 3 4 5 chunks +35 lines, -9 lines 0 comments Download
M buildbot/master.chromium.fyi/slaves.cfg View 1 2 3 4 1 chunk +7 lines, -0 lines 0 comments Download
M buildbot/scripts/slave/compile.py View 1 2 3 4 3 chunks +23 lines, -3 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
Nico
Needs to be rebaselined on top of evan's CL once that's in. The plan is ...
10 years, 2 months ago (2010-09-27 18:02:30 UTC) #1
Mark Mentovai
Can we do this so that the clang we use is checked in, and the ...
10 years, 2 months ago (2010-09-27 18:04:01 UTC) #2
M-A Ruel
lgtm, I'm fine with checking in clang the same way it's done with valgrind in ...
10 years, 2 months ago (2010-09-27 18:05:10 UTC) #3
Evan Martin
Ok, landed my change, please rebase http://codereview.chromium.org/3390027/diff/7001/8003 File buildbot/scripts/slave/compile.py (right): http://codereview.chromium.org/3390027/diff/7001/8003#newcode107 buildbot/scripts/slave/compile.py:107: env['CC'] = 'clang++' ...
10 years, 2 months ago (2010-09-27 18:07:19 UTC) #4
Nico
Mark: You mean check in the binary? If so, will do in a follow-up. http://codereview.chromium.org/3390027/diff/7001/8003 ...
10 years, 2 months ago (2010-09-27 19:03:03 UTC) #5
Mark Mentovai
thakis@chromium.org wrote: > Mark: You mean check in the binary? If so, will do in ...
10 years, 2 months ago (2010-09-27 19:05:01 UTC) #6
Evan Martin
FWIW, I don't think it's worth checking in a binary until we have a real ...
10 years, 2 months ago (2010-09-27 19:05:58 UTC) #7
Nico
On Mon, Sep 27, 2010 at 12:04 PM, Mark Mentovai <mark@chromium.org> wrote: > thakis@chromium.org wrote: ...
10 years, 2 months ago (2010-09-27 19:06:02 UTC) #8
Nico
10 years, 2 months ago (2010-09-27 19:09:36 UTC) #9
On Mon, Sep 27, 2010 at 12:05 PM,  <evan@chromium.org> wrote:
> FWIW, I don't think it's worth checking in a binary until we have a real
> builder
> for this (ETA like one month).  Otherwise it depends on Nico's workstation
> anyway.

I have a os x builder I can use (codf62).

Powered by Google App Engine
This is Rietveld 408576698