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

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

Issue 1376453002: chromium_tests: raise infra failure when no compile targets are specified (Closed) Base URL: svn://svn.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_fyi_CFI_Linux.json
diff --git a/scripts/slave/recipes/chromium.expected/full_chromium_fyi_CFI_Linux.json b/scripts/slave/recipes/chromium.expected/full_chromium_fyi_CFI_Linux.json
index 4848cfe3513f6fe8073d1e2e2b8a6314dd64f6cc..24bb185efeafd9fd2416be6d9738343b8a6deb07 100644
--- a/scripts/slave/recipes/chromium.expected/full_chromium_fyi_CFI_Linux.json
+++ b/scripts/slave/recipes/chromium.expected/full_chromium_fyi_CFI_Linux.json
@@ -128,43 +128,8 @@
"name": "cleanup_temp"
},
{
- "allow_subannotations": true,
- "cmd": [
- "python",
- "-u",
- "[BUILD]/scripts/slave/clang_revision.py",
- "--src-dir",
- "[SLAVE_BUILD]/src",
- "--output-json",
- "/path/to/tmp/json"
- ],
- "cwd": "[SLAVE_BUILD]",
- "name": "clang_revision",
- "~followup_annotations": [
- "@@@STEP_LOG_LINE@json.output@{@@@",
- "@@@STEP_LOG_LINE@json.output@ \"clang_revision\": \"123456-7\"@@@",
- "@@@STEP_LOG_LINE@json.output@}@@@",
- "@@@STEP_LOG_END@json.output@@@"
- ]
- },
- {
- "cmd": [
- "python",
- "-u",
- "[BUILD]/scripts/slave/compile.py",
- "--target",
- "Release",
- "--src-dir",
- "[SLAVE_BUILD]/src",
- "--build-tool",
- "ninja",
- "--compiler",
- "clang",
- "--",
- "All",
- "chromium_builder_tests"
- ],
- "cwd": "[SLAVE_BUILD]",
- "name": "compile"
+ "name": "$final_result",
+ "reason": "Misconfigured bot: no compile targets",
+ "status_code": 1
}
]

Powered by Google App Engine
This is Rietveld 408576698