[cmake-developers] [CMake] CMake 2.8.3-rc1 ready for testing!

Alexander Neundorf neundorf at kde.org
Sun Sep 26 03:57:14 EDT 2010


On Saturday 25 September 2010, Eric Noulard wrote:
> 2010/9/25 Alexander Neundorf <neundorf at kde.org>:
> [..]
>
> > The other option would be to make sure that
> > INCLUDE(FindPackageHandleStandardArgs)
> > when used in cmake's own module would always load
> > FindPackageHandleStandardArgs.cmake from cmake, i.e. code like:
> >
> > GET_FILENAME_COMPONENT(_currentDir "${CMAKE_CURRENT_LIST_FILE}" PATH)
> > INCLUDE("${_currentDir}/FindPackageHandleStandardArgs.cmake")
> >
> >
> > Should all
> > INCLUDE(FindPackageHandleStandardArgs)
> > in all find-modules from cmake be changed to this ?
>
> I don't think it's a good idea.
> People may purposely want to override CMake provided module
> in order to implement specific behavior and/or to patch locally.

This is what we do in KDE, and I think it would be good for us.

Using 
include(FPHSA CURRENT_DIR)
in the files in cmake would mean that the find-modules in cmake also get the 
FPHSA they expect to get, and if we would use that in KDE too, we would be 
sure we would get our modified version.

Alex



More information about the cmake-developers mailing list