From 6d49ef99714c2e5faa5ef64522653307f7e6640f Mon Sep 17 00:00:00 2001 From: Christopher Harris Date: Fri, 4 Dec 2015 11:41:31 -0600 Subject: [PATCH] Python's builtin os.walk Using python's builtin os.walk to look for remap files instead of shell command output string parsing. --- utils/apply-fixit-edits.py | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/utils/apply-fixit-edits.py b/utils/apply-fixit-edits.py index 674ee7745c0e1..9ccc0187d3941 100755 --- a/utils/apply-fixit-edits.py +++ b/utils/apply-fixit-edits.py @@ -17,16 +17,14 @@ import json import argparse import sys +import os def find_remap_files(path): - out = None - try: - out = subprocess.check_output(["find", path, "-name", "*.remap"]) - except subprocess.CalledProcessError: - return None - lines = out.split('\n') - lines.pop(-1) - return lines + for root, dirs, files in os.walk(path): + for filename in files: + if not filename.endswith(".remap"): + continue + yield os.path.join(root, filename) def apply_edits(path): remap_files = find_remap_files(path)