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

Unified Diff: scripts/slave/recipes/chromium_trybot.expected/clobber_analyze.json

Issue 1212413002: Pass all compile targets to "analyze" (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 5 years, 6 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_trybot.expected/clobber_analyze.json
diff --git a/scripts/slave/recipes/chromium_trybot.expected/clobber_analyze.json b/scripts/slave/recipes/chromium_trybot.expected/clobber_analyze.json
index 7b44683a4aa1939318c8b0fd6446e40ded9d9882..f3add744f068d4653d2190ee530caf104fa628df 100644
--- a/scripts/slave/recipes/chromium_trybot.expected/clobber_analyze.json
+++ b/scripts/slave/recipes/chromium_trybot.expected/clobber_analyze.json
@@ -165,7 +165,7 @@
"-u",
"[SLAVE_BUILD]/src/build/gyp_chromium",
"--analyzer",
- "{\"files\": [\"foo.cc\"], \"targets\": []}",
+ "{\"files\": [\"foo.cc\"], \"targets\": [\"all\"]}",
"/path/to/tmp/json"
],
"cwd": "[SLAVE_BUILD]",
@@ -191,7 +191,9 @@
"@@@STEP_LOG_LINE@analyze_details@ \"original_compile_targets\": [@@@",
"@@@STEP_LOG_LINE@analyze_details@ \"all\"@@@",
"@@@STEP_LOG_LINE@analyze_details@ ], @@@",
- "@@@STEP_LOG_LINE@analyze_details@ \"original_exes\": [], @@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"original_exes\": [@@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"all\"@@@",
+ "@@@STEP_LOG_LINE@analyze_details@ ], @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"self.m.filter.compile_targets\": [@@@",
"@@@STEP_LOG_LINE@analyze_details@ \"base_unittests\", @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"net_unittests\"@@@",

Powered by Google App Engine
This is Rietveld 408576698