Skip to content
This repository has been archived by the owner on Feb 11, 2021. It is now read-only.

Commit

Permalink
Fixed bug 1782 - SDL_opengl.h header disabled on FreeBSD
Browse files Browse the repository at this point in the history
q66

The SDL_opengl.h header contains this:

#ifdef __FreeBSD__  /* !!! FIXME: temp compiler warning fix... */
#define NO_SDL_GLEXT 1
#endif

However, I can't seem to find what kind of compiler warning it was and it makes it unusable to use on FreeBSD. If I comment out these lines on my machine, everything works fine - I use FreeBSD 9-STABLE (x86_64, gcc and clang both, the same in a x86 chroot). All I could find is that this was causing an error on FreeBSD 8, but I can't test that on my machine (maybe if I set up some FreeBSD 8 chroot).

I set up a 8.2 chroot and investigated the problem. Apparently this issue was fixed in Mesa 7.6 (and in Git, June 4 2009, but it didn't get into 7.5). By the time those lines were added, FreeBSD contained the libGL port version 7.4.4, which suffered from the issue, but on April 2012 the version was updated to 7.6, which is available for FreeBSD 8 and FreeBSD 9 alike, which means those three lines should be safe to remove (it'll work fine for everyone with sufficiently up to date ports).
  • Loading branch information
slouken committed Apr 2, 2013
1 parent 05a645d commit 15fa241
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions include/SDL_opengl.h
Expand Up @@ -44,10 +44,6 @@
#define NO_SDL_GLEXT 1
#endif

#ifdef __FreeBSD__ /* !!! FIXME: temp compiler warning fix... */
#define NO_SDL_GLEXT 1
#endif

#ifdef __glext_h_
/* Someone has already included glext.h */
#define NO_SDL_GLEXT
Expand Down

0 comments on commit 15fa241

Please sign in to comment.