[Lldb-commits] [lldb] e7174a8 - [lldb][test] Provide proper path to LLVM utils in API tests (#110837)

via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 7 05:50:06 PDT 2024


Author: Vladislav Dzhidzhoev
Date: 2024-10-07T14:50:01+02:00
New Revision: e7174a8378faf291e843892323527d001220d645

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

LOG: [lldb][test] Provide proper path to LLVM utils in API tests (#110837)

In aea066849, API tests were supposed to use LLVM tools.
However, a path to a utility is made up incorrectly there: util name
should be prefixed with `llvm-`.

Hence, it's fixed here.

Added: 
    

Modified: 
    lldb/packages/Python/lldbsuite/test/builders/builder.py

Removed: 
    


################################################################################
diff  --git a/lldb/packages/Python/lldbsuite/test/builders/builder.py b/lldb/packages/Python/lldbsuite/test/builders/builder.py
index e3099219e437e3..f813d68e46e82a 100644
--- a/lldb/packages/Python/lldbsuite/test/builders/builder.py
+++ b/lldb/packages/Python/lldbsuite/test/builders/builder.py
@@ -176,7 +176,7 @@ def getToolchainUtil(util_name):
                 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(name)
+                        util_paths[var] = getToolchainUtil("llvm-" + name)
                     else:
                         util_paths[var] = os.getenv(var)
                 utils.extend(["AR=%s" % util_paths["ARCHIVER"]])


        


More information about the lldb-commits mailing list