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

Commit Hook null at kde.org
Thu Jul 4 22:33:31 BST 2013


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


This review has been submitted with commit f0a932154766cd9014fc4f79cfefcf8b92b82cfc by Luca Beltrame to branch KDE/4.10.

- Commit Hook


On July 2, 2013, 6:58 p.m., Vadim Zhukov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111371/
> -----------------------------------------------------------
> 
> (Updated July 2, 2013, 6:58 p.m.)
> 
> 
> 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/20130704/81800302/attachment.htm>


More information about the kde-core-devel mailing list