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

Unified Diff: scripts/slave/recipe_modules/chromium/api.py

Issue 2388683002: Re-enable DepsCache using GOMA_DEPS_CACHE_FILE (Closed)
Patch Set: Fix indent Created 4 years, 2 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 | « scripts/slave/compile.py ('k') | scripts/slave/recipe_modules/chromium/example.expected/basic_out_dir.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/chromium/api.py
diff --git a/scripts/slave/recipe_modules/chromium/api.py b/scripts/slave/recipe_modules/chromium/api.py
index d709175717da6963c05988630ac71c7aa7b37e62..093159021af460ddd1cef417adea4988bef8474f 100644
--- a/scripts/slave/recipe_modules/chromium/api.py
+++ b/scripts/slave/recipe_modules/chromium/api.py
@@ -187,8 +187,10 @@ class ChromiumApi(recipe_api.RecipeApi):
# Use explicit goma deps cache dir if it's set in the path config.
ukai 2016/10/04 01:24:43 update comment?
shinyak 2016/10/04 08:19:16 Removed the comment.
# Otherwise use the default one inside build output directory.
try:
- args.extend(['--goma-deps-cache-dir', self.m.path['goma_deps_cache']])
- except KeyError:
+ safe_buildername = re.sub(r'[^a-zA-Z0-9]', '_',
+ self.m.properties['buildername']) + '.gomadeps'
+ args.extend(['--goma-deps-cache-file', safe_buildername])
+ except KeyError: # pragma: nocover
Paweł Hajdan Jr. 2016/10/03 10:04:54 Could you remove the try-except? We could get KeyE
shinyak 2016/10/04 08:19:16 Done.
pass
if self.c.compile_py.build_args:
« no previous file with comments | « scripts/slave/compile.py ('k') | scripts/slave/recipe_modules/chromium/example.expected/basic_out_dir.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698