[Lldb-commits] [lldb] 058ede0 - [lldb][test] Use `xcrun -f strip` for API tests on Darwin (#111842)

[via lldb-commits]

Author: Vladislav Dzhidzhoev
Date: 2024-10-10T17:14:13+02:00
New Revision: 058ede06c4ffd4e3c9f54d947e3bfb027c2d0557

URL: https://github.com/llvm/llvm-project/commit/058ede06c4ffd4e3c9f54d947e3bfb027c2d0557
DIFF: https://github.com/llvm/llvm-project/commit/058ede06c4ffd4e3c9f54d947e3bfb027c2d0557.diff

LOG: [lldb][test] Use `xcrun -f strip` for API tests on Darwin (#111842)

A follow-up for https://github.com/llvm/llvm-project/pull/111816.

This is to fix buildbot failure
https://lab.llvm.org/staging/#/builders/195/builds/4242.

TestSymbolFileJSON.py doesn't pass with llvm-strip on macOS. Apparently,
llvm-strip/llvm-objcopy can't clean symbols from Mach-O nlists.

Added: 
    

Modified: 
    lldb/packages/Python/lldbsuite/test/builders/builder.py
    lldb/test/API/functionalities/json/symbol-file/Makefile

Removed: 
    


################################################################################
diff  --git a/lldb/packages/Python/lldbsuite/test/builders/builder.py b/lldb/packages/Python/lldbsuite/test/builders/builder.py
index d399a5b228c131..de057324694486 100644
--- a/lldb/packages/Python/lldbsuite/test/builders/builder.py
+++ b/lldb/packages/Python/lldbsuite/test/builders/builder.py
@@ -10,6 +10,7 @@
 import lldbsuite.test.lldbutil as lldbutil
 from lldbsuite.test import configuration
 from lldbsuite.test_event import build_exception
+from lldbsuite.support import seven
 
 
 class Builder:
@@ -190,6 +191,9 @@ def getToolchainUtil(util_name):
                 if not util_paths["DWP"]:
                     del util_paths["DWP"]
 
+            if lldbplatformutil.platformIsDarwin():
+                util_paths["STRIP"] = seven.get_command_output("xcrun -f strip")
+
             for var, path in util_paths.items():
                 utils.append("%s=%s" % (var, path))
 

diff  --git a/lldb/test/API/functionalities/json/symbol-file/Makefile b/lldb/test/API/functionalities/json/symbol-file/Makefile
index aff841c364299c..13bc164582eeee 100644
--- a/lldb/test/API/functionalities/json/symbol-file/Makefile
+++ b/lldb/test/API/functionalities/json/symbol-file/Makefile
@@ -3,6 +3,6 @@ C_SOURCES := main.c
 all: stripped.out
 
 stripped.out : a.out
-	strip a.out -o stripped.out
+	$(STRIP) a.out -o stripped.out
 
 include Makefile.rules