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

Unified Diff: scripts/slave/recipes/chromium_gn.expected/full_tryserver_chromium_linux_android_chromium_gn_compile_rel.json

Issue 1153933004: Flip GN bots to be purely driven by //testing/buildbot/*.json files for tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: add comment explaining the lack of android tests Created 5 years, 7 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/full_tryserver_chromium_linux_android_chromium_gn_compile_rel.json
diff --git a/scripts/slave/recipes/chromium_gn.expected/full_tryserver_chromium_linux_android_chromium_gn_compile_rel.json b/scripts/slave/recipes/chromium_gn.expected/full_tryserver_chromium_linux_android_chromium_gn_compile_rel.json
index 7944cb40a7f92a24bb6ea226a3ebca89a815318a..d47c3ef94d173c10790e1852da351c568a059aaa 100644
--- a/scripts/slave/recipes/chromium_gn.expected/full_tryserver_chromium_linux_android_chromium_gn_compile_rel.json
+++ b/scripts/slave/recipes/chromium_gn.expected/full_tryserver_chromium_linux_android_chromium_gn_compile_rel.json
@@ -115,9 +115,7 @@
"@@@STEP_TEXT@path: [SLAVE_BUILD]/src/testing/buildbot/tryserver.chromium.linux.json@@@",
"@@@STEP_LOG_LINE@json.output@{@@@",
"@@@STEP_LOG_LINE@json.output@ \"android_chromium_gn_compile_rel\": {@@@",
- "@@@STEP_LOG_LINE@json.output@ \"gtest_tests\": [@@@",
- "@@@STEP_LOG_LINE@json.output@ \"base_unittests\"@@@",
- "@@@STEP_LOG_LINE@json.output@ ]@@@",
+ "@@@STEP_LOG_LINE@json.output@ \"gtest_tests\": []@@@",
"@@@STEP_LOG_LINE@json.output@ }@@@",
"@@@STEP_LOG_LINE@json.output@}@@@",
"@@@STEP_LOG_END@json.output@@@"
@@ -198,63 +196,28 @@
"android_chromium_gn_compile_rel",
"-v",
"//out/Release",
- "{\"files\": [\"foo.cc\"], \"targets\": [\"base_unittests_apk\", \"chrome_shell_apk\"]}",
+ "{\"files\": [\"foo.cc\"], \"targets\": [\"chrome_shell_apk\"]}",
"/path/to/tmp/json"
],
"cwd": "[SLAVE_BUILD]",
"name": "analyze",
"~followup_annotations": [
"@@@STEP_LOG_LINE@json.output@{@@@",
- "@@@STEP_LOG_LINE@json.output@ \"build_targets\": [@@@",
- "@@@STEP_LOG_LINE@json.output@ \"base_unittests\"@@@",
- "@@@STEP_LOG_LINE@json.output@ ], @@@",
+ "@@@STEP_LOG_LINE@json.output@ \"build_targets\": [], @@@",
"@@@STEP_LOG_LINE@json.output@ \"status\": \"Found dependency\", @@@",
- "@@@STEP_LOG_LINE@json.output@ \"targets\": [@@@",
- "@@@STEP_LOG_LINE@json.output@ \"base_unittests\"@@@",
- "@@@STEP_LOG_LINE@json.output@ ]@@@",
+ "@@@STEP_LOG_LINE@json.output@ \"targets\": []@@@",
"@@@STEP_LOG_LINE@json.output@}@@@",
"@@@STEP_LOG_END@json.output@@@",
"@@@STEP_LOG_LINE@analyze_details@{@@@",
- "@@@STEP_LOG_LINE@analyze_details@ \"compile_targets\": [@@@",
- "@@@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_apk\"@@@",
- "@@@STEP_LOG_LINE@analyze_details@ ], @@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"compile_targets\": [], @@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"original_compile_targets\": [], @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"original_exes\": [@@@",
- "@@@STEP_LOG_LINE@analyze_details@ \"base_unittests_apk\", @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"chrome_shell_apk\"@@@",
"@@@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@ ], @@@",
- "@@@STEP_LOG_LINE@analyze_details@ \"self.m.filter.matching_exes\": [@@@",
- "@@@STEP_LOG_LINE@analyze_details@ \"base_unittests\"@@@",
- "@@@STEP_LOG_LINE@analyze_details@ ]@@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"self.m.filter.compile_targets\": [], @@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"self.m.filter.matching_exes\": []@@@",
"@@@STEP_LOG_LINE@analyze_details@}@@@",
"@@@STEP_LOG_END@analyze_details@@@"
]
- },
- {
- "cmd": [
- "python",
- "-u",
- "[BUILD]/scripts/slave/compile.py",
- "--target",
- "Release",
- "--src-dir",
- "[SLAVE_BUILD]/src",
- "--build-tool",
- "ninja",
- "--compiler",
- "goma",
- "--goma-dir",
- "[BUILD]/goma",
- "--goma-disable-local-fallback",
- "--",
- "base_unittests"
- ],
- "cwd": "[SLAVE_BUILD]",
- "name": "compile"
}
]

Powered by Google App Engine
This is Rietveld 408576698