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

Unified Diff: build/android/buildbot/bb_run_bot.py

Issue 142393002: android: Use use_goma=1 gomadir gyp defines instead of magic GOMA_DIR env var. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nowat Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | build/android/envsetup.sh » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/buildbot/bb_run_bot.py
diff --git a/build/android/buildbot/bb_run_bot.py b/build/android/buildbot/bb_run_bot.py
index 158bea9579a93ad4d71410d5419fb9bff3b8853f..50e70ee758598f2ee2100f573a1bbf3fd1b7de3a 100755
--- a/build/android/buildbot/bb_run_bot.py
+++ b/build/android/buildbot/bb_run_bot.py
@@ -48,7 +48,6 @@ def DictDiff(d1, d2):
def GetEnvironment(host_obj, testing, extra_env_vars=None):
init_env = dict(os.environ)
init_env['GYP_GENERATORS'] = 'ninja'
- init_env['GOMA_DIR'] = bb_utils.GOMA_DIR
if extra_env_vars:
init_env.update(extra_env_vars)
envsetup_cmd = '. build/android/envsetup.sh'
@@ -70,7 +69,8 @@ def GetEnvironment(host_obj, testing, extra_env_vars=None):
print >> sys.stderr, envsetup_output
sys.exit(1)
env = json.loads(json_env)
- env['GYP_DEFINES'] = env.get('GYP_DEFINES', '') + ' fastbuild=1'
+ env['GYP_DEFINES'] = env.get('GYP_DEFINES', '') + \
+ ' fastbuild=1 use_goma=1 gomadir=%s' % bb_utils.GOMA_DIR
extra_gyp = host_obj.extra_gyp_defines
if extra_gyp:
env['GYP_DEFINES'] += ' %s' % extra_gyp
« no previous file with comments | « no previous file | build/android/envsetup.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698