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

Unified Diff: build/android/BUILD.gn

Issue 1784373002: Include isolate.py in data for Android unit tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments Created 4 years, 9 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
« no previous file with comments | « PRESUBMIT_test_mocks.py ('k') | build/android/test_runner.pydeps » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/BUILD.gn
diff --git a/build/android/BUILD.gn b/build/android/BUILD.gn
index 73b25cde9299a7c4789948d992dd4816a1c9b67d..af819ed3ce62c8e4cb3482e4f6b787177e8687e5 100644
--- a/build/android/BUILD.gn
+++ b/build/android/BUILD.gn
@@ -84,30 +84,25 @@ action("cpplib_stripped") {
]
}
-group("devil_py") {
- data = [
- "devil_chromium.json",
- "devil_chromium.py",
- "//third_party/android_tools/sdk/build-tools/23.0.1/aapt",
- "//third_party/android_tools/sdk/build-tools/23.0.1/dexdump",
- "//third_party/android_tools/sdk/build-tools/23.0.1/lib/libc++.so",
- "//third_party/android_tools/sdk/build-tools/23.0.1/split-select",
- "//third_party/android_tools/sdk/platform-tools/adb",
- "//third_party/catapult/catapult_base/catapult_base/",
- "//third_party/catapult/dependency_manager/dependency_manager/",
- "//third_party/catapult/third_party/gsutil/",
- "//third_party/catapult/devil/devil/",
- ]
-}
-
group("test_runner_py") {
- data = [
- "test_runner.py",
- "pylib/",
- "//build/util/lib/common/",
- ]
+ _py_files = read_file("test_runner.pydeps", "list lines")
+
+ # Filter out comments.
+ set_sources_assignment_filter([ "#*" ])
+ sources = _py_files
+
+ data = sources + [
+ "devil_chromium.json",
+ "//third_party/android_tools/sdk/build-tools/23.0.1/aapt",
+ "//third_party/android_tools/sdk/build-tools/23.0.1/dexdump",
+ "//third_party/android_tools/sdk/build-tools/23.0.1/lib/libc++.so",
+ "//third_party/android_tools/sdk/build-tools/23.0.1/split-select",
+ "//third_party/android_tools/sdk/platform-tools/adb",
+ "//third_party/catapult/third_party/gsutil/",
+ "//third_party/catapult/devil/devil/devil_dependencies.json",
+ ]
data_deps = [
- ":devil_py",
+ "//tools/swarming_client:isolate_py",
]
}
« no previous file with comments | « PRESUBMIT_test_mocks.py ('k') | build/android/test_runner.pydeps » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698