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