[Lldb-commits] [lldb] Revert "[lldb] Change Python site-packages path" (PR #207910)

[via lldb-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Nikita Popov (nikic)

<details>
<summary>Changes</summary>

Reverts llvm/llvm-project#<!-- -->207771.

`/usr/lib/pythonN.M/site-packages` is a standard path that everyone uses to install python modules on posix systems. There is no such thing as `/usr/lib/site-packages`. This change breaks lldb packaging for Linux distros.

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


1 Files Affected:

- (modified) lldb/bindings/python/get-python-config.py (+19-7) 


``````````diff
diff --git a/lldb/bindings/python/get-python-config.py b/lldb/bindings/python/get-python-config.py
index b2e54c3ee3773..5575f3a4c90bd 100755
--- a/lldb/bindings/python/get-python-config.py
+++ b/lldb/bindings/python/get-python-config.py
@@ -26,13 +26,25 @@ def main():
         # LLDB_PYTHON_RELATIVE_PATH is the relative path from lldb's prefix
         # to where lldb's python libraries will be installed.
         #
-        # This will always be lib/site-packages (or lib\site-packages).
-        if os.name == "posix":
-            print("lib/site-packages")
-        elif os.name == "nt":
-            print("Lib\\site-packages")
-        else:
-            raise
+        # The way we're going to compute this is to take the relative path from
+        # PYTHON'S prefix to where python libraries are supposed to be
+        # installed.
+        #
+        # The result is if LLDB and python are give the same prefix, then
+        # lldb's python lib will be put in the correct place for python to find it.
+        # If not, you'll have to use lldb -P or lldb -print-script-interpreter-info
+        # to figure out where it is.
+        try:
+            print(relpath_nodots(sysconfig.get_path("platlib"), sys.prefix))
+        except ValueError:
+            # Try to fall back to something reasonable if sysconfig's platlib
+            # is outside of sys.prefix
+            if os.name == "posix":
+                print("lib/python%d.%d/site-packages" % sys.version_info[:2])
+            elif os.name == "nt":
+                print("Lib\\site-packages")
+            else:
+                raise
     elif args.variable_name == "LLDB_PYTHON_EXE_RELATIVE_PATH":
         tried = list()
         exe = sys.executable

``````````

</details>


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