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

Unified Diff: scripts/slave/recipe_modules/filter/example.expected/analyzes_matches_all_exes.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/analyzes_matches_all_exes.json
diff --git a/scripts/slave/recipe_modules/filter/example.expected/analyzes_matches_all_exes.json b/scripts/slave/recipe_modules/filter/example.expected/analyzes_matches_all_exes.json
index 3d64b53fff9148efb4168fcf1a3b98dc381d736e..8bce37da84d2ca9b59d7d4328f4a1b51d265441c 100644
--- a/scripts/slave/recipe_modules/filter/example.expected/analyzes_matches_all_exes.json
+++ b/scripts/slave/recipe_modules/filter/example.expected/analyzes_matches_all_exes.json
@@ -16,7 +16,7 @@
"-u",
"[SLAVE_BUILD]/build/gyp_chromium",
"--analyzer",
- "{\"files\": [\"foo.cc\"], \"targets\": [\"foo\", \"bar\"]}",
+ "{\"files\": [\"foo.cc\"], \"targets\": []}",
"/path/to/tmp/json"
],
"env": {
@@ -25,10 +25,7 @@
"name": "analyze",
"~followup_annotations": [
"@@@STEP_LOG_LINE@json.output@{@@@",
- "@@@STEP_LOG_LINE@json.output@ \"status\": \"Found dependency (all)\", @@@",
- "@@@STEP_LOG_LINE@json.output@ \"targets\": [@@@",
- "@@@STEP_LOG_LINE@json.output@ \"foo\"@@@",
- "@@@STEP_LOG_LINE@json.output@ ]@@@",
+ "@@@STEP_LOG_LINE@json.output@ \"status\": \"Found dependency (all)\"@@@",
"@@@STEP_LOG_LINE@json.output@}@@@",
"@@@STEP_LOG_END@json.output@@@"
]

Powered by Google App Engine
This is Rietveld 408576698