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

Support for OSX #1

Closed
cpwfx opened this issue Dec 20, 2017 · 5 comments
Closed

Support for OSX #1

cpwfx opened this issue Dec 20, 2017 · 5 comments
Assignees
Milestone

Comments

@cpwfx
Copy link

cpwfx commented Dec 20, 2017

I was just checking Nick Lantz's work and bumped in to you version. Nice one!
Running and compiling on OSX will try to load the vertex_src_1_30, fragment_src_1_30 shaders by default and fails since the created OpenGL context is 2.1. With GLFW+Glad it seems to be easier to choose context on OSX, however not sure if GLFW is ported as widely as FreeGlut, so probably no Mobile Intel 965 support there 😄

Freetype.cpp:182

#ifdef __APPLE__
    vertex_src = vertex_src_1_10;
    fragment_src = fragment_src_1_10;
#else
    if (GLEW_VERSION_3_0) {
        vertex_src = vertex_src_1_30;
        fragment_src = fragment_src_1_30;
    } else {
        vertex_src = vertex_src_1_10;
        fragment_src = fragment_src_1_10;
    }
#endif

@Postrediori
Copy link
Owner

Hi! Thank you for your feedback!
From the description it looks like GLEW isn't very great at detecting supported OpenGL versions. I've uploaded the fix that just uses brute force to select the supported version of shaders.
Moving to GLFW+glad is something to be done since the performance is also smoother with Mesa3D drivers;)

@Postrediori Postrediori self-assigned this Dec 22, 2017
@Postrediori
Copy link
Owner

The fix for OSX is now in the master branch;-)

@Postrediori
Copy link
Owner

Added OSX workaround in #2

@cpwfx
Copy link
Author

cpwfx commented Jan 17, 2018

Thank you on behalf of all us OSXians!

@cpwfx cpwfx closed this as completed Jan 17, 2018
@Postrediori
Copy link
Owner

Created issue for new libraries #3

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

No branches or pull requests

2 participants