This is an archive of the discontinued LLVM Phabricator instance.

lldb: change lib32 or lib64 to lib on patches returned by get_python_lib()
Needs ReviewPublic

Authored by pawelo on Jun 7 2014, 2:33 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

This is the second problem I encountered when trying to build and install LLDB on my x86_64 gentoo machine. The scripts/lldb_python_module.cmake have .../lib/... part of the path written literally to compose a path to things that should be copied during 'make install'. Unfortunately, at build time these things are placed to a directory composed with the 'lib' part returned by get_python_lib() function. On my 64-bit system it is 'lib64'. This patch uses sed command to get rid of unwanted '64' or '32' part.

Diff Detail

Event Timeline

pawelo updated this revision to Diff 10212.Jun 7 2014, 2:33 PM
pawelo retitled this revision from to lldb: change lib32 or lib64 to lib on patches returned by get_python_lib().
pawelo updated this object.
pawelo edited the test plan for this revision. (Show Details)
pawelo changed the visibility from "Public (No Login Required)" to "All Users".
pawelo added a subscriber: Unknown Object (MLST).Jun 7 2014, 2:36 PM
pawelo changed the visibility from "All Users" to "Public (No Login Required)".Jun 12 2014, 12:34 PM
pawelo added a comment.Jul 2 2014, 1:28 PM

Closing double-quotes in the scripts/Python/build-swig-Python.sh are in wrong places causing syntax error that goes unnoticed silently. The scripts/Python/finish-swig-Python-LLDB.sh file has these things written in correct places.