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

Unified Diff: scripts/slave/recipes/chromium.expected/full_chromium_fyi_Chromium_Win_10.json

Issue 1472893002: chromium_tests: require explicit compile targets (except for Android) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 5 years, 1 month 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/chromium.expected/full_chromium_fyi_Chromium_Win_10.json
diff --git a/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Chromium_Win_10.json b/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Chromium_Win_10.json
index 66d20f9cd6f057db21b5bd2db5b5c81abd964cb3..aae0308c8d330b40b638b7ed53b5853f19d20a7f 100644
--- a/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Chromium_Win_10.json
+++ b/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Chromium_Win_10.json
@@ -161,33 +161,19 @@
"cmd": [
"python",
"-u",
- "[BUILD]\\scripts\\slave\\compile.py",
- "--target",
- "Release_x64",
- "--src-dir",
- "[SLAVE_BUILD]\\src",
- "--build-tool",
- "ninja",
- "--compiler",
- "goma",
- "--goma-jsonstatus",
- "/path/to/tmp/json",
- "--goma-dir",
- "[BUILD]\\goma",
- "--",
- "All",
- "chromium_builder_tests"
+ "import sys; sys.exit(1)"
],
"cwd": "[SLAVE_BUILD]",
"name": "compile",
"~followup_annotations": [
- "@@@STEP_LOG_LINE@json.output (invalid)@null@@@",
- "@@@STEP_LOG_END@json.output (invalid)@@@"
+ "step returned non-zero exit code: 1",
+ "@@@STEP_TEXT@Please add explicit compile targets for the build@@@",
+ "@@@STEP_FAILURE@@@"
]
},
{
"name": "$result",
- "recipe_result": null,
- "status_code": 0
+ "reason": "Step('compile') failed with return_code 1",
+ "status_code": 1
}
]

Powered by Google App Engine
This is Rietveld 408576698