Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-11-28 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112785/#review44712 --- This review has been submitted with commit

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-11-25 Thread Chusslove Illich
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112785/#review44453 --- Ship it! I understood that it is important not to have to add

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-11-11 Thread Stephen Kelly
Kevin Ottens wrote: Any chance to see an update or it is abandonned? Regarding what I wrote before, CMake is now (in master) able to extract the uic options from the KF5::KI18n target. The uic executable in Qt 5.3 accepts an -include argument

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-11-10 Thread Kevin Ottens
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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-14 Thread Stephen Kelly
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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-14 Thread Stephen Kelly
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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-14 Thread Stephen Kelly
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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-14 Thread Chusslove Illich
On Sept. 23, 2013, 12:37 p.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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-14 Thread Stephen Kelly
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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-14 Thread Chusslove Illich
On Sept. 23, 2013, 12:37 p.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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-14 Thread Stephen Kelly
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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-09 Thread Kevin Ottens
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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-10-09 Thread Chusslove Illich
On Sept. 23, 2013, 12:37 p.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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-09-24 Thread Kevin Ottens
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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-09-24 Thread Chusslove Illich
On Sept. 23, 2013, 12:37 p.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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-09-24 Thread Jeremy Whiting
On Sept. 23, 2013, 4: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.

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-09-23 Thread Jeremy Whiting
On Sept. 23, 2013, 4:37 a.m., Kevin Ottens wrote: I'm surprised it doesn't use qt5_wrap_ui. It seems to reinvent it at least partly. 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

Re: Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-09-19 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112785/#review40322 --- https://git.reviewboard.kde.org/r/112795/diff/ might provide

Review Request 112785: Add ki18n_wrap_ui macro to ki18nMacros

2013-09-17 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112785/ --- Review request for KDE Frameworks and Alexander Neundorf. Description