--===============1815009240379597667== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit ----------------------------------------------------------- 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 > > --===============1815009240379597667== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit
This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111371/ |
This review has been submitted with commit f0a932154766cd9014fc4f79cfefcf8b92b82cfc by Luca Beltrame to branch KDE/4.10.
- Commit
On July 2nd, 2013, 6:58 p.m. UTC, Vadim Zhukov wrote:
Review request for kdelibs.
By Vadim Zhukov.
Updated July 2, 2013, 6:58 p.m. Description
Testing
Diffs
|