Replace Epoxy with Glad #1766

Merged
merged 9 commits into from Nov 24, 2016

Conversation

Projects
None yet
3 participants
@JeffProgrammer
Contributor

JeffProgrammer commented Sep 12, 2016

Sorry folks. Looks like epoxy is turning out to be a lemon.

  1. epoxy isn't really maintained anymore.
  2. epoxy as we know had trouble with VS2015 and we are thus compiling it with optimizations turned off in release.

And now we get a number 3!

  1. So, I was adding support back into Torque 3D so that you can compile the engine as a DLL using CMake. It turns out epoxy defines its own DLLMain method, making it impossible to statically link into another DLL unless you modify the library itself.

Timmy suggested we use Glad, and we both have used it in the past. It is super simple to use. The glad generator is the OpenGL 4.5 core profile + all of the extensions that are usable in the core profile. WGL and GLX are also added.

If we could have a Linux tester be so kind to let me know if the GLX portion is working (do we even need that since we use SDL) that would be great.

As they say, three strikes and you are out...

@Lopuska

This comment has been minimized.

Show comment
Hide comment
@Lopuska

Lopuska Oct 23, 2016

Member

As long everything work fine is great!
Someone have time to test it quickly for double check?
Also there's a little conflict to resolve.

Member

Lopuska commented Oct 23, 2016

As long everything work fine is great!
Someone have time to test it quickly for double check?
Also there's a little conflict to resolve.

Fix merge conflict and add check for unix and not apple for glx
Merge branch 'development-gg' into gladdev

# Conflicts:
#	Tools/CMake/libraries/glad.cmake
@JeffProgrammer

This comment has been minimized.

Show comment
Hide comment
@JeffProgrammer

JeffProgrammer Nov 3, 2016

Contributor

fixed merge conflicts @Areloch @Lopuska

Contributor

JeffProgrammer commented Nov 3, 2016

fixed merge conflicts @Areloch @Lopuska

@Lopuska

This comment has been minimized.

Show comment
Hide comment
@Lopuska

Lopuska Nov 3, 2016

Member

Awesome. Does someone tested that on Linux?

Member

Lopuska commented Nov 3, 2016

Awesome. Does someone tested that on Linux?

@Areloch

This comment has been minimized.

Show comment
Hide comment
@Areloch

Areloch Nov 4, 2016

Contributor

Looks like the UNIX systems(both lin and mac) have some issues with the cmake config.

I'll have a look into it.

Contributor

Areloch commented Nov 4, 2016

Looks like the UNIX systems(both lin and mac) have some issues with the cmake config.

I'll have a look into it.

@JeffProgrammer

This comment has been minimized.

Show comment
Hide comment
@JeffProgrammer

JeffProgrammer Nov 24, 2016

Contributor

K we should be good now.

Contributor

JeffProgrammer commented Nov 24, 2016

K we should be good now.

@Areloch Areloch merged commit 248a240 into GarageGames:development Nov 24, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment