>> [: Kevin Ottens :]
>> Of course it should be removed when we get a proper fix via CMake 3
>> around. But in the meantime it'll do the trick and allow removing
>> dependencies on KDE4Support just for that.
>
> [: Aleix Pol :]
> +1
>
> Then I suggest to let this go in:
> https://git.reviewboard.kde.org/r/112785/diff/#index_header

I don't see much point in that. To simply go on with the things, as I
suggested, just replace every kde4_add_ui_files with qt5_wrap_ui. Later all
these calls must be revisited anyway, to set translation domain and semantic
markup flag. This holds either way; and with Stephen's solution, on
revisiting only some new lines would be added, and qt5_wrap_ui calls left as
they are.

Alternatively, if one worries that with this things might be forgotten later
(as Jeremy did originally), then just add KI18NMacros.cmake with a three-
line ki18n_wrap_ui macro that passes the files to qt_wrap_ui.

-- 
Chusslove Illich (Часлав Илић)

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to