search

matus-chochlik / oglplus

OGLplus is a collection of open-source, cross-platform libraries which implement an object-oriented facade over the OpenGL® (version 3 and higher) and also OpenAL® (version 1.1) and EGL (version 1.4) C-language APIs. It provides wrappers which automate resource and object management and make the use of these libraries in C++ safer and more convenient.
http://oglplus.org/
Boost Software License 1.0
491 stars 72 forks source link

detect GL version on Windows #61

Closed ucacaxm closed 10 years ago

ucacaxm commented 10 years ago

Hi,

I have just updated and the GL version detection does not work on Windows anymore. I have traced and I arrived to the init_GL.ipp in _build/GL/cmakeFiles/CmakeTmp/... The project does not compile because I think the @GLX_FOUND@ is not replace by 0

if

include <oglplus/glx/context.hpp>

include <oglplus/glx/fb_configs.hpp>

include <oglplus/glx/version.hpp>

include <oglplus/glx/pbuffer.hpp>

include <oglplus/x11/color_map.hpp>

include <oglplus/x11/visual_info.hpp>

include <oglplus/x11/display.hpp>

elif 0

include <GLFW/glfw3.h>

elif 0

include <GL/glfw.h>

elif 1

include <GL/glut.h>

elif 0

include <SDL/SDL.h>

endif

But I do not understand how the config/gl/init_GL.ipp.in is converted to _build/GL/cmakeFiles/CmakeTmp/init_GL.ipp

Thank you for your help, alex

matus-chochlik commented 10 years ago

aww, I probably see the problem: Please try adding:

else()
    set(GLX_FOUND 0)

to config/FindGLX.cmake before the endif() on the last line.

ucacaxm commented 10 years ago

Yes it solves the problem. thanks

2014-04-15 21:28 GMT+02:00 Matus Chochlik notifications@github.com:

aww, I probably see the problem: Please try adding: else() set(GLX_FOUND 0)

to config/FindGLX.cmake before the endif() on the last line.

— Reply to this email directly or view it on GitHubhttps://github.com/matus-chochlik/oglplus/issues/61#issuecomment-40523644 .

matus-chochlik commented 10 years ago

The fix for this was pushed to the current release branch, it will be merged into develop and master after the release cycle, probably tomorrow.