llvmorg-github-actions[bot] wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Dave Lee (kastiglione)

<details>
<summary>Changes</summary>

Fixes the following `--help` bug I ran into:

```
% python3 ./lldb/test/API/dotest.py --help
Traceback (most recent call last):
  File "path/to/llvm-project/lldb/test/API/dotest.py", line 8, in &lt;module&gt;
    lldbsuite.test.run_suite()
  File "path/to/llvm-project/lldb/packages/Python/lldbsuite/test/dotest.py", 
line 1099, in run_suite
    parseOptionsAndInitTestdirs()
  File "path/to/llvm-project/lldb/packages/Python/lldbsuite/test/dotest.py", 
line 318, in parseOptionsAndInitTestdirs
    configuration.cmake_build_type = args.cmake_build_type.lower()
AttributeError: 'NoneType' object has no attribute 'lower'
```

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


1 Files Affected:

- (modified) lldb/packages/Python/lldbsuite/test/dotest.py (+2-1) 


``````````diff
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py 
b/lldb/packages/Python/lldbsuite/test/dotest.py
index 6555c60dc3bff..dec46c7715e40 100644
--- a/lldb/packages/Python/lldbsuite/test/dotest.py
+++ b/lldb/packages/Python/lldbsuite/test/dotest.py
@@ -316,7 +316,8 @@ def parseOptionsAndInitTestdirs():
         configuration.libcxx_include_target_dir = None
         configuration.libcxx_library_dir = None
 
-    configuration.cmake_build_type = args.cmake_build_type.lower()
+    if args.cmake_build_type:
+        configuration.cmake_build_type = args.cmake_build_type.lower()
 
     if args.channels:
         lldbtest_config.channels = args.channels

``````````

</details>


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

Reply via email to