Replace GLEW with libepoxy #256

Merged
merged 2 commits into from Mar 29, 2015

Projects

None yet

5 participants

@linkmauve
Contributor

This removes the dependency on X11, and allows for a much more future-proof OpenGL context, with core, GLES, EGL and everything supported, see https://github.com/anholt/libepoxy#why-not-use-libglew

Also drops the cmake version required back to 2.8.8.

@TheJJ TheJJ commented on the diff Mar 23, 2015
cpp/crossplatform/opengl.h
@@ -1,14 +0,0 @@
-// Copyright 2014-2014 the openage authors. See copying.md for legal info.
-
-#ifndef OPENAGE_CROSSPLATFORM_OPENGL_H_
-#define OPENAGE_CROSSPLATFORM_OPENGL_H_
-
-#include <GL/glew.h>
-
-#ifdef __APPLE__
-# include <OpenGL/gl.h>
-#else
-# include <GL/gl.h>
-#endif
@TheJJ
TheJJ Mar 23, 2015 Member

can anyone try or confirm that this'll still work on osx systems?

@linkmauve
Contributor

For the record, the last missing thing for this PR is a PPA containing libepoxy for Ubuntu 12.04.

@mic-e
Member
mic-e commented Mar 24, 2015

oooh, Wayland support :D

Congratulations on the round issue number!

Travis just got to upgrade their build bots to Ubuntu 14.04 at some point... after all, Ubuntu 12.04 will reach EOL in about a month...

@TheJJ
Member
TheJJ commented Mar 24, 2015

nah, LTS versions got 5 years, two more to go..
i'll see if i can package libepoxy in our ppa.

@mic-e
Member
mic-e commented Mar 24, 2015

5 years support is only for Ubuntu Server LTS. The client stuff has 3 years.

@franciscod
Contributor

holy cow, this looks awesome!!

@mic-e
Member
mic-e commented Mar 26, 2015

libepoxy currently has an unmerged pull request that would supposedly allow cross-compiling to Windows.

@janisozaur
Contributor

(Unfortunately) Travis are using server flavour, so there's no pressure on them so far from this side.
http://docs.travis-ci.com/user/ci-environment/#CI-environment-OS

@TheJJ TheJJ merged commit f505da2 into SFTtech:master Mar 29, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@TheJJ
Member
TheJJ commented Mar 29, 2015

bäm 👍

@mic-e
Member
mic-e commented Mar 29, 2015

wheeee (has any OSX user actually tested this before it was merged, or is it their fault for not reacting to our pleas?)

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