<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 18, 2013 at 9:10 PM, Chusslove Illich <span dir="ltr"><<a href="mailto:caslav.ilic@gmx.net" target="_blank">caslav.ilic@gmx.net</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>> [: Kevin Ottens :]<br>
<div class="im">>> Of course it should be removed when we get a proper fix via CMake 3<br>
>> around. But in the meantime it'll do the trick and allow removing<br>
>> dependencies on KDE4Support just for that.<br>
><br>
</div>> [: Aleix Pol :]<br>
<div class="im">> +1<br>
><br>
> Then I suggest to let this go in:<br>
> <a href="https://git.reviewboard.kde.org/r/112785/diff/#index_header" target="_blank">https://git.reviewboard.kde.org/r/112785/diff/#index_header</a><br>
<br>
</div>I don't see much point in that. To simply go on with the things, as I<br>
suggested, just replace every kde4_add_ui_files with qt5_wrap_ui. Later all<br>
these calls must be revisited anyway, to set translation domain and semantic<br>
markup flag. This holds either way; and with Stephen's solution, on<br>
revisiting only some new lines would be added, and qt5_wrap_ui calls left as<br>
they are.<br>
<br>
Alternatively, if one worries that with this things might be forgotten later<br>
(as Jeremy did originally), then just add KI18NMacros.cmake with a three-<br>
line ki18n_wrap_ui macro that passes the files to qt_wrap_ui.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Chusslove Illich (Часлав Илић)<br>
</div></div><br>_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
<br></blockquote></div><br></div><div class="gmail_extra">That's not true. If you use qt5_wrap_ui nothing compiles because most of our code will expect KLocalizedString to be included. I don't think adding the includes is a good fix.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Aleix </div></div>