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 13:21:12 -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.

Darrell