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

Side by Side Diff: build/android/lint_action.gypi

Issue 458653002: Update lint and findbugs to use jars instead of class files (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add a landmine Created 6 years, 4 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 unified diff | Download patch
« no previous file with comments | « build/android/gyp/lint.py ('k') | build/android/push_libraries.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # This file is meant to be included into an action to provide a rule to 5 # This file is meant to be included into an action to provide a rule to
6 # run lint on java/class files. 6 # run lint on java/class files.
7 7
8 { 8 {
9 'action_name': 'lint_<(_target_name)', 9 'action_name': 'lint_<(_target_name)',
10 'message': 'Linting <(_target_name)', 10 'message': 'Linting <(_target_name)',
11 'variables': { 11 'variables': {
12 'conditions': [ 12 'conditions': [
13 ['chromium_code != 0 and android_lint != 0 and never_lint == 0', { 13 ['chromium_code != 0 and android_lint != 0 and never_lint == 0', {
14 'is_enabled': '--enable', 14 'is_enabled': '--enable',
15 }, { 15 }, {
16 'is_enabled': '', 16 'is_enabled': '',
17 }] 17 }]
18 ] 18 ]
19 }, 19 },
20 'inputs': [ 20 'inputs': [
21 '<(DEPTH)/build/android/gyp/util/build_utils.py', 21 '<(DEPTH)/build/android/gyp/util/build_utils.py',
22 '<(DEPTH)/build/android/gyp/lint.py', 22 '<(DEPTH)/build/android/gyp/lint.py',
23 '<(DEPTH)/build/android/lint/suppressions.xml', 23 '<(DEPTH)/build/android/lint/suppressions.xml',
24 '<(DEPTH)/build/android/AndroidManifest.xml', 24 '<(DEPTH)/build/android/AndroidManifest.xml',
25 '<(lint_jar_path)',
25 ], 26 ],
26 'action': [ 27 'action': [
27 'python', '<(DEPTH)/build/android/gyp/lint.py', 28 'python', '<(DEPTH)/build/android/gyp/lint.py',
28 '--lint-path=<(android_sdk_root)/tools/lint', 29 '--lint-path=<(android_sdk_root)/tools/lint',
29 '--config-path=<(DEPTH)/build/android/lint/suppressions.xml', 30 '--config-path=<(DEPTH)/build/android/lint/suppressions.xml',
30 '--processed-config-path=<(config_path)', 31 '--processed-config-path=<(config_path)',
31 '--manifest-path=<(DEPTH)/build/android/AndroidManifest.xml', 32 '--manifest-path=<(DEPTH)/build/android/AndroidManifest.xml',
32 '--result-path=<(result_path)', 33 '--result-path=<(result_path)',
33 '--product-dir=<(PRODUCT_DIR)', 34 '--product-dir=<(PRODUCT_DIR)',
34 '--src-dirs=>(src_dirs)', 35 '--src-dirs=>(src_dirs)',
35 '--classes-dir=<(classes_dir)', 36 '--jar-path=<(lint_jar_path)',
36 '--stamp=<(stamp_path)', 37 '--stamp=<(stamp_path)',
37 '<(is_enabled)', 38 '<(is_enabled)',
38 ], 39 ],
39 } 40 }
OLDNEW
« no previous file with comments | « build/android/gyp/lint.py ('k') | build/android/push_libraries.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698