| #!/usr/bin/env python |
| # |
| # Copyright (c) 2013 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. |
| |
| """Runs Android's lint tool.""" |
| |
| |
| import optparse |
| import os |
| import sys |
| from xml.dom import minidom |
| |
| from util import build_utils |
| |
| |
| _SRC_ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__), |
| '..', '..', '..')) |
| |
| |
| def _RunLint(lint_path, config_path, processed_config_path, manifest_path, |
| result_path, product_dir, src_dirs, classes_dir): |
| |
| def _RelativizePath(path): |
| """Returns relative path to top-level src dir. |
| |
| Args: |
| path: A path relative to cwd. |
| """ |
| return os.path.relpath(os.path.abspath(path), _SRC_ROOT) |
| |
| def _ProcessConfigFile(): |
| if not build_utils.IsTimeStale(processed_config_path, [config_path]): |
| return |
| |
| with open(config_path, 'rb') as f: |
| content = f.read().replace( |
| 'PRODUCT_DIR', _RelativizePath(product_dir)) |
| |
| with open(processed_config_path, 'wb') as f: |
| f.write(content) |
| |
| def _ProcessResultFile(): |
| with open(result_path, 'rb') as f: |
| content = f.read().replace( |
| _RelativizePath(product_dir), 'PRODUCT_DIR') |
| |
| with open(result_path, 'wb') as f: |
| f.write(content) |
| |
| def _ParseAndShowResultFile(): |
| dom = minidom.parse(result_path) |
| issues = dom.getElementsByTagName('issue') |
| print >> sys.stderr |
| for issue in issues: |
| issue_id = issue.attributes['id'].value |
| severity = issue.attributes['severity'].value |
| message = issue.attributes['message'].value |
| location_elem = issue.getElementsByTagName('location')[0] |
| path = location_elem.attributes['file'].value |
| line = location_elem.getAttribute('line') |
| if line: |
| error = '%s:%s %s: %s [%s]' % (path, line, severity, message, |
| issue_id) |
| else: |
| # Issues in class files don't have a line number. |
| error = '%s %s: %s [%s]' % (path, severity, message, issue_id) |
| print >> sys.stderr, error |
| for attr in ['errorLine1', 'errorLine2']: |
| error_line = issue.getAttribute(attr) |
| if error_line: |
| print >> sys.stderr, error_line |
| return len(issues) |
| |
| _ProcessConfigFile() |
| |
| cmd = [ |
| lint_path, '-Werror', '--exitcode', '--showall', |
| '--config', _RelativizePath(processed_config_path), |
| '--classpath', _RelativizePath(classes_dir), |
| '--xml', _RelativizePath(result_path), |
| ] |
| for src in src_dirs: |
| cmd.extend(['--sources', _RelativizePath(src)]) |
| cmd.append(_RelativizePath(os.path.join(manifest_path, os.pardir))) |
| |
| if os.path.exists(result_path): |
| os.remove(result_path) |
| |
| try: |
| build_utils.CheckOutput(cmd, cwd=_SRC_ROOT) |
| except build_utils.CalledProcessError: |
| # There is a problem with lint usage |
| if not os.path.exists(result_path): |
| raise |
| # There are actual lint issues |
| else: |
| num_issues = _ParseAndShowResultFile() |
| _ProcessResultFile() |
| msg = ('\nLint found %d new issues.\n' |
| ' - For full explanation refer to %s\n' |
| ' - Wanna suppress these issues?\n' |
| ' 1. Read comment in %s\n' |
| ' 2. Run "python %s %s"\n' % |
| (num_issues, |
| _RelativizePath(result_path), |
| _RelativizePath(config_path), |
| _RelativizePath(os.path.join(_SRC_ROOT, 'build', 'android', |
| 'lint', 'suppress.py')), |
| _RelativizePath(result_path))) |
| print >> sys.stderr, msg |
| # Lint errors do not fail the build. |
| return 0 |
| |
| return 0 |
| |
| |
| def main(): |
| parser = optparse.OptionParser() |
| parser.add_option('--lint-path', help='Path to lint executable.') |
| parser.add_option('--config-path', help='Path to lint suppressions file.') |
| parser.add_option('--processed-config-path', |
| help='Path to processed lint suppressions file.') |
| parser.add_option('--manifest-path', help='Path to AndroidManifest.xml') |
| parser.add_option('--result-path', help='Path to XML lint result file.') |
| parser.add_option('--product-dir', help='Path to product dir.') |
| parser.add_option('--src-dirs', help='Directories containing java files.') |
| parser.add_option('--classes-dir', help='Directory containing class files.') |
| parser.add_option('--stamp', help='Path to touch on success.') |
| parser.add_option('--enable', action='store_true', |
| help='Run lint instead of just touching stamp.') |
| |
| options, _ = parser.parse_args() |
| |
| build_utils.CheckOptions( |
| options, parser, required=['lint_path', 'config_path', |
| 'processed_config_path', 'manifest_path', |
| 'result_path', 'product_dir', 'src_dirs', |
| 'classes_dir']) |
| |
| src_dirs = build_utils.ParseGypList(options.src_dirs) |
| |
| rc = 0 |
| |
| if options.enable: |
| rc = _RunLint(options.lint_path, options.config_path, |
| options.processed_config_path, |
| options.manifest_path, options.result_path, |
| options.product_dir, src_dirs, options.classes_dir) |
| |
| if options.stamp and not rc: |
| build_utils.Touch(options.stamp) |
| |
| return rc |
| |
| |
| if __name__ == '__main__': |
| sys.exit(main()) |