[Lldb-commits] [PATCH] D68442: [lldb] Unifying lldb python path

Haibo Huang via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sat Oct 5 10:00:06 PDT 2019

hhb marked an inline comment as not done.
hhb added inline comments.

Comment at: lldb/CMakeLists.txt:42
+        -c "import distutils.sysconfig, sys; print(distutils.sysconfig.get_python_lib(True, False, sys.argv[1]))"
hhb wrote:
> mgorny wrote:
> > I still like my `(False, False, '')` version better than having to recalculate path afterwards.
> I don't have a strong opinion here. Let's see what labath@ think.
> That been said, I did this because some distribution modified get_python_lib() to return differently based on prefix. One example is Debian/Ubuntu, where 'dist-packages' will be used if prefix is '', '/usr' or '/usr/local'.
> In reality, that only makes difference when CMAKE_INSTALL_PREFIX is set. But I guess it doesn't really matter whether we use 'dist-packages' or 'site-packages' that time, as long as it is consistent everywhere.
Considering DESTINT, maybe empty string is better...

By the way, what's the first parameter plat_specific? In all platforms I have, it doesn't make any difference...

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list