mirror of
https://github.com/Mbed-TLS/mbedtls.git
synced 2025-04-17 20:42:44 +00:00
Clean up subprocess invocation in get_src_files
Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
This commit is contained in:
parent
e25c43bd66
commit
22eb82cb8d
@ -65,29 +65,27 @@ def list_generated_files() -> FrozenSet[str]:
|
|||||||
|
|
||||||
def get_src_files() -> List[str]:
|
def get_src_files() -> List[str]:
|
||||||
"""
|
"""
|
||||||
Use git ls-files to get a list of the source files
|
Use git to get a list of the source files.
|
||||||
|
|
||||||
|
Only C files are included, and certain files (generated, or 3rdparty)
|
||||||
|
are excluded.
|
||||||
"""
|
"""
|
||||||
git_ls_files_cmd = ["git", "ls-files",
|
git_ls_files_cmd = ["git", "ls-files",
|
||||||
"*.[hc]",
|
"*.[hc]",
|
||||||
"tests/suites/*.function",
|
"tests/suites/*.function",
|
||||||
"scripts/data_files/*.fmt"]
|
"scripts/data_files/*.fmt"]
|
||||||
|
output = subprocess.check_output(git_ls_files_cmd,
|
||||||
|
universal_newlines=True)
|
||||||
|
src_files = output.split()
|
||||||
|
|
||||||
result = subprocess.run(git_ls_files_cmd, stdout=subprocess.PIPE,
|
generated_files = list_generated_files()
|
||||||
check=False)
|
# Don't correct style for third-party files (and, for simplicity,
|
||||||
|
# companion files in the same subtree), or for automatically
|
||||||
if result.returncode != 0:
|
# generated files (we're correcting the templates instead).
|
||||||
print_err("git ls-files returned: " + str(result.returncode))
|
src_files = [filename for filename in src_files
|
||||||
return []
|
if not (filename.startswith("3rdparty/") or
|
||||||
else:
|
filename in generated_files)]
|
||||||
generated_files = list_generated_files()
|
return src_files
|
||||||
src_files = str(result.stdout, "utf-8").split()
|
|
||||||
# Don't correct style for third-party files (and, for simplicity,
|
|
||||||
# companion files in the same subtree), or for automatically
|
|
||||||
# generated files (we're correcting the templates instead).
|
|
||||||
src_files = [filename for filename in src_files
|
|
||||||
if not (filename.startswith("3rdparty/") or
|
|
||||||
filename in generated_files)]
|
|
||||||
return src_files
|
|
||||||
|
|
||||||
def get_uncrustify_version() -> str:
|
def get_uncrustify_version() -> str:
|
||||||
"""
|
"""
|
||||||
|
Loading…
x
Reference in New Issue
Block a user