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

Issue 10392073: Add a new gyp define to be used by the swarm master. (Closed)

Created:
8 years, 7 months ago by M-A Ruel
Modified:
8 years, 7 months ago
Reviewers:
MAD
CC:
chromium-reviews, erikwright (departed), cbentzel+watch_chromium.org, darin-cc_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

Add a new gyp define to be used by the swarm master. Rename tests_run to test_isolation_mode. TBR=mad@chromium.org BUG=98636 TEST= NOTRY=true Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137127

Patch Set 1 #

Patch Set 2 : Rename variables #

Patch Set 3 : Add slash, otherwise gyp does silly things #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -7 lines) Patch
M base/base.gyp View 1 1 chunk +2 lines, -1 line 0 comments Download
M build/common.gypi View 1 2 2 chunks +6 lines, -2 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 2 chunks +4 lines, -2 lines 0 comments Download
M net/net.gyp View 1 1 chunk +2 lines, -1 line 0 comments Download
M third_party/cacheinvalidation/cacheinvalidation.gyp View 1 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
M-A Ruel
Chris, what do you think about this? Then we could override this value in chromium.swarm ...
8 years, 7 months ago (2012-05-14 01:40:15 UTC) #1
csharp
On 2012/05/14 01:40:15, Marc-Antoine Ruel wrote: > Chris, what do you think about this? Then ...
8 years, 7 months ago (2012-05-14 12:49:49 UTC) #2
M-A Ruel
On 2012/05/14 12:49:49, csharp wrote: > On 2012/05/14 01:40:15, Marc-Antoine Ruel wrote: > > Chris, ...
8 years, 7 months ago (2012-05-14 13:27:47 UTC) #3
MAD
LGTM I think it's fine as is... Unless you think that these outdirs would eventually ...
8 years, 7 months ago (2012-05-14 21:51:08 UTC) #4
M-A Ruel
I'll keep on using test_isolation_foo, I 'think' it is a better name.
8 years, 7 months ago (2012-05-15 02:35:52 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/10392073/6002
8 years, 7 months ago (2012-05-15 14:55:49 UTC) #6
commit-bot: I haz the power
8 years, 7 months ago (2012-05-15 14:56:35 UTC) #7
Change committed as 137127

Powered by Google App Engine
This is Rietveld 408576698