[Lldb-commits] [PATCH] D19067: Make sure to use lib instead of lib64 for LLDB_LIB_DIR

[Pavel Labath via lldb-commits]

labath added a comment.

In http://reviews.llvm.org/D19067#401416, @fjricci wrote:

> When I use LLVM_LIBDIR_SUFFIX=64, it looks like the python _lldb.so symlink still points to build/lib/liblldb.so, instead of using the lib64 directory. So if I do this, none of the tests will run on the check-lldb target, since importing lldb in python fails.
>
> Perhaps this symlink is the bug that should be fixed? However, it appears that finish-swig-Python-LLDB.sh already has the correct logic for this, so it's not immediately clear to me why the symlink is incorrect (if this seems like a bug we want fixed, I'll investigate):
>
>   ln -s "../../../liblldb${SOEXT}" _lldb.so
>


Yes, I believe this would be a better fix for your problem.

Note that `finish-swig-Python-LLDB.sh` is not used anymore in the cmake build. It's replacement is `finishSwigPythonLLDB.py` so you'll need to look for the problem there.


http://reviews.llvm.org/D19067