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

Unified Diff: build/android/BUILD.gn

Issue 1829683002: Revert of 🌈 Android: Run lint using a cache in the output directory (fix-up) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 | « no previous file | build/android/android_lint_cache.gyp » ('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..01deb142f8ff5ddfb62bb681597e5f523d63f70c 100644
--- a/build/android/BUILD.gn
+++ b/build/android/BUILD.gn
@@ -10,9 +10,37 @@
# Create or update the API versions cache if necessary by running a
# functionally empty lint task. This prevents racy creation of the
# cache while linting java targets in android_lint.
-android_lint("prepare_android_lint_cache") {
- android_manifest = "//build/android/AndroidManifest.xml"
- create_cache = true
+action("prepare_android_lint_cache") {
+ _cache_dir = "${root_out_dir}/android_lint_cache"
+ depfile = "${_cache_dir}/prepare_android_lint_cache.d"
+ _manifest_file = "//build/android/AndroidManifest.xml"
+ _result_file = "${_cache_dir}/result.xml"
+ script = "//build/android/gyp/lint.py"
+
+ inputs = [
+ "${android_sdk_root}/platform-tools/api/api-versions.xml",
+ ]
+ outputs = [
+ depfile,
+ ]
+ args = [
+ "--build-tools-version",
+ android_sdk_build_tools_version,
+ "--cache-dir",
+ rebase_path(_cache_dir, root_build_dir),
+ "--depfile",
+ rebase_path(depfile, root_build_dir),
+ "--lint-path",
+ "$rebased_android_sdk_root/tools/lint",
+ "--manifest-path",
+ rebase_path(_manifest_file, root_build_dir),
+ "--product-dir",
+ ".",
+ "--result-path",
+ rebase_path(_result_file, root_build_dir),
+ "--silent",
+ "--enable",
+ ]
}
action("find_sun_tools_jar") {
« no previous file with comments | « no previous file | build/android/android_lint_cache.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698