| # Copyright 2021 The Bazel Authors. All rights reserved. | |
| # | |
| # Licensed under the Apache License, Version 2.0 (the "License"); | |
| # you may not use this file except in compliance with the License. | |
| # You may obtain a copy of the License at | |
| # | |
| # http://www.apache.org/licenses/LICENSE-2.0 | |
| # | |
| # Unless required by applicable law or agreed to in writing, software | |
| # distributed under the License is distributed on an "AS IS" BASIS, | |
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | |
| # See the License for the specific language governing permissions and | |
| # limitations under the License. | |
| """Creates the android lint action for java rules""" | |
| load("//java/common:java_semantics.bzl", "semantics") | |
| # copybara: default visibility | |
| def _tokenize_opts(opts_depset): | |
| opts = reversed(opts_depset.to_list()) | |
| return semantics.tokenize_javacopts(opts) | |
| def _flags_from_disable_lint_checks(args, disable_lint_checks): | |
| invalid_ids = [check for check in disable_lint_checks if not _check_id_is_valid(check)] | |
| if invalid_ids: | |
| fail("Found not allowed character in disabled check ID(s) %s" % str(invalid_ids)) | |
| args.add_joined("--disable", disable_lint_checks, join_with = ",") | |
| # Returns whether the passed Android Lint check ID is valid | |
| def _check_id_is_valid(check): | |
| # We want to prevent whitespace so that extra flags can't be 'injected' into Android Lint | |
| return check.replace(".", "").replace("_", "").isalnum() | |
| def _android_lint_action(ctx, source_files, source_jars, compilation_info, is_library, disable_lint_checks = []): | |
| """ | |
| Creates an action that runs Android lint against Java source files. | |
| You need to add `ANDROID_LINT_IMPLICIT_ATTRS` to any rule or aspect using this call. | |
| To lint generated source jars (java_info.java_outputs.gen_source_jar) | |
| add them to the `source_jar` parameter. | |
| `compilation_info` parameter should supply the classpath and Javac options | |
| that were used during Java compilation. | |
| The Android lint tool is obtained from Java toolchain. | |
| Args: | |
| ctx: (RuleContext) Used to register the action. | |
| source_files: (list[File]) A list of .java source files | |
| source_jars: (list[File]) A list of .jar or .srcjar files containing | |
| source files. It should also include generated source jars. | |
| compilation_info: (struct) Information about compilation. | |
| is_library: (bool) Whether the target is a library. | |
| disable_lint_checks: (list[str]) A list of AndroidLint checks to be skipped. | |
| Returns: | |
| (None|File) The Android lint output file or None if no source files were | |
| present. | |
| """ | |
| # assuming that linting is enabled for all java rules i.e. | |
| # --experimental_limit_android_lint_to_android_constrained_java=false | |
| # --experimental_run_android_lint_on_java_rules= is checked in basic_java_library.bzl | |
| if not (source_files or source_jars): | |
| return None | |
| toolchain = semantics.find_java_toolchain(ctx) | |
| java_runtime = toolchain.java_runtime | |
| linter = toolchain._android_linter | |
| if not linter: | |
| # TODO(hvd): enable after enabling in tests | |
| # fail("android linter not set in java_toolchain") | |
| return None | |
| args = ctx.actions.args() | |
| executable = linter.tool.executable | |
| transitive_inputs = [] | |
| if executable.extension != "jar": | |
| tools = [linter.tool] | |
| transitive_inputs.append(linter.data) | |
| args_list = [args] | |
| else: | |
| jvm_args = ctx.actions.args() | |
| jvm_args.add_all(toolchain.jvm_opt) | |
| jvm_args.add_all(linter.jvm_opts) | |
| jvm_args.add("-jar", executable) | |
| executable = java_runtime.java_executable_exec_path | |
| tools = [java_runtime.files, linter.tool.executable] | |
| transitive_inputs.append(linter.data) | |
| args_list = [jvm_args, args] | |
| classpath = compilation_info.compilation_classpath | |
| # TODO(hvd): get from toolchain if we need this - probably android only | |
| bootclasspath_aux = [] | |
| if bootclasspath_aux: | |
| classpath = depset(transitive = [classpath, bootclasspath_aux]) | |
| transitive_inputs.append(classpath) | |
| bootclasspath = toolchain.bootclasspath | |
| transitive_inputs.append(bootclasspath) | |
| transitive_inputs.append(compilation_info.plugins.processor_jars) | |
| transitive_inputs.append(compilation_info.plugins.processor_data) | |
| args.add_all("--sources", source_files) | |
| args.add_all("--source_jars", source_jars) | |
| args.add_all("--bootclasspath", bootclasspath) | |
| args.add_all("--classpath", classpath) | |
| args.add_all("--lint_rules", compilation_info.plugins.processor_jars) | |
| args.add("--target_label", ctx.label) | |
| args.add("--library", str(is_library).lower()) | |
| javac_opts = compilation_info.javac_options | |
| if javac_opts: | |
| # wrap in a list so that map_each passes the depset to _tokenize_opts | |
| args.add_all("--javacopts", [javac_opts], map_each = _tokenize_opts) | |
| args.add("--") | |
| args.add("--lintopts") | |
| args.add_all(linter.lint_opts) | |
| _flags_from_disable_lint_checks(args, disable_lint_checks) | |
| for package_config in linter.package_config: | |
| if package_config.matches(package_config.package_specs, ctx.label): | |
| # wrap in a list so that map_each passes the depset to _tokenize_opts | |
| package_opts = [package_config.javac_opts] | |
| args.add_all(package_opts, map_each = _tokenize_opts) | |
| transitive_inputs.append(package_config.data) | |
| android_lint_out = ctx.actions.declare_file("%s_android_lint_output.xml" % ctx.label.name) | |
| args.add("--xml", android_lint_out) | |
| args.set_param_file_format(format = "multiline") | |
| args.use_param_file(param_file_arg = "@%s", use_always = True) | |
| ctx.actions.run( | |
| mnemonic = "AndroidLint", | |
| progress_message = semantics.LINT_PROGRESS_MESSAGE, | |
| executable = executable, | |
| inputs = depset( | |
| # TODO(b/213551463) benchmark using a transitive depset instead | |
| source_files + source_jars, | |
| transitive = transitive_inputs, | |
| ), | |
| outputs = [android_lint_out], | |
| tools = tools, | |
| arguments = args_list, | |
| execution_requirements = {"supports-workers": "1"}, | |
| use_default_shell_env = True, | |
| ) | |
| return android_lint_out | |
| android_lint_subrule = subrule( | |
| implementation = _android_lint_action, | |
| toolchains = [semantics.JAVA_TOOLCHAIN_TYPE], | |
| ) |