KDE/kdelibs/cmake/modules
Alexander Neundorf
neundorf at kde.org
Sun Oct 4 17:31:38 CEST 2009
On Saturday 03 October 2009, Andreas Hartmetz wrote:
> SVN commit 1031058 by ahartmetz:
>
> Fix / adapt to FindQt4.cmake changes. Found the error in kdenetwork which
> uses reeeally old ui files.
> CCMAIL: kde-buildsystem at kde.org
> CCMAIL: neundorf at kde.org
>
>
> M +1 -1 KDE4Macros.cmake
>
>
> --- trunk/KDE/kdelibs/cmake/modules/KDE4Macros.cmake #1031057:1031058
> @@ -159,7 +159,7 @@
> #usage: KDE4_ADD_UI3_FILES(foo_SRCS ${ui_files})
> macro (KDE4_ADD_UI3_FILES _sources )
>
> - qt4_get_moc_inc_dirs(_moc_INCS)
> + qt4_get_moc_flags(_moc_INCS)
>
Hmm, what do we do here...
qt4_get_moc_inc_dirs() and also qt4_get_moc_flags() were intended to be
internal macros, i.e. not used outside FindQt4.cmake (that's why they are not
documented).
Now there is actually a file which uses this internal macro.
So, what do we do ?
Live with the changed name and assume that the one change in KDE4Macros.cmake
is the only outside user, and for everybody else it's his own fault if he
uses internal stuff ?
Or should I just add a wrapper macro with the old name, so stuff which uses
this internal macro stays working ?
Alex
More information about the Kde-buildsystem
mailing list