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

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

Issue 1372133003: Revert of chromium_tests: raise infra failure when no compile targets are specified (patchset #1 id: (Closed) Base URL: http://src.chromium.org/chrome/trunk/tools/build/
Patch Set: Created 5 years, 3 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/chromium.expected/full_chromium_linux_Android_GN__dbg_.json
===================================================================
--- scripts/slave/recipes/chromium.expected/full_chromium_linux_Android_GN__dbg_.json (revision 296915)
+++ scripts/slave/recipes/chromium.expected/full_chromium_linux_Android_GN__dbg_.json (working copy)
@@ -182,19 +182,23 @@
"cmd": [
"python",
"-u",
- "import sys; sys.exit(1)"
+ "[BUILD]/scripts/slave/compile.py",
+ "--target",
+ "Debug",
+ "--src-dir",
+ "[SLAVE_BUILD]/src",
+ "--build-tool",
+ "ninja",
+ "--compiler",
+ "goma",
+ "--goma-dir",
+ "[BUILD]/goma",
+ "--"
],
"cwd": "[SLAVE_BUILD]",
- "name": "compile",
- "~followup_annotations": [
- "step returned non-zero exit code: 1",
- "@@@STEP_TEXT@Misconfigured bot: no compile targets@@@",
- "@@@STEP_FAILURE@@@"
- ]
- },
- {
- "name": "$final_result",
- "reason": "Step('compile') failed with return_code 1",
- "status_code": 1
+ "env": {
+ "PATH": "[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
+ },
+ "name": "compile"
}
]

Powered by Google App Engine
This is Rietveld 408576698