| Index: PRESUBMIT.py
|
| diff --git a/PRESUBMIT.py b/PRESUBMIT.py
|
| index 6a836e0fefbf25f672a05bc71398e541e3dc1eec..70f576a81a4de6498eb54c20020219b6cf8ca385 100644
|
| --- a/PRESUBMIT.py
|
| +++ b/PRESUBMIT.py
|
| @@ -31,6 +31,9 @@ See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
|
| for more details about the presubmit API built into gcl.
|
| """
|
|
|
| +import sys
|
| +
|
| +
|
| def _V8PresubmitChecks(input_api, output_api):
|
| """Runs the V8 presubmit checks."""
|
| import sys
|
| @@ -49,12 +52,66 @@ def _V8PresubmitChecks(input_api, output_api):
|
| return results
|
|
|
|
|
| +def _CheckUnwantedDependencies(input_api, output_api):
|
| + """Runs checkdeps on #include statements added in this
|
| + change. Breaking - rules is an error, breaking ! rules is a
|
| + warning.
|
| + """
|
| + # We need to wait until we have an input_api object and use this
|
| + # roundabout construct to import checkdeps because this file is
|
| + # eval-ed and thus doesn't have __file__.
|
| + original_sys_path = sys.path
|
| + try:
|
| + sys.path = sys.path + [input_api.os_path.join(
|
| + input_api.PresubmitLocalPath(), 'buildtools', 'checkdeps')]
|
| + import checkdeps
|
| + from cpp_checker import CppChecker
|
| + from rules import Rule
|
| + finally:
|
| + # Restore sys.path to what it was before.
|
| + sys.path = original_sys_path
|
| +
|
| + added_includes = []
|
| + for f in input_api.AffectedFiles():
|
| + if not CppChecker.IsCppFile(f.LocalPath()):
|
| + continue
|
| +
|
| + changed_lines = [line for line_num, line in f.ChangedContents()]
|
| + added_includes.append([f.LocalPath(), changed_lines])
|
| +
|
| + deps_checker = checkdeps.DepsChecker(input_api.PresubmitLocalPath())
|
| +
|
| + error_descriptions = []
|
| + warning_descriptions = []
|
| + for path, rule_type, rule_description in deps_checker.CheckAddedCppIncludes(
|
| + added_includes):
|
| + description_with_path = '%s\n %s' % (path, rule_description)
|
| + if rule_type == Rule.DISALLOW:
|
| + error_descriptions.append(description_with_path)
|
| + else:
|
| + warning_descriptions.append(description_with_path)
|
| +
|
| + results = []
|
| + if error_descriptions:
|
| + results.append(output_api.PresubmitError(
|
| + 'You added one or more #includes that violate checkdeps rules.',
|
| + error_descriptions))
|
| + if warning_descriptions:
|
| + results.append(output_api.PresubmitPromptOrNotify(
|
| + 'You added one or more #includes of files that are temporarily\n'
|
| + 'allowed but being removed. Can you avoid introducing the\n'
|
| + '#include? See relevant DEPS file(s) for details and contacts.',
|
| + warning_descriptions))
|
| + return results
|
| +
|
| +
|
| def _CommonChecks(input_api, output_api):
|
| """Checks common to both upload and commit."""
|
| results = []
|
| results.extend(input_api.canned_checks.CheckOwners(
|
| input_api, output_api, source_file_filter=None))
|
| results.extend(_V8PresubmitChecks(input_api, output_api))
|
| + results.extend(_CheckUnwantedDependencies(input_api, output_api))
|
| return results
|
|
|
|
|
|
|