Am Montag, 26. Juli 2010 schrieb Kornel Benko:
> To my big surprise the compilation did not get past the link stage. 
> Unsatisfied extternal gluErrorString.
> I had to add the library libGLU to common libs.
> 
> The next hurdle was the documentation. Here the file color-profiles.texi was 
> not found.
> The following patch cured it for me, but at least the line
>    FIND_PACKAGE(GLU REQUIRED)
> looks suspicious, since nobody else had it. I cannot check on other systems 
> at the moment.
> 
> I will check later on ubuntu.
> 
>       Kornel
> 

Hmm, not needed on ubuntu, but harmless. OTOH there is always a hack for apple
with
        FIND_PACKAGE(OpenGL)

This would be ok for SuSE too. The only nuisance is, on SuSE it should be
       FIND_PACKAGE(OpenGL REQUIRED)
but this is less annoying than not having it at all.

I will commit appropriate (attached) change for cmake, if nobody screams.

        Kornel
diff -r cd12da20db1a CMakeLists.txt
--- a/CMakeLists.txt	Mon Jul 26 18:30:28 2010 +0200
+++ b/CMakeLists.txt	Tue Jul 27 11:52:27 2010 +0200
@@ -283,14 +283,14 @@
   ENDIF(WIN32)
   FIND_PACKAGE(GLUT REQUIRED)
   FIND_PACKAGE(GLEW REQUIRED)
-  IF(APPLE)
+
     FIND_PACKAGE(OpenGL)
 	IF(OPENGL_FOUND)
 	  ADD_DEFINITIONS(-DHAVE_APPLE_OPENGL_FRAMEWORK)
 	  include_directories(${OPENGL_INCLUDE_DIR}})
 	  LIST(APPEND common_libs ${OPENGL_LIBRARIES})
 	ENDIF(OPENGL_FOUND)
-  ENDIF(APPLE)
+
 
   ADD_DEFINITIONS(-DHAVE_LIBGLEW)
   include_directories(${GLEW_INCLUDE_DIR} ${GLUT_INCLUDE_DIR})

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to