From c251e0d75a9a2c69e832b605ac6ef3aa54ef3f21 Mon Sep 17 00:00:00 2001 From: Gilles Peskine Date: Tue, 24 Mar 2020 22:01:28 +0100 Subject: [PATCH] Clarify confusion between file names and suffixes of file names To test a file name exactly, prepend a / to the base name. files_to_check actually checks suffixes, not file names, so rename it to extensions_to_check. Signed-off-by: Gilles Peskine --- tests/scripts/check-files.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/scripts/check-files.py b/tests/scripts/check-files.py index e2d3f4ab6..5ce19cd5f 100755 --- a/tests/scripts/check-files.py +++ b/tests/scripts/check-files.py @@ -143,8 +143,8 @@ class TabIssueTracker(LineIssueTracker): heading = "Tabs present:" files_exemptions = frozenset([ - "Makefile", - "generate_visualc_files.pl", + "/Makefile", + "/generate_visualc_files.pl", ]) def issue_with_line(self, line, _filepath): @@ -179,9 +179,9 @@ class IntegrityChecker(object): self.check_repo_path() self.logger = None self.setup_logger(log_file) - self.files_to_check = ( + self.extensions_to_check = ( ".c", ".h", ".sh", ".pl", ".py", ".md", ".function", ".data", - "Makefile", "CMakeLists.txt", "ChangeLog" + "/Makefile", "/CMakeLists.txt", "/ChangeLog" ) self.excluded_directories = ['.git', 'mbed-os'] self.excluded_paths = list(map(os.path.normpath, [ @@ -226,7 +226,7 @@ class IntegrityChecker(object): dirs[:] = sorted(d for d in dirs if not self.prune_branch(root, d)) for filename in sorted(files): filepath = os.path.join(root, filename) - if not filepath.endswith(self.files_to_check): + if not filepath.endswith(self.extensions_to_check): continue for issue_to_check in self.issues_to_check: if issue_to_check.should_check_file(filepath):