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

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

Issue 1870073002: Properly set up classpath for Android Lint (and fix its stderr filter) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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
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)',
(...skipping 19 matching lines...) Expand all
30 '<(lint_jar_path)', 30 '<(lint_jar_path)',
31 '<(suppressions_file)', 31 '<(suppressions_file)',
32 '<(platform_xml_path)', 32 '<(platform_xml_path)',
33 ], 33 ],
34 'action': [ 34 'action': [
35 'python', '<(DEPTH)/build/android/gyp/lint.py', 35 'python', '<(DEPTH)/build/android/gyp/lint.py',
36 '--lint-path=<(android_sdk_root)/tools/lint', 36 '--lint-path=<(android_sdk_root)/tools/lint',
37 '--config-path=<(suppressions_file)', 37 '--config-path=<(suppressions_file)',
38 '--processed-config-path=<(config_path)', 38 '--processed-config-path=<(config_path)',
39 '--cache-dir', '<(PRODUCT_DIR)/android_lint_cache', 39 '--cache-dir', '<(PRODUCT_DIR)/android_lint_cache',
40 '--android-sdk-version=<(android_sdk_version)',
agrieve 2016/04/08 15:16:12 Since at the moment lint warnings fail GYP builds
mlopatkin 2016/04/08 16:55:16 We can make --android-sdk-version optional for now
40 '--platform-xml-path', '<(platform_xml_path)', 41 '--platform-xml-path', '<(platform_xml_path)',
41 '--manifest-path=<(android_manifest_path)', 42 '--manifest-path=<(android_manifest_path)',
42 '--result-path=<(result_path)', 43 '--result-path=<(result_path)',
43 '--resource-dir=<(resource_dir)', 44 '--resource-dir=<(resource_dir)',
44 '--product-dir=<(PRODUCT_DIR)', 45 '--product-dir=<(PRODUCT_DIR)',
45 '--src-dirs=>(src_dirs)', 46 '--src-dirs=>(src_dirs)',
46 '--jar-path=<(lint_jar_path)', 47 '--jar-path=<(lint_jar_path)',
47 '--can-fail-build', 48 '--can-fail-build',
48 '--stamp=<(stamp_path)', 49 '--stamp=<(stamp_path)',
49 '<@(additional_args)', 50 '<@(additional_args)',
50 ], 51 ],
51 } 52 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698