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

Unified Diff: scripts/slave/recipes/chromium_gn.expected/full_chromium_linux_Android_GN.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_chromium_linux_Android_GN.json
diff --git a/scripts/slave/recipes/chromium_gn.expected/full_chromium_linux_Android_GN.json b/scripts/slave/recipes/chromium_gn.expected/full_chromium_linux_Android_GN.json
index 75c5daa1c85fd390a3b63f2fc156504cff214fb3..2285c2a55d0196b3f8146776e5687c23b69ea321 100644
--- a/scripts/slave/recipes/chromium_gn.expected/full_chromium_linux_Android_GN.json
+++ b/scripts/slave/recipes/chromium_gn.expected/full_chromium_linux_Android_GN.json
@@ -109,9 +109,7 @@
"@@@STEP_TEXT@path: [SLAVE_BUILD]/src/testing/buildbot/chromium.linux.json@@@",
"@@@STEP_LOG_LINE@json.output@{@@@",
"@@@STEP_LOG_LINE@json.output@ \"Android GN\": {@@@",
- "@@@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@@@"
@@ -155,22 +153,9 @@
"--goma-dir",
"[BUILD]/goma",
"--",
- "base_unittests_apk",
"chrome_shell_apk"
],
"cwd": "[SLAVE_BUILD]",
"name": "compile"
- },
- {
- "cmd": [
- "python",
- "-u",
- "[DEPOT_TOOLS]/gn.py",
- "--root=[SLAVE_BUILD]/src",
- "check",
- "//out/Release"
- ],
- "cwd": "[SLAVE_BUILD]",
- "name": "gn check"
}
]

Powered by Google App Engine
This is Rietveld 408576698