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

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

Issue 1419643002: Ignore compile_targets returned by analyze and rely completely on matching_exes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 5 years, 2 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/compile_because_of_analyze_with_filtered_compile_targets.json
diff --git a/scripts/slave/recipes/chromium_trybot.expected/compile_because_of_analyze_with_filtered_compile_targets.json b/scripts/slave/recipes/chromium_trybot.expected/compile_because_of_analyze_with_filtered_compile_targets.json
index 91cf01e3c6df4eee338843f5adc3655498f985f6..136bf75c06cb0eae6e017ac674b40b778add347b 100644
--- a/scripts/slave/recipes/chromium_trybot.expected/compile_because_of_analyze_with_filtered_compile_targets.json
+++ b/scripts/slave/recipes/chromium_trybot.expected/compile_because_of_analyze_with_filtered_compile_targets.json
@@ -263,6 +263,10 @@
"@@@STEP_LOG_LINE@analyze_details@ \"browser_tests\", @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"base_unittests\"@@@",
"@@@STEP_LOG_LINE@analyze_details@ ], @@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"matching_exes\": [@@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"browser_tests\", @@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"base_unittests\"@@@",
+ "@@@STEP_LOG_LINE@analyze_details@ ], @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"original_compile_targets\": [@@@",
"@@@STEP_LOG_LINE@analyze_details@ \"angle_unittests_run\", @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"chrome\", @@@",
@@ -346,8 +350,7 @@
"--ninja-ensure-up-to-date",
"--",
"base_unittests",
- "browser_tests",
- "chrome"
+ "browser_tests"
],
"cwd": "[SLAVE_BUILD]",
"name": "compile (with patch)",

Powered by Google App Engine
This is Rietveld 408576698