llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: None (n2h9)

<details>
<summary>Changes</summary>

While taking a look at the code of lldb test-suite packages, I have noticed 
that in `get_triple_str` in `darwin.py` env is added inside a `components` 
list, which is probably supposed to be `component` (defined on the line 61). 

---
Full diff: https://github.com/llvm/llvm-project/pull/169254.diff


1 Files Affected:

- (modified) lldb/packages/Python/lldbsuite/test/builders/darwin.py (+1-1) 


``````````diff
diff --git a/lldb/packages/Python/lldbsuite/test/builders/darwin.py 
b/lldb/packages/Python/lldbsuite/test/builders/darwin.py
index a023bda3ad801..eebe0ef47fd85 100644
--- a/lldb/packages/Python/lldbsuite/test/builders/darwin.py
+++ b/lldb/packages/Python/lldbsuite/test/builders/darwin.py
@@ -60,7 +60,7 @@ def get_triple_str(arch, vendor, os, version, env):
 
     component = [arch, vendor, os + version]
     if env:
-        components.append(env)
+        component.append(env)
     return "-".join(component)
 
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/169254
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to