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. =]
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
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).