Am Dienstag, 27. Juli 2010 schrieb Kornel Benko:
> 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
> 

I was too fast. Here the correct patch.
diff -r cd12da20db1a CMakeLists.txt
--- a/CMakeLists.txt	Mon Jul 26 18:30:28 2010 +0200
+++ b/CMakeLists.txt	Tue Jul 27 12:08:05 2010 +0200
@@ -283,14 +283,16 @@
   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)
+
+  FIND_PACKAGE(OpenGL)
+  IF(OPENGL_FOUND)
+    if(APPLE)
+            ADD_DEFINITIONS(-DHAVE_APPLE_OPENGL_FRAMEWORK)
+            include_directories(${OPENGL_INCLUDE_DIR}})
+    endif(APPLE)
+    LIST(APPEND common_libs ${OPENGL_LIBRARIES})
+  ENDIF(OPENGL_FOUND)
+
 
   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