From 919a042833202817492d1744e087836b6b02f63f Mon Sep 17 00:00:00 2001 From: der richter Date: Sat, 13 Apr 2024 12:42:32 +0200 Subject: osxbundle: use arg list instead of a string for signing subprocess call Fixes #13872 --- TOOLS/osxbundle.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) (limited to 'TOOLS') diff --git a/TOOLS/osxbundle.py b/TOOLS/osxbundle.py index fc8379c748..0e156a0b02 100755 --- a/TOOLS/osxbundle.py +++ b/TOOLS/osxbundle.py @@ -1,14 +1,11 @@ #!/usr/bin/env python3 import os import shutil -import sys import fileinput import dylib_unhell +import subprocess from optparse import OptionParser -def sh(command): - return os.popen(command).read().strip() - def bundle_path(binary_name): return "%s.app" % binary_name @@ -45,8 +42,8 @@ def sign_bundle(binary_name): resolved_dir = os.path.join(bundle_path(binary_name), dir) for root, _dirs, files in os.walk(resolved_dir): for f in files: - sh('codesign --force -s - ' + os.path.join(root, f)) - sh('codesign --force -s - ' + bundle_path(binary_name)) + subprocess.run(['codesign', '--force', '-s', '-', os.path.join(root, f)]) + subprocess.run(['codesign', '--force', '-s', '-', bundle_path(binary_name)]) def bundle_version(src_path): version = 'UNKNOWN' -- cgit v1.2.3