KDE/kdelibs/cmake/modules
Alexander Neundorf
neundorf at kde.org
Mon Mar 9 23:48:57 CET 2009
On Wednesday 04 March 2009, David Faure wrote:
> SVN commit 935251 by dfaure:
>
> Support for .kcfg files being generated using configure_file like tmcguire
> wants to do in kmail.
> CCMAIL: kde-buildsystem at kde.org
>
>
> M +10 -2 KDE4Macros.cmake
>
>
> --- trunk/KDE/kdelibs/cmake/modules/KDE4Macros.cmake #935250:935251
> @@ -70,13 +70,21 @@
> set(_src_FILE ${CMAKE_CURRENT_BINARY_DIR}/${_basename}.cpp)
> set(_header_FILE ${CMAKE_CURRENT_BINARY_DIR}/${_basename}.h)
> set(_moc_FILE ${CMAKE_CURRENT_BINARY_DIR}/${_basename}.moc)
> + set(_kcfg_PATH ${_abs_PATH}/${_kcfg_FILE})
I think this variable should end with "_FILE", not with "_PATH", since in this
macro all other variables which end in "_PATH" are only a directory, while
all other variables which are a file end in "_FILE".
Beside that I think it looks good.
Alex
More information about the Kde-buildsystem
mailing list