trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: November 2012

Re: [trinity-devel] cmake bug with 64-bit?

From: Darrell Anderson <humanreadable@...>
Date: Sun, 11 Nov 2012 15:48:49 -0800 (PST)
> > As I said, correct version is:
> > 
> > tde_save_and_set( CMAKE_REQUIRED_INCLUDES
> ${FRIBIDI_INCLUDE_DIRS} )
> > check_include_file( fribidi/fribidi-types.h
> FRIBIDI_NEW_FILENAME )
> > tde_restore( CMAKE_REQUIRED_INCLUDES  )
> 
> Okay, thank you.
> 
> If I understand correctly, the correct patch should look
> like this:
> 
> diff -urN tdegraphics/ksvg/ConfigureChecks.cmake tdegraphics.new/ksvg/ConfigureChecks.cmake
> --- tdegraphics/ksvg/ConfigureChecks.cmake 2012-09-14 17:59:55.000000000 -0500
> +++ tdegraphics.new/ksvg/ConfigureChecks.cmake 2012-11-11 15:17:24.000000000 -0600
> @@ -41,7 +41,9 @@
>  # fribidi
>  pkg_search_module( FRIBIDI fribidi )
>  if( FRIBIDI_FOUND )
> +  tde_save_and_set( CMAKE_REQUIRED_INCLUDES ${FRIBIDI_INCLUDE_DIRS} )
>    check_include_file( fribidi/fribidi-types.h FRIBIDI_NEW_FILENAME )
> +  tde_restore( CMAKE_REQUIRED_INCLUDES  )
>  else( )
>    tde_message_fatal( "fribidi is required, but was not found on your system" )
>  endif( )
> 
> I will test again.

The updated patch worked.

Darrell