FindLibical.cmake (Fwd: KDE/kdepimlibs/cmake/modules)

Alexander Neundorf neundorf at kde.org
Fri Feb 6 09:11:10 CET 2009


On Friday 06 February 2009, Allen Winter wrote:
> Please review and make sure this patch is correct.
> I think it is.. but I'm not exactly sure.
>
>
> ----------  Forwarded Message  ----------
>
> Subject: KDE/kdepimlibs/cmake/modules
> Date: Thursday 05 February 2009
> From: Romain Pokrzywka <romain at kdab.net>
> To: kde-commits at kde.org
>
> SVN commit 921930 by pokrzywka:
>
> feel free to revert if that breaks elsewhere, but the test program doesn't
> compile without it on windows (bc the include path doesn't include
> libical/)
>
>  M  +1 -1      FindLibical.cmake
>
>
> --- trunk/KDE/kdepimlibs/cmake/modules/FindLibical.cmake #921929:921930
> @@ -47,7 +47,7 @@
>
>  if(LIBICAL_INCLUDE_DIRS AND LIBICAL_LIBRARIES)
>    set(FIND_LIBICAL_VERSION_SOURCE
> -    "#include <ical.h>\n int main()\n {\n
> printf(\"%s\",ICAL_VERSION);return 1;\n }\n") +    "#include
> <libical/ical.h>\n int main()\n {\n printf(\"%s\",ICAL_VERSION);return 1;\n
> }\n") set(FIND_LIBICAL_VERSION_SOURCE_FILE
> ${CMAKE_BINARY_DIR}/CMakeTmp/FindLIBICAL.cxx) file(WRITE
> "${FIND_LIBICAL_VERSION_SOURCE_FILE}" "${FIND_LIBICAL_VERSION_SOURCE}")

From what I heard in the libical discussion, I think it's ok. 
But I also think you know better than me :-)

Alex


More information about the Kde-buildsystem mailing list