[Lldb-commits] [lldb] [lldb][test] Only calculate LLDB python path once (PR #201327)

[via lldb-commits]

github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {darker}-->


:warning: Python code formatter, darker found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
darker --check --diff -r origin/main...HEAD lldb/packages/Python/lldbsuite/test/configuration.py lldb/packages/Python/lldbsuite/test/dotest.py lldb/packages/Python/lldbsuite/test/dotest_args.py lldb/test/API/commands/quit/TestQuit.py lldb/test/API/lit.cfg.py
``````````

:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:

</details>

<details>
<summary>
View the diff from darker here.
</summary>

``````````diff
--- packages/Python/lldbsuite/test/dotest_args.py	2026-06-03 11:40:45.000000 +0000
+++ packages/Python/lldbsuite/test/dotest_args.py	2026-06-03 11:43:51.497342 +0000
@@ -289,11 +289,11 @@
     )
     group.add_argument(
         "--lldb-python-dir",
         dest="lldb_python_dir",
         metavar="path",
-        help="Path to the directory that contains the 'lldb' Python module. "
+        help="Path to the directory that contains the 'lldb' Python module. ",
     )
 
     # Configuration options
     group = parser.add_argument_group("Remote platform options")
     group.add_argument(
--- test/API/lit.cfg.py	2026-06-03 11:40:45.000000 +0000
+++ test/API/lit.cfg.py	2026-06-03 11:43:51.610894 +0000
@@ -263,13 +263,11 @@
         ).strip()
         for line in version_output.splitlines():
             lit_config.note(line.strip())
     except (subprocess.CalledProcessError, OSError) as e:
         lit_config.warning(
-            "Could not get lldb version from {}: {}".format(
-                config.lldb_executable, e
-            )
+            "Could not get lldb version from {}: {}".format(config.lldb_executable, e)
         )
 
     # Discover the directory that contains the 'lldb' Python module once here,
     # so each dotest invocation doesn't have to spawn '<lldb> -P' itself.
     try:

``````````

</details>


https://github.com/llvm/llvm-project/pull/201327