[lldb] Remove obsolete signBinary helper (#79656)

On Darwin, the Makefile already (ad-hoc) signs everything it builds.
There's also no need to use lldb_codesign for this.
This commit is contained in:
Jonas Devlieghere 2024-01-26 20:37:44 -08:00 committed by GitHub
parent 701ec45f2f
commit 7595287303
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
8 changed files with 0 additions and 26 deletions

View File

@ -442,8 +442,6 @@ def parseOptionsAndInitTestdirs():
os.path.realpath(os.path.abspath(x)) for x in args.args
]
lldbtest_config.codesign_identity = args.codesign_identity
def registerFaulthandler():
try:

View File

@ -217,12 +217,6 @@ def create_parser():
action="store_true",
help="Leave logs/traces even for successful test runs (useful for creating reference log files during debugging.)",
)
group.add_argument(
"--codesign-identity",
metavar="Codesigning identity",
default="lldb_codesign",
help="The codesigning identity to use",
)
group.add_argument(
"--build-dir",
dest="test_build_dir",

View File

@ -1543,14 +1543,6 @@ class Base(unittest2.TestCase):
d = {"CXX_SOURCES": sources, "EXE": exe_name}
self.build(dictionary=d)
def signBinary(self, binary_path):
if sys.platform.startswith("darwin"):
codesign_cmd = 'codesign --force --sign "%s" %s' % (
lldbtest_config.codesign_identity,
binary_path,
)
call(codesign_cmd, shell=True)
def findBuiltClang(self):
"""Tries to find and use Clang from the build directory as the compiler (instead of the system compiler)."""
paths_to_try = [

View File

@ -98,12 +98,6 @@ if(CMAKE_HOST_APPLE)
set(LLDB_FRAMEWORK_DIR ${LLDB_FRAMEWORK_ABSOLUTE_BUILD_DIR}/LLDB.framework)
endif()
# Use the same identity for testing
get_property(code_sign_identity_used GLOBAL PROPERTY LLDB_DEBUGSERVER_CODESIGN_IDENTITY)
if(code_sign_identity_used)
list(APPEND LLDB_TEST_COMMON_ARGS_VAR --codesign-identity "${code_sign_identity_used}")
endif()
if(LLDB_USE_SYSTEM_DEBUGSERVER)
lldb_find_system_debugserver(system_debugserver_path)
if(LLDB_BUILD_FRAMEWORK)

View File

@ -18,7 +18,6 @@ class TestSBCommandReturnObject(TestBase):
self.driver_exe = self.getBuildArtifact("command-return-object")
self.buildDriver("main.cpp", self.driver_exe)
self.addTearDownHook(lambda: os.remove(self.driver_exe))
self.signBinary(self.driver_exe)
if self.TraceOn():
print("Running test %s" % self.driver_exe)

View File

@ -24,7 +24,6 @@ class TestMultipleSimultaneousDebuggers(TestBase):
self.driver_exe = self.getBuildArtifact("multi-process-driver")
self.buildDriver("multi-process-driver.cpp", self.driver_exe)
self.addTearDownHook(lambda: os.remove(self.driver_exe))
self.signBinary(self.driver_exe)
self.inferior_exe = self.getBuildArtifact("testprog")
self.buildDriver("testprog.cpp", self.inferior_exe)

View File

@ -24,7 +24,6 @@ class TestMultipleTargets(TestBase):
self.driver_exe = self.getBuildArtifact("multi-target")
self.buildDriver("main.cpp", self.driver_exe)
self.addTearDownHook(lambda: os.remove(self.driver_exe))
self.signBinary(self.driver_exe)
# check_call will raise a CalledProcessError if multi-process-driver doesn't return
# exit code 0 to indicate success. We can let this exception go - the test harness

View File

@ -106,7 +106,6 @@ class SBBreakpointCallbackCase(TestBase):
self.addTearDownHook(lambda: os.remove(self.getBuildArtifact(test_name)))
test_exe = self.getBuildArtifact(test_name)
self.signBinary(test_exe)
exe = [test_exe, self.getBuildArtifact(self.inferior)]
env = {self.dylibPath: self.getLLDBLibraryEnvVal()}