[Lldb-commits] [lldb] Revert "[lldb] Change Python site-packages path" (PR #207910)
llvmorg-github-actions[bot] wrote:
@llvm/pr-subscribers-lldb
Author: Nikita Popov (nikic)
Changes
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
``
https://github.com/llvm/llvm-project/pull/207910
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
[Lldb-commits] [lldb] Revert "[lldb] Change Python site-packages path" (PR #207910)
https://github.com/nikic created
https://github.com/llvm/llvm-project/pull/207910
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.
>From 60b7a2dbadbd22e9dfc428d887c42fb37eb534fd Mon Sep 17 00:00:00 2001
From: Nikita Popov
Date: Tue, 7 Jul 2026 09:05:58 +0200
Subject: [PATCH] Revert "[lldb] Change Python site-packages path (#207771)"
This reverts commit 8475e376043ddacc53b7062c71438ea4e2a082e0.
---
lldb/bindings/python/get-python-config.py | 26 +--
1 file changed, 19 insertions(+), 7 deletions(-)
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
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
