Review Request 111371: Fix duplicate targets in PythonMacros.cmake, unbreaking Ninja builds for Python-based KDE apps

Vadim Zhukov persgray at gmail.com
Tue Jul 2 19:58:39 BST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111371/
-----------------------------------------------------------

Review request for kdelibs.


Description
-------

Currently, PYTHON_INSTALL macro in PythonMacros.cmake creates same target "compile_python_files" for all Python modules. This somehow works for Make-based builds, but breaks when using Ninja backend for CMake. CMake itself says that it does not guarantee that targets with same name will ever work.

The patch creates unique target for each python module being pre-compiled and installed.

The patch can be applied without modifications to 4.10 and 4.11, too.


Diffs
-----

  cmake/modules/PythonMacros.cmake 95c9315 

Diff: http://git.reviewboard.kde.org/r/111371/diff/


Testing
-------

OpenBSD-CURRENT, PyKDE 4.10.4 and 4.10.5.


Thanks,

Vadim Zhukov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130702/769df9f7/attachment.htm>


More information about the kde-core-devel mailing list