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

Disable glad on apple #2009

Closed
wants to merge 1 commit into from
Closed

Disable glad on apple #2009

wants to merge 1 commit into from

Conversation

alecjacobson
Copy link
Contributor

Fixes #751.

We should either merge this or a libtool/cmake flag fix discussed on #751 as a stopgap. Especially if we can't identify any need for glad on Apple. Unless a need for glad on Apple is raised, I'm inclined to avoid compilation flags.

(So far, this PR doesn't remove glad from the cmake fetching and building on mac.)

(Might also fix libigl/libigl-example-project#33 but not tested)

@jdumas
Copy link
Collaborator

jdumas commented Mar 25, 2022

I am not too keen on removing glad altogether for macOS. I have a counter-pull-request #2012 that uses bindings generated with glad2 instead of glad, and they don't seem to suffer from the issue described in #751 (which I've been able to reproduce locally on my machine).

@alecjacobson
Copy link
Contributor Author

opting for #2012 instead

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