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

Issue 10984012: Add support for extra_gyp_defines in factory props (Closed)

Created:
8 years, 3 months ago by Isaac (away)
Modified:
8 years, 3 months ago
Reviewers:
Yaron
CC:
chromium-reviews, peter+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy+watch_chromium.org
Visibility:
Public.

Description

Add support for extra_gyp_defines in factory props - Will be used for clang builder, and possibly other bot customizations. R=yfriedman@chromium.org BUG=143931 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=158474

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M build/android/buildbot/buildbot_functions.sh View 2 chunks +3 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Isaac (away)
8 years, 3 months ago (2012-09-24 23:49:38 UTC) #1
Yaron
lgtm
8 years, 3 months ago (2012-09-25 00:01:30 UTC) #2
Isaac (away)
8 years, 3 months ago (2012-09-25 00:58:29 UTC) #3
Isaac's CQ is trying da patch

$ git cl dcommit
Cannot dcommit with a dirty tree.
$ sudo git cl dcommit
Done

Powered by Google App Engine
This is Rietveld 408576698