This is an archive of the discontinued LLVM Phabricator instance.

Switch python install to use an install(DIRECTORY ...) cmake command rather than a cmake script
ClosedPublic

Authored by chandlerc on Jun 15 2014, 5:20 PM.

Details

Reviewers
tfiala
Summary

In addition to cleaning things up, this allows more easy access to the variables. In the old version, it tried to pass variables as -D flags to cmake, but this didn't actually work. CMake drops all of those arguments on the floor (try passing garbage through them) and just picks up the limited subset of pre-defined macros. So, for example, this fixes the build with LLVM_LIBDIR_SUFFIX=64 which is how I ended up here. =]

Diff Detail

Event Timeline

chandlerc updated this revision to Diff 10433.Jun 15 2014, 5:20 PM
chandlerc retitled this revision from to Switch python install to use an install(DIRECTORY ...) cmake command rather than a cmake script.
chandlerc updated this object.
chandlerc edited the test plan for this revision. (Show Details)
chandlerc added a subscriber: Unknown Object (MLST).
tfiala added a subscriber: tfiala.Jun 15 2014, 11:15 PM

Trying this out now.

tfiala accepted this revision.Jun 15 2014, 11:43 PM
tfiala added a reviewer: tfiala.

LGTM.

Tested on Ubuntu 12.04 gcc 4.8.2 with stock system python 2.7 and with a custom debug python 2.7. Both configured and built properly (with LD_LIBRARY_PATH and PATH adjusted for the custom python).

This revision is now accepted and ready to land.Jun 15 2014, 11:43 PM
chandlerc closed this revision.Jun 16 2014, 8:10 AM

Thanks, submitted in r211028.