----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111371/#review35551 -----------------------------------------------------------
Ship it! Ship It! - Luca Beltrame 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 > >