----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119198/#review62214 -----------------------------------------------------------
Ship it! Yep, seems sensible to me. We definitely want the lconvert that matches the lrelease we're using. - Alex Merry On July 9, 2014, 2:12 p.m., Michael Palimaka wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119198/ > ----------------------------------------------------------- > > (Updated July 9, 2014, 2:12 p.m.) > > > Review request for Build System, Extra Cmake Modules and KDE Frameworks. > > > Repository: extra-cmake-modules > > > Description > ------- > > Otherwise, if lconvert exists in normal system paths (eg. /usr/bin) that one > will be used instead of the one alongside Qt5::lrelease. This could cause Qt4 > lconvert to be incorrectly used on some systems. > > > Diffs > ----- > > modules/ECMPoQmTools.cmake 3ce695817cb3da5ec9eebec86e632438c5941ee6 > modules/ECMCreateQmFromPoFiles.cmake > 4a31a93e7900780dc5b9424b148f19f5c22061af > > Diff: https://git.reviewboard.kde.org/r/119198/diff/ > > > Testing > ------- > > When building solid-5.0.0, Qt5 lconvert from lrelease_path is now correctly > used. Previously Qt4 version was used as it exists in /usr/bin. > > > Thanks, > > Michael Palimaka > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel