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

Unified Diff: scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng.json

Issue 2070003002: V8: Let peek-gn use mb to reuse v8-side configs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Review Created 4 years, 6 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
Index: scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng.json
diff --git a/scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng.json b/scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng.json
index c58b3cd9dbd859eaefe655a606fec2f6861b8d36..27e17ae1d613ec1cfa8940cd23316723e5f17b93 100644
--- a/scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng.json
+++ b/scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng.json
@@ -168,13 +168,32 @@
"cmd": [
"python",
"-u",
- "[DEPOT_TOOLS]/gn.py",
- "--root=[SLAVE_BUILD]/v8",
+ "RECIPE_MODULE[build::v8]/resources/patch_mb_config.py",
+ "[SLAVE_BUILD]/v8/infra/mb/mb_config.pyl",
+ "/path/to/tmp/"
+ ],
+ "name": "patch mb config (fyi)"
+ },
+ {
+ "cmd": [
+ "python",
+ "-u",
+ "[SLAVE_BUILD]/v8/tools/mb/mb.py",
"gen",
- "[TMP_BASE]/v8_gn_tmp_1",
- "--args=is_debug=false target_cpu=\"x64\" use_goma=true goma_dir=\"[CACHE]/cipd/goma\""
+ "-m",
+ "tryserver.v8",
+ "-b",
+ "v8_linux64_rel_ng",
+ "--config-file",
+ "[mb config]",
+ "--goma-dir",
+ "[CACHE]/cipd/goma",
+ "[TMP_BASE]/v8_gn_tmp_1"
],
- "name": "gn"
+ "env": {
+ "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account-goma-client.json"
+ },
+ "name": "generate_build_files with gn (fyi)"
},
{
"cmd": [

Powered by Google App Engine
This is Rietveld 408576698