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 <[email protected]>
> To: [email protected]
>
> 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
_______________________________________________
Kde-buildsystem mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-buildsystem

Reply via email to