kjs/CMakeLists.txt change on KDE-4.0 branch breaks build -- apparent cmake bug?

[Andreas Pakulat]

On 14.01.08 11:11:33, Maksim Orlovich wrote:
> > Hi,
> > current KDE-4.0 branch doesn't build here after the recent
> > kjs/CMakeLists.txt
> > change:
> 
> <snip>
> 
>  /usr/src/ark/BUILD/kdelibs/build/CMakeFiles/CMakeTmp/CheckSymbolExists.c:2:18:
> > error: pcre.h: No such file or directory
> 
> Sorry for the trouble... Does the attached help?

> Index: CMakeLists.txt
> ===================================================================
> --- CMakeLists.txt	(revision 760932)
> +++ CMakeLists.txt	(working copy)
> @@ -42,9 +42,17 @@
>  # the check for pcre is in kdelibs/CMakeLists.txt
>  if(PCRE_FOUND)
>     include_directories(${PCRE_INCLUDE_DIR})
> +   
> +   # tell check_symbol_exists to -I pcre dirs.
> +   # (this the required_includes stuff is from FindQt4.cmake)
> +   SET(CMAKE_REQUIRED_INCLUDES_SAVE ${CMAKE_REQUIRED_INCLUDES})
> +   SET(CMAKE_REQUIRED_INCLUDES "${CMAKE_REQUIRED_INCLUDES};${PCRE_INCLUDE_DIR}")

There's a macro for pushing and popping all those required-vars, check
the top of the kdelibs/kjs/CMakeLists.txt 

Andreas

-- 
Tuesday is the Wednesday of the rest of your life.