[Lldb-commits] [lldb] b773da0 - [lldb][test] Use $(STRIP) instead of strip in API tests (Darwin-only change) (#111816)
via lldb-commits
lldb-commits at lists.llvm.org
Thu Oct 10 05:21:29 PDT 2024
Author: Vladislav Dzhidzhoev
Date: 2024-10-10T14:21:25+02:00
New Revision: b773da0c5eed06f21f4caeea5eae47cacefb376c
URL: https://github.com/llvm/llvm-project/commit/b773da0c5eed06f21f4caeea5eae47cacefb376c
DIFF: https://github.com/llvm/llvm-project/commit/b773da0c5eed06f21f4caeea5eae47cacefb376c.diff
LOG: [lldb][test] Use $(STRIP) instead of strip in API tests (Darwin-only change) (#111816)
This makes tests more portable.
Make variables for LLVM utils are passed to `make` on Darwin as well.
Co-authored-by: Vladimir Vereschaka <vvereschaka at accesssoftek.com>
Added:
Modified:
lldb/packages/Python/lldbsuite/test/builders/builder.py
lldb/test/API/lang/objc/hidden-ivars/Makefile
lldb/test/API/lang/objc/objc-ivar-stripped/Makefile
lldb/test/API/lang/objc/objc-static-method-stripped/Makefile
lldb/test/API/macosx/add-dsym/Makefile
lldb/test/API/tools/lldb-dap/module/Makefile
lldb/test/API/tools/lldb-dap/terminated-event/Makefile
Removed:
################################################################################
diff --git a/lldb/packages/Python/lldbsuite/test/builders/builder.py b/lldb/packages/Python/lldbsuite/test/builders/builder.py
index f813d68e46e82a..d399a5b228c131 100644
--- a/lldb/packages/Python/lldbsuite/test/builders/builder.py
+++ b/lldb/packages/Python/lldbsuite/test/builders/builder.py
@@ -169,31 +169,31 @@ def getToolchainUtil(util_name):
if not os.getenv("LLVM_AR"):
utils.extend(["LLVM_AR=%s" % getToolchainUtil("llvm-ar")])
- if not lldbplatformutil.platformIsDarwin():
- if cc_type in ["clang", "cc", "gcc"]:
- util_paths = {}
- # Assembly a toolchain side tool cmd based on passed CC.
- for var, name in util_names.items():
- # Do not override explicity specified tool from the cmd line.
- if not os.getenv(var):
- util_paths[var] = getToolchainUtil("llvm-" + name)
- else:
- util_paths[var] = os.getenv(var)
- utils.extend(["AR=%s" % util_paths["ARCHIVER"]])
-
- # Look for llvm-dwp or gnu dwp
- if not lldbutil.which(util_paths["DWP"]):
- util_paths["DWP"] = getToolchainUtil("llvm-dwp")
- if not lldbutil.which(util_paths["DWP"]):
- util_paths["DWP"] = lldbutil.which("llvm-dwp")
+ if cc_type in ["clang", "cc", "gcc"]:
+ util_paths = {}
+ # Assembly a toolchain side tool cmd based on passed CC.
+ for var, name in util_names.items():
+ # Do not override explicity specified tool from the cmd line.
+ if not os.getenv(var):
+ util_paths[var] = getToolchainUtil("llvm-" + name)
+ else:
+ util_paths[var] = os.getenv(var)
+ utils.extend(["AR=%s" % util_paths["ARCHIVER"]])
+
+ # Look for llvm-dwp or gnu dwp
+ if not lldbutil.which(util_paths["DWP"]):
+ util_paths["DWP"] = getToolchainUtil("llvm-dwp")
+ if not lldbutil.which(util_paths["DWP"]):
+ util_paths["DWP"] = lldbutil.which("llvm-dwp")
+ if not util_paths["DWP"]:
+ util_paths["DWP"] = lldbutil.which("dwp")
if not util_paths["DWP"]:
- util_paths["DWP"] = lldbutil.which("dwp")
- if not util_paths["DWP"]:
- del util_paths["DWP"]
+ del util_paths["DWP"]
- for var, path in util_paths.items():
- utils.append("%s=%s" % (var, path))
- else:
+ for var, path in util_paths.items():
+ utils.append("%s=%s" % (var, path))
+
+ if lldbplatformutil.platformIsDarwin():
utils.extend(["AR=%slibtool" % os.getenv("CROSS_COMPILE", "")])
return [
diff --git a/lldb/test/API/lang/objc/hidden-ivars/Makefile b/lldb/test/API/lang/objc/hidden-ivars/Makefile
index 283e8a118fb16a..c94c0dee1b9ce9 100644
--- a/lldb/test/API/lang/objc/hidden-ivars/Makefile
+++ b/lldb/test/API/lang/objc/hidden-ivars/Makefile
@@ -14,8 +14,8 @@ endif
stripped: a.out libInternalDefiner.dylib
mkdir stripped
- strip -Sx a.out -o stripped/a.out
- strip -Sx libInternalDefiner.dylib -o stripped/libInternalDefiner.dylib
+ $(STRIP) -Sx a.out -o stripped/a.out
+ $(STRIP) -Sx libInternalDefiner.dylib -o stripped/libInternalDefiner.dylib
ifneq "$(CODESIGN)" ""
$(CODESIGN) -fs - stripped/a.out
endif
diff --git a/lldb/test/API/lang/objc/objc-ivar-stripped/Makefile b/lldb/test/API/lang/objc/objc-ivar-stripped/Makefile
index 8b63215d6d9da6..eed66d2a965d11 100644
--- a/lldb/test/API/lang/objc/objc-ivar-stripped/Makefile
+++ b/lldb/test/API/lang/objc/objc-ivar-stripped/Makefile
@@ -6,7 +6,7 @@ all: a.out.stripped
include Makefile.rules
a.out.stripped: a.out.dSYM
- strip -o a.out.stripped a.out
+ $(STRIP) -o a.out.stripped a.out
ifneq "$(CODESIGN)" ""
$(CODESIGN) -fs - a.out.stripped
endif
diff --git a/lldb/test/API/lang/objc/objc-static-method-stripped/Makefile b/lldb/test/API/lang/objc/objc-static-method-stripped/Makefile
index ed312938c9cd11..4936553c56f7c0 100644
--- a/lldb/test/API/lang/objc/objc-static-method-stripped/Makefile
+++ b/lldb/test/API/lang/objc/objc-static-method-stripped/Makefile
@@ -4,7 +4,7 @@ LD_EXTRAS := -lobjc -framework Foundation
default: a.out.stripped
a.out.stripped: a.out.dSYM
- strip -o a.out.stripped a.out
+ $(STRIP) -o a.out.stripped a.out
ln -sf a.out.dSYM a.out.stripped.dSYM
include Makefile.rules
diff --git a/lldb/test/API/macosx/add-dsym/Makefile b/lldb/test/API/macosx/add-dsym/Makefile
index 4e1ec2202d0b09..b949b308d3acce 100644
--- a/lldb/test/API/macosx/add-dsym/Makefile
+++ b/lldb/test/API/macosx/add-dsym/Makefile
@@ -8,7 +8,7 @@ hide.app/Contents/a.out.dSYM:
mkdir hide.app
mkdir hide.app/Contents
mv a.out.dSYM hide.app/Contents
- strip -x a.out
+ $(STRIP) -x a.out
ifneq "$(CODESIGN)" ""
$(CODESIGN) -fs - a.out
endif
diff --git a/lldb/test/API/tools/lldb-dap/module/Makefile b/lldb/test/API/tools/lldb-dap/module/Makefile
index b30baf48b972ef..c7d626a1a7e4c1 100644
--- a/lldb/test/API/tools/lldb-dap/module/Makefile
+++ b/lldb/test/API/tools/lldb-dap/module/Makefile
@@ -10,7 +10,7 @@ include Makefile.rules
all: a.out.stripped
a.out.stripped:
- strip -o a.out.stripped a.out
+ $(STRIP) -o a.out.stripped a.out
ifneq "$(CODESIGN)" ""
$(CODESIGN) -fs - a.out.stripped
diff --git a/lldb/test/API/tools/lldb-dap/terminated-event/Makefile b/lldb/test/API/tools/lldb-dap/terminated-event/Makefile
index b30baf48b972ef..c7d626a1a7e4c1 100644
--- a/lldb/test/API/tools/lldb-dap/terminated-event/Makefile
+++ b/lldb/test/API/tools/lldb-dap/terminated-event/Makefile
@@ -10,7 +10,7 @@ include Makefile.rules
all: a.out.stripped
a.out.stripped:
- strip -o a.out.stripped a.out
+ $(STRIP) -o a.out.stripped a.out
ifneq "$(CODESIGN)" ""
$(CODESIGN) -fs - a.out.stripped
More information about the lldb-commits
mailing list