[Lldb-commits] [lldb] [lldb][Python] Include limited API library directory (PR #197185)

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Tue May 12 22:06:13 PDT 2026


https://github.com/JDevlieghere approved this pull request.

LGTM. If we want to avoid duplicating the comment, we can hoist this out of the if to have something like:

```
if (WIN32)
  set(PYTHON_SBIN_LIBRARY_DIR ${Python3_LIBRARY_DIRS})
endif()

and then unconditionally add this:

```
target_link_directories(lldbPluginScriptInterpreterPython PUBLIC ${PYTHON_SBIN_LIBRARY_DIR}
```

Not sure if that works if the variable is empty. If not we can just wrap it in an `if()` again (which makes this suggestion slightly less appealing) 

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


More information about the lldb-commits mailing list