Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(176)

Side by Side Diff: tools-webrtc/check_package_boundaries.py

Issue 2736233003: PyLint fixes for tools-webrtc and webrtc/tools (Closed)
Patch Set: Fix alignment Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 2
3 # Copyright (c) 2017 The WebRTC project authors. All Rights Reserved. 3 # Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
4 # 4 #
5 # Use of this source code is governed by a BSD-style license 5 # Use of this source code is governed by a BSD-style license
6 # that can be found in the LICENSE file in the root of the source 6 # that can be found in the LICENSE file in the root of the source
7 # tree. An additional intellectual property rights grant can be found 7 # tree. An additional intellectual property rights grant can be found
8 # in the file PATENTS. All contributing project authors may 8 # in the file PATENTS. All contributing project authors may
9 # be found in the AUTHORS file in the root of the source tree. 9 # be found in the AUTHORS file in the root of the source tree.
10 10
(...skipping 21 matching lines...) Expand all
32 ERROR_MESSAGE = ("{}:{} in target '{}':\n" 32 ERROR_MESSAGE = ("{}:{} in target '{}':\n"
33 " Source file '{}'\n" 33 " Source file '{}'\n"
34 " crosses boundary of package '{}'.\n") 34 " crosses boundary of package '{}'.\n")
35 35
36 36
37 class Logger(object): 37 class Logger(object):
38 def __init__(self, messages_left=None): 38 def __init__(self, messages_left=None):
39 self.log_level = DISPLAY_LEVEL 39 self.log_level = DISPLAY_LEVEL
40 self.messages_left = messages_left 40 self.messages_left = messages_left
41 41
42 def log(self, build_file_path, line_number, target_name, source_file, 42 def Log(self, build_file_path, line_number, target_name, source_file,
43 subpackage): 43 subpackage):
44 if self.messages_left is not None: 44 if self.messages_left is not None:
45 if not self.messages_left: 45 if not self.messages_left:
46 self.log_level = IGNORE_LEVEL 46 self.log_level = IGNORE_LEVEL
47 else: 47 else:
48 self.messages_left -= 1 48 self.messages_left -= 1
49 message = ERROR_MESSAGE.format(build_file_path, line_number, target_name, 49 message = ERROR_MESSAGE.format(build_file_path, line_number, target_name,
50 source_file, subpackage) 50 source_file, subpackage)
51 logging.log(self.log_level, message) 51 logging.log(self.log_level, message)
52 52
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 target_name = target_match.group('target_name') 84 target_name = target_match.group('target_name')
85 target_contents = target_match.group('target_contents') 85 target_contents = target_match.group('target_contents')
86 for sources_match in SOURCES_RE.finditer(target_contents): 86 for sources_match in SOURCES_RE.finditer(target_contents):
87 sources = sources_match.group('sources') 87 sources = sources_match.group('sources')
88 for subpackages_match in subpackages_re.finditer(sources): 88 for subpackages_match in subpackages_re.finditer(sources):
89 subpackage = subpackages_match.group('subpackage') 89 subpackage = subpackages_match.group('subpackage')
90 source_file = subpackages_match.group('source_file') 90 source_file = subpackages_match.group('source_file')
91 line_number = subpackages_match.group('line_number') 91 line_number = subpackages_match.group('line_number')
92 if subpackage: 92 if subpackage:
93 found_violations = True 93 found_violations = True
94 logger.log(build_file_path, line_number, target_name, source_file, 94 logger.Log(build_file_path, line_number, target_name, source_file,
95 subpackage) 95 subpackage)
96 96
97 return found_violations 97 return found_violations
98 98
99 99
100 def CheckPackageBoundaries(root_dir, logger, build_files=None): 100 def CheckPackageBoundaries(root_dir, logger, build_files=None):
101 packages = [root for root, _, files in os.walk(root_dir) 101 packages = [root for root, _, files in os.walk(root_dir)
102 if 'BUILD.gn' in files] 102 if 'BUILD.gn' in files]
103 default_build_files = [os.path.join(package, 'BUILD.gn') 103 default_build_files = [os.path.join(package, 'BUILD.gn')
104 for package in packages] 104 for package in packages]
(...skipping 23 matching lines...) Expand all
128 128
129 logging.basicConfig(format=LOG_FORMAT) 129 logging.basicConfig(format=LOG_FORMAT)
130 logging.getLogger().setLevel(DISPLAY_LEVEL) 130 logging.getLogger().setLevel(DISPLAY_LEVEL)
131 logger = Logger(args.max_messages) 131 logger = Logger(args.max_messages)
132 132
133 return CheckPackageBoundaries(args.root_dir, logger, args.build_files) 133 return CheckPackageBoundaries(args.root_dir, logger, args.build_files)
134 134
135 135
136 if __name__ == '__main__': 136 if __name__ == '__main__':
137 sys.exit(main()) 137 sys.exit(main())
OLDNEW
« no previous file with comments | « tools-webrtc/autoroller/unittests/roll_deps_test.py ('k') | tools-webrtc/check_package_boundaries_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698