bruns created this revision. bruns added a reviewer: Frameworks. Herald added projects: Frameworks, Build System. Herald added subscribers: kde-buildsystem, kde-frameworks-devel. bruns requested review of this revision.
REVISION SUMMARY The correct install directory is distro and arch specific, and should match the configuration of the python installation the binding is generated for. The directories can be queried using pythons distutils.sysconfig. In the unlikely case this does not match expectations, the directory can be overridden using -DGPB_PYTHON{2,3}_SITEARCH="<dir>". TEST PLAN On a distro where sitearch is not below /usr/lib/pythonM.m/, but /usr/lib64/pythonM.m/ (e.g. RH, SUSE 64bit), try to do: cmake ..; make; install Without the patch, the binding are installed into the wrong directory, afterwards the correct path is used. This should also yield the correct path on Debian and derivatives, where dist-packages instead of site-packages is used (untested). REPOSITORY R240 Extra CMake Modules BRANCH python3_support REVISION DETAIL https://phabricator.kde.org/D15070 AFFECTED FILES find-modules/FindPythonModuleGeneration.cmake To: bruns, #frameworks Cc: kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns