LineageOS/android_development
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Print the project name if available when running Checkstyle.

Browse Source 
Add [project/foo/bar] to the beginning of each Checkstyle error line
if the REPO_PROJECT environmental variable is set (set by repo hooks).

Test: manually attempted to run checkstyle.
Change-Id: I8ccf76d9ce9306668109029267081924300cb6ce
This commit is contained in:
Aurimas Liutikas
2016-11-03 09:37:21 -07:00
parent e958ddb230
commit e847694a3d
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
tools/checkstyle/checkstyle.py
View File

@@ -188,7 +188,11 @@ def _ParseAndFilterOutput(stdout,
if error.hasAttribute('column'): if error.hasAttribute('column'):
column = '%s:' % error.attributes['column'].value column = '%s:' % error.attributes['column'].value
message = error.attributes['message'].value message = error.attributes['message'].value
result = ' %s:%s:%s %s' % (file_name, line, column, message) project = ''
if os.environ.get('REPO_PROJECT'):
project = '[' + os.environ.get('REPO_PROJECT') + '] '
result = ' %s%s:%s:%s %s' % (project, file_name, line, column, message)
severity = error.attributes['severity'].value severity = error.attributes['severity'].value
if severity == 'error': if severity == 'error':