Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added compatibility with GLFW3 #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added compatibility with GLFW3 #5

wants to merge 2 commits into from

Conversation

paperManu
Copy link

This patch allows the use of GLV with GLFW, by setting the flag GLV_GLFW.

@LancePutnam
Copy link
Member

Do you know if this patch also works for OSX and Windows? Otherwise, it's fine as Linux-only for now.

@paperManu
Copy link
Author

I didn't have the opportunity to test it on anything but Linux, though I guess it should work on OSX too.

@DrewRWx
Copy link
Member

DrewRWx commented Jun 8, 2014

I did a brief test on OS X 10.9 by modifying the OS X elif clause like the Linux one. There's an issue with GLV_PLATFORM_INIT_CONTEXT not being defined, but commenting out that function call allows the example to compile and run, albeit without the screen tearing prevention that the #define provided.

Unless I'm missing something, it looks like this commit allows including GLFW headers, which also wrap including OpenGL heads, but doesn't make use of them and instead still uses GLUT.

@paperManu
Copy link
Author

Should this be ported to Windows before merging?

@DrewRWx
Copy link
Member

DrewRWx commented Sep 17, 2014

Yeah, Windows support is a priority for one of our architects.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants