This is an archive of the discontinued LLVM Phabricator instance.

lldb: PYTHON_EXECUTABLE should be propagated more carefully
AbandonedPublic

Authored by pawelo on Jun 7 2014, 2:24 PM.

Details

Reviewers
None
Summary

This is one of two problems I encountered when I tried to build and install lldb on Gentoo Linux where default Python interpreter is version 3.3.
To force use of Python 2.7 which lldb requires, one has to type cmake -DPYTHON_EXECUTABLE=/usr/bin/python2.7
Unfortunately, it seems like this option is not propagated properly and 'make install' still uses /usr/bin/python to determine interpreter version.
Proposed patch fixes this weird behaviour.

Diff Detail

Event Timeline

pawelo updated this revision to Diff 10211.Jun 7 2014, 2:24 PM
pawelo retitled this revision from to lldb: PYTHON_EXECUTABLE should be propagated more carefully.
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:37 PM
pawelo changed the visibility from "All Users" to "Public (No Login Required)".Jun 12 2014, 12:33 PM
pawelo abandoned this revision.Jul 2 2014, 8:59 PM

Obsoleted by upstream improvements.