Alex Light | c29da37 | 2017-03-31 09:20:02 -0700 | [diff] [blame] | 1 | #!/usr/bin/python3 |
| 2 | # |
| 3 | # Copyright 2017, The Android Open Source Project |
| 4 | # |
| 5 | # Licensed under the Apache License, Version 2.0 (the "License"); |
| 6 | # you may not use this file except in compliance with the License. |
| 7 | # You may obtain a copy of the License at |
| 8 | # |
| 9 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 10 | # |
| 11 | # Unless required by applicable law or agreed to in writing, software |
| 12 | # distributed under the License is distributed on an "AS IS" BASIS, |
| 13 | # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 14 | # See the License for the specific language governing permissions and |
| 15 | # limitations under the License. |
| 16 | |
| 17 | # TODO We should unify this with build/Android.cpplint.mk. |
| 18 | |
| 19 | import os |
| 20 | import pathlib |
| 21 | import subprocess |
| 22 | import sys |
| 23 | |
Andreas Gampe | 06c42a5 | 2017-07-26 14:17:14 -0700 | [diff] [blame] | 24 | IGNORED_FILES = {"runtime/elf.h", "openjdkjvmti/include/jvmti.h"} |
Alex Light | c29da37 | 2017-03-31 09:20:02 -0700 | [diff] [blame] | 25 | |
| 26 | INTERESTING_SUFFIXES = {".h", ".cc"} |
| 27 | |
| 28 | CPPLINT_FLAGS = [ |
| 29 | '--filter=-whitespace/line_length,-build/include,-readability/function,-readability/streams,-readability/todo,-runtime/references,-runtime/sizeof,-runtime/threadsafe_fn,-runtime/printf', |
| 30 | '--quiet', |
| 31 | ] |
| 32 | |
| 33 | def is_interesting(f): |
| 34 | """ |
| 35 | Returns true if this is a file we want to run through cpplint before uploading. False otherwise. |
| 36 | """ |
| 37 | path = pathlib.Path(f) |
| 38 | return f not in IGNORED_FILES and path.suffix in INTERESTING_SUFFIXES and path.exists() |
| 39 | |
| 40 | def get_changed_files(commit): |
| 41 | """ |
| 42 | Gets the files changed in the given commit. |
| 43 | """ |
| 44 | return subprocess.check_output( |
| 45 | ["git", 'diff-tree', '--no-commit-id', '--name-only', '-r', commit], |
| 46 | stderr=subprocess.STDOUT, |
| 47 | universal_newlines=True).split() |
| 48 | |
| 49 | def run_cpplint(files): |
| 50 | """ |
| 51 | Runs cpplint on the given files. |
| 52 | """ |
| 53 | if len(files) == 0: |
| 54 | return |
| 55 | sys.exit(subprocess.call(['tools/cpplint.py'] + CPPLINT_FLAGS + files)) |
| 56 | |
| 57 | def main(): |
| 58 | if 'PREUPLOAD_COMMIT' in os.environ: |
| 59 | commit = os.environ['PREUPLOAD_COMMIT'] |
| 60 | else: |
| 61 | print("WARNING: Not running as a pre-upload hook. Assuming commit to check = 'HEAD'") |
| 62 | commit = "HEAD" |
| 63 | files_to_check = [f for f in get_changed_files(commit) if is_interesting(f)] |
| 64 | run_cpplint(files_to_check) |
| 65 | |
| 66 | if __name__ == '__main__': |
| 67 | main() |