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

fix conflicting declaration OpenGL on some linux distros #6187

Merged
merged 5 commits into from
Nov 26, 2018

Conversation

icq4ever
Copy link
Contributor

@@ -209,7 +209,7 @@ enum ofTargetPlatform{
#define GL_GLEXT_PROTOTYPES
#include <GL/glew.h>
#include <GL/gl.h>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you comment this line as well? also for osx and windows can you comment out the includes for GL/* except glew to see what happens?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comment line :211 also no issue.
I'm not sure on other OS (osx/windows). but I'll test and post here later.

Copy link
Contributor Author

@icq4ever icq4ever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove include GL/gl.h

Copy link
Contributor Author

@icq4ever icq4ever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comment out GL_GLEXT_PROTOTYPES define.
no issue on linux

no issue on MacOS 10.14.1 Mojave
@arturoc
Copy link
Member

arturoc commented Nov 23, 2018

hey thanks, you can just comment the rest of the platforms out (only desktop though) and the ci servers we'll test if everything keeps compiling or not

@icq4ever
Copy link
Contributor Author

cool. I cannot find proper line for windows. I think it's done.

@arturoc arturoc merged commit c23f117 into openframeworks:patch-release Nov 26, 2018
@arturoc
Copy link
Member

arturoc commented Nov 26, 2018

thanks!

@sphaero
Copy link
Contributor

sphaero commented Nov 29, 2018

can this get into master as well?

@arturoc
Copy link
Member

arturoc commented Nov 29, 2018

it is already

arturoc pushed a commit that referenced this pull request Dec 21, 2018
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