> On Sept. 23, 2013, 10:37 a.m., Kevin Ottens wrote: > > I'm surprised it doesn't use qt5_wrap_ui. It seems to reinvent it at least > > partly. > > Jeremy Whiting wrote: > well, qt5_wrap_ui wasn't around when this was created (as > kde4_add_ui_files iirc). All I did was copy it and rename it. didn't look > into making it use qt5_wrap_ui.
Could you please look into it? - Kevin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112785/#review40518 ----------------------------------------------------------- On Sept. 17, 2013, 7:56 p.m., Jeremy Whiting wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112785/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2013, 7:56 p.m.) > > > Review request for KDE Frameworks and Alexander Neundorf. > > > Description > ------- > > It builds and installs, but doesn't seem to be usable from within kdelibs, > i.e. ki18n_wrap_ui in knewstuff/src/CMakeLists.txt fails with this. I suspect > one more thing is needed to make it work from within kdelibs builds. > > > Diffs > ----- > > tier2/ki18n/CMakeLists.txt d0ed448 > tier2/ki18n/KI18nConfig.cmake.in 18b6e2f > tier2/ki18n/cmake/KI18NMacros.cmake PRE-CREATION > tier2/ki18n/cmake/ki18nuic.cmake PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/112785/diff/ > > > Testing > ------- > > Builds and installs into PREFIX/lib64/cmake/KI18N next to KI18nConfig.cmake > > > Thanks, > > Jeremy Whiting > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
