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

Unified Diff: scripts/slave/recipe_modules/filter/example.expected/bad_retcode_doesnt_fail.json

Issue 485873004: Adds ability for builders to only compile targets affected by change (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: merge 2 trunk Created 6 years, 4 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/recipe_modules/filter/example.expected/bad_retcode_doesnt_fail.json
diff --git a/scripts/slave/recipe_modules/filter/example.expected/bad_retcode_doesnt_fail.json b/scripts/slave/recipe_modules/filter/example.expected/bad_retcode_doesnt_fail.json
index abd4851a4b440e6266a7182962e2b17ab4299703..35ba3d4c76793d80decd9213d14f080d0710e34d 100644
--- a/scripts/slave/recipe_modules/filter/example.expected/bad_retcode_doesnt_fail.json
+++ b/scripts/slave/recipe_modules/filter/example.expected/bad_retcode_doesnt_fail.json
@@ -16,7 +16,7 @@
"-u",
"[SLAVE_BUILD]/build/gyp_chromium",
"--analyzer",
- "{\"files\": [\"foo.cc\"], \"targets\": null}",
+ "{\"files\": [\"foo.cc\"], \"targets\": []}",
"/path/to/tmp/json"
],
"env": {
@@ -26,6 +26,7 @@
"~followup_annotations": [
"step returned non-zero exit code: -1",
"@@@STEP_LOG_LINE@json.output@{@@@",
+ "@@@STEP_LOG_LINE@json.output@ \"build_targets\": [], @@@",
"@@@STEP_LOG_LINE@json.output@ \"status\": \"No dependency\", @@@",
"@@@STEP_LOG_LINE@json.output@ \"targets\": []@@@",
"@@@STEP_LOG_LINE@json.output@}@@@",

Powered by Google App Engine
This is Rietveld 408576698