🌈 Android: Run lint using a cache in the output directory (fix-up)

build/config/android/internal_rules.gni

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/android/config.gni")
 import("//build/config/sanitizers/sanitizers.gni")
 import("//build/config/zip.gni")
 import("//third_party/ijar/ijar.gni")
 
 assert(is_android)
 
 rebased_android_sdk = rebase_path(android_sdk, root_build_dir)
 rebased_android_sdk_root = rebase_path(android_sdk_root, root_build_dir)
 rebased_android_sdk_build_tools =
     rebase_path(android_sdk_build_tools, root_build_dir)
 
 android_sdk_jar = "$android_sdk/android.jar"
 rebased_android_sdk_jar = rebase_path(android_sdk_jar, root_build_dir)
 android_default_aapt_path = "$rebased_android_sdk_build_tools/aapt"
 
 android_configuration_name = "Release"
 if (is_debug) {
   android_configuration_name = "Debug"
 }
 
 template("android_lint") {
-  set_sources_assignment_filter([])
-
-  jar_path = invoker.jar_path
-  android_manifest = invoker.android_manifest
-  java_files = invoker.java_files
-  base_path = "$target_gen_dir/$target_name"
-
   action(target_name) {
     deps = []
     forward_variables_from(invoker,
                            [
                              "deps",
                              "data_deps",
                              "public_deps",
                              "testonly",
                            ])
+    _cache_dir = "$root_build_dir/android_lint_cache"
+    _result_path = "$target_gen_dir/$target_name/result.xml"
+    _config_path = "$target_gen_dir/$target_name/config.xml"
+    _suppressions_file = "//build/android/lint/suppressions.xml"
+    _cache_file = "$_cache_dir/.android/cache/api-versions-6-${android_sdk_build_tools_version}.bin"
+    _create_cache = defined(invoker.create_cache) && invoker.create_cache
+
     script = "//build/android/gyp/lint.py"
-    result_path = base_path + "/result.xml"
-    config_path = base_path + "/config.xml"
-    suppressions_file = "//build/android/lint/suppressions.xml"
+    depfile = "$target_gen_dir/$target_name.d"
     inputs = [
-               suppressions_file,
-               android_manifest,
-               jar_path,
-             ] + java_files
+      "${android_sdk_root}/platform-tools/api/api-versions.xml",
+      _suppressions_file,
+      invoker.android_manifest,
+    ]
 
     outputs = [
-      config_path,
-      result_path,
+      depfile,
+      _config_path,
+      _result_path,
     ]
 
-    deps += [ "//build/android:prepare_android_lint_cache" ]
-
-    rebased_java_files = rebase_path(java_files, root_build_dir)
+    if (_create_cache) {
+      outputs += [ _cache_file ]
+    } else {
+      inputs += [
+                  _cache_file,
+                  invoker.jar_path,
+                ] + invoker.java_files
+      deps += [ "//build/android:prepare_android_lint_cache" ]
+    }
 
     args = [
       "--lint-path=$rebased_android_sdk_root/tools/lint",
+      "--cache-dir",
+      rebase_path(_cache_dir, root_build_dir),
+      "--build-tools-version",
+      android_sdk_build_tools_version,
+      "--depfile",

[jbudorick, 2016/03/22 19:27:22]

I had issues with the depfile here previously causing lint to be run even if no
inputs changed. Does that happen with this addition?

[agrieve, 2016/03/22 19:57:18]

I haven't noticed that it does. I verified the depfile has only .py files in it,
so it shouldn't.

lint.py also uses md5check, so it will do it's own double-checking that it needs
to be run.

[jbudorick, 2016/03/22 20:36:25]

Hm, maybe I had it misconfigured, then.

+      rebase_path(depfile, root_build_dir),
       "--config-path",
-      rebase_path(suppressions_file, root_build_dir),
+      rebase_path(_suppressions_file, root_build_dir),
       "--manifest-path",
-      rebase_path(android_manifest, root_build_dir),
+      rebase_path(invoker.android_manifest, root_build_dir),
       "--product-dir=.",
-      "--jar-path",
-      rebase_path(jar_path, root_build_dir),
       "--processed-config-path",
-      rebase_path(config_path, root_build_dir),
+      rebase_path(_config_path, root_build_dir),
       "--result-path",
-      rebase_path(result_path, root_build_dir),
-      "--java-files=$rebased_java_files",
+      rebase_path(_result_path, root_build_dir),
       "--enable",
     ]
+    if (_create_cache) {

[jbudorick, 2016/03/22 19:27:22]

nit: What about merging the two if (_create_cache) blocks?

[agrieve, 2016/03/22 19:57:18]

Done.

+      args += [
+        "--create-cache",
+        "--silent",
+      ]
+    } else {
+      _rebased_java_files = rebase_path(invoker.java_files, root_build_dir)
+      args += [
+        "--jar-path",
+        rebase_path(invoker.jar_path, root_build_dir),
+        "--java-files=$_rebased_java_files",
+      ]
+    }
   }
 }
 
 template("proguard") {
   action(target_name) {
     set_sources_assignment_filter([])
     forward_variables_from(invoker,
                            [
                              "deps",
                              "data_deps",
@@ skipping 2082 matching lines @@
     ]
     args = [
       "--depfile",
       rebase_path(depfile, root_build_dir),
       "--script-output-path",
       rebase_path(generated_script, root_build_dir),
     ]
     args += test_runner_args
   }
 }