Skip to content

Commit

Permalink
Merge pull request #7855 from ytec-nl/master
Browse files Browse the repository at this point in the history
Prevent CLI argument issues in scripts/glibc_check.py
  • Loading branch information
ThomasWaldmann committed Oct 8, 2023
2 parents 8fc0944 + 7afa8d4 commit d4ea884
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions scripts/glibc_check.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
import sys

verbose = True
objdump = "objdump -T %s"
glibc_re = re.compile(r"GLIBC_([0-9]\.[0-9]+)")


Expand All @@ -32,7 +31,7 @@ def main():
overall_versions = set()
for filename in filenames:
try:
output = subprocess.check_output(objdump % filename, shell=True, stderr=subprocess.STDOUT)
output = subprocess.check_output(["objdump", "-T", filename], stderr=subprocess.STDOUT)
output = output.decode()
versions = {parse_version(match.group(1)) for match in glibc_re.finditer(output)}
requires_glibc = max(versions)
Expand Down

0 comments on commit d4ea884

Please sign in to comment.