1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
|
# Copyright 2022 The Chromium Authors
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
PRESUBMIT_VERSION = '2.0.0'
import textwrap
def CheckNoBadDeps(input_api, output_api):
"""Prevent additions of bad dependencies from the //build prefix."""
build_file_patterns = [
r'(.+/)?BUILD\.gn',
r'.+\.gni',
]
exclude_file_patterns = [
r'build/rust/tests',
]
blocklist_pattern = input_api.re.compile(r'^[^#]*"//(?!build).+?/.*"')
allowlist_pattern = input_api.re.compile(r'^[^#]*"//third_party/junit')
warning_message = textwrap.dedent("""
The //build directory is meant to be as hermetic as possible so that
other projects (webrtc, v8, angle) can make use of it. If you are adding
a new dep from //build onto another directory, you should consider:
1) Can that dep live within //build?
2) Can the dep be guarded by "build_with_chromium"?
3) Have you made this new dep easy to pull in for other projects (ideally
a matter of adding a DEPS entry).:""")
def FilterFile(affected_file):
return input_api.FilterSourceFile(affected_file,
files_to_check=build_file_patterns,
files_to_skip=exclude_file_patterns)
problems = []
for f in input_api.AffectedSourceFiles(FilterFile):
local_path = f.LocalPath()
for line_number, line in f.ChangedContents():
if blocklist_pattern.search(line) and not allowlist_pattern.search(line):
problems.append('%s:%d\n %s' %
(local_path, line_number, line.strip()))
if problems:
return [output_api.PresubmitPromptOrNotify(warning_message, problems)]
else:
return []
def CheckPythonTests(input_api, output_api):
return input_api.RunTests(
input_api.canned_checks.GetUnitTestsInDirectory(
input_api,
output_api,
input_api.PresubmitLocalPath(),
files_to_check=[r'.+_(?:unit)?test\.py$']))
|
