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

Unified Diff: scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___builder.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_client_v8_V8_Linux___builder.json
diff --git a/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___builder.json b/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___builder.json
index 89acea2d5724be076120bd087991ee9718b0e264..98c00c6bde5b5a7b6a6c29d9a3557bf3db842a63 100644
--- a/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___builder.json
+++ b/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___builder.json
@@ -162,13 +162,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=\"x86\" use_goma=true goma_dir=\"[CACHE]/cipd/goma\""
+ "-m",
+ "client.v8",
+ "-b",
+ "V8 Linux - builder",
+ "--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