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

Unified Diff: scripts/slave/recipes/chromium_gn.expected/compile_failure.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_gn.expected/compile_failure.json
diff --git a/scripts/slave/recipes/chromium_gn.expected/compile_failure.json b/scripts/slave/recipes/chromium_gn.expected/compile_failure.json
index 026f3f5fbe1d9caeb1a19dc8878ffe5cf382f8c6..7074848dd58d6d6a7633d76cdc2881c85b915fe1 100644
--- a/scripts/slave/recipes/chromium_gn.expected/compile_failure.json
+++ b/scripts/slave/recipes/chromium_gn.expected/compile_failure.json
@@ -228,6 +228,9 @@
"@@@STEP_LOG_LINE@analyze_details@ \"compile_targets\": [@@@",
"@@@STEP_LOG_LINE@analyze_details@ \"base_unittests\"@@@",
"@@@STEP_LOG_LINE@analyze_details@ ], @@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"matching_exes\": [@@@",
+ "@@@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@ \"base_unittests\", @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"all\"@@@",

Powered by Google App Engine
This is Rietveld 408576698