[Bf-committers] [PATCH] cmake build: use find_package() instead of include()

Alexander Neundorf neundorf at kde.org
Sat Jun 27 15:22:18 CEST 2009


Hi,

the attached patch replaces the include() commands for the FindXXX.cmake 
modules with the appropriate find_package(Foo) calls.
This doesn't change the behaviour, but gives more features.
E.g. you could now say
find_package(JPEG REQUIRED)
and cmake will abort with an error if the package is not found.
Also it makes it clearer what is going on.

Additionally the patch removes the line 
 INCLUDE(${CMAKE_ROOT}/Modules/Platform/Windows-cl.cmake)
in the Windows if-branch.
Why was this there ? This file should be included anyway under Windows when 
using the MS compiler.

Alex
-------------- next part --------------
A non-text attachment was scrubbed...
Name: use_find_package_instead_include.patch
Type: text/x-diff
Size: 1890 bytes
Desc: not available
Url : http://lists.blender.org/pipermail/bf-committers/attachments/20090627/a89c6528/attachment.patch 


More information about the Bf-committers mailing list