Commit e5379697 authored by frankf@chromium.org's avatar frankf@chromium.org

[Android] Enable lint warnings by default.

Lint has been running on FYI bots for a while,
time to enable them by default on bots and for developers.

BUG=None
R=yfriedman@chromium.org

Review URL: https://codereview.chromium.org/127383002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@244256 0039d316-1c4b-4281-b951-d872f2087c98
parent 0121c105
...@@ -150,7 +150,7 @@ def GetBotStepMap(): ...@@ -150,7 +150,7 @@ def GetBotStepMap():
H(compile_step + std_host_tests, experimental, target_arch='x86')), H(compile_step + std_host_tests, experimental, target_arch='x86')),
B('fyi-builder-dbg', B('fyi-builder-dbg',
H(std_build_steps + std_host_tests, experimental, H(std_build_steps + std_host_tests, experimental,
extra_gyp='emma_coverage=1 android_lint=1')), extra_gyp='emma_coverage=1')),
B('x86-builder-dbg', B('x86-builder-dbg',
H(compile_step + std_host_tests, target_arch='x86')), H(compile_step + std_host_tests, target_arch='x86')),
B('fyi-builder-rel', H(std_build_steps, experimental)), B('fyi-builder-rel', H(std_build_steps, experimental)),
......
...@@ -55,17 +55,15 @@ def _RunLint(lint_path, config_path, processed_config_path, manifest_path, ...@@ -55,17 +55,15 @@ def _RunLint(lint_path, config_path, processed_config_path, manifest_path,
print >> sys.stderr print >> sys.stderr
for issue in issues: for issue in issues:
issue_id = issue.attributes['id'].value issue_id = issue.attributes['id'].value
severity = issue.attributes['severity'].value
message = issue.attributes['message'].value message = issue.attributes['message'].value
location_elem = issue.getElementsByTagName('location')[0] location_elem = issue.getElementsByTagName('location')[0]
path = location_elem.attributes['file'].value path = location_elem.attributes['file'].value
line = location_elem.getAttribute('line') line = location_elem.getAttribute('line')
if line: if line:
error = '%s:%s %s: %s [%s]' % (path, line, severity, message, error = '%s:%s %s: %s [warning]' % (path, line, message, issue_id)
issue_id)
else: else:
# Issues in class files don't have a line number. # Issues in class files don't have a line number.
error = '%s %s: %s [%s]' % (path, severity, message, issue_id) error = '%s %s: %s [warning]' % (path, message, issue_id)
print >> sys.stderr, error print >> sys.stderr, error
for attr in ['errorLine1', 'errorLine2']: for attr in ['errorLine1', 'errorLine2']:
error_line = issue.getAttribute(attr) error_line = issue.getAttribute(attr)
......
...@@ -1027,7 +1027,7 @@ ...@@ -1027,7 +1027,7 @@
'emma_filter%': '', 'emma_filter%': '',
# Set to 1 to enable running Android lint on java/class files. # Set to 1 to enable running Android lint on java/class files.
'android_lint%': 0, 'android_lint%': 1,
# Set to 1 to force Visual C++ to use legacy debug information format /Z7. # Set to 1 to force Visual C++ to use legacy debug information format /Z7.
# This is useful for parallel compilation tools which can't support /Zi. # This is useful for parallel compilation tools which can't support /Zi.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment