← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/glsl1 into lp:widelands

 

Review: Needs Fixing

Ok, it does not work on windows, even with my "fixes" in https://code.launchpad.net/~widelands-dev/widelands/glsl1_win32.

Two issues:
- cmake does find OpenGL and defines OPENGL_FOUND,OPENGL_gl_LIBRARY and OPENGL_glu_LIBRARY but not OPENGL_INCLUDE_DIR and OPENGL_LIBRARIES
- it still does not link : http://pastebin.com/z6hhrwv4

I am not sure if avoiding GLEW and introduce different includes for Linux/Win/MacOS makes maintaining the code easier? Or is GLEW in some way incompatible to the desired OpenGL standard?
-- 
https://code.launchpad.net/~widelands-dev/widelands/glsl1/+merge/241179
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/glsl1.


References