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

SDL "default" fails to compile with Mesa Master. #3394

Closed
SDLBugzilla opened this issue Feb 11, 2021 · 0 comments
Closed

SDL "default" fails to compile with Mesa Master. #3394

SDLBugzilla opened this issue Feb 11, 2021 · 0 comments

Comments

@SDLBugzilla
Copy link
Collaborator

This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

Reported in version: HG 2.1
Reported for operating system, platform: Linux, x86

Comments on the original bug report:

On 2019-09-09 20:55:54 +0000, wrote:

I think something in their headers changed in June. Compiling against a Mesa master I get

CC build/SDL_video.lo
In file included from include/SDL_opengles.h:33:0,
from /srcbuild/SDL/src/video/SDL_video.c:41:
/opt/include/GLES/gl.h:63:25: error: conflicting types for 'GLsizeiptr'
typedef khronos_ssize_t GLsizeiptr;
^~~~~~~~~~
In file included from include/SDL_opengl.h:2105:0,
from /srcbuild/SDL/src/video/SDL_video.c:37:
include/SDL_opengl_glext.h:472:19: note: previous declaration of 'GLsizeiptr' was here
typedef ptrdiff_t GLsizeiptr;
^~~~~~~~~~
In file included from include/SDL_opengles.h:33:0,
from /srcbuild/SDL/src/video/SDL_video.c:41:
/opt/include/GLES/gl.h:64:26: error: conflicting types for 'GLintptr'
typedef khronos_intptr_t GLintptr;
^~~~~~~~
In file included from include/SDL_opengl.h:2105:0,
from /srcbuild/SDL/src/video/SDL_video.c:37:
include/SDL_opengl_glext.h:473:19: note: previous declaration of 'GLintptr' was here
typedef ptrdiff_t GLintptr;
^~~~~~~~
CC build/SDL_vulkan_utils.lo
make: *** [Makefile:552: build/SDL_video.lo] Error 1
make: *** Waiting for unfinished jobs....
nerdopolis@nerdopolis:~$

Thanks

On 2019-09-11 02:48:15 +0000, wrote:

build args:
./configure --prefix=$INSTALLDIR --libdir=$INSTALLDIR/lib/$(dpkg-architecture -qDEB_HOST_MULTIARCH) --enable-wayland-shared=no

config.status: executing summary commands
SDL2 Configure Summary:
Building Shared Libraries
Building Static Libraries
Enabled modules : atomic audio video render events joystick haptic sensor power filesystem threads timers file loadso cpuinfo assembly
Assembly Math : mmx 3dnow sse
Audio drivers : disk dummy oss alsa(dynamic) pulse(dynamic)
Video drivers : dummy x11(dynamic) opengl opengl_es1 opengl_es2 vulkan wayland
X11 libraries : xcursor xdbe xinerama xinput2 xinput2_multitouch xrandr xscrnsaver xshape xvidmode
Input drivers : linuxev linuxkd
Using libsamplerate : NO
Using libudev : YES
Using dbus : YES
Using ime : YES
Using ibus : YES
Using fcitx : NO
PackagePrepareBuild finished in 9 seconds

On 2019-09-20 02:44:35 +0000, wrote:

I also forgot to add, this is 32 bit

On 2019-09-20 20:47:37 +0000, Ryan C. Gordon wrote:

We're changing how we do SDL release versions; now releases will be even numbers (2.0.10, 2.0.12, etc), and as soon as we tag a release, we'll move the internal version number to an odd number (2.0.12 ships, we tag the latest in revision control as 2.0.13 immediately, which will become 2.0.14 on release, etc).

As such, I'm moving the bugs tagged with target-2.0.11 to target 2.0.12. Sorry if you get a lot of email from this change!

Thanks,
--ryan.

On 2019-09-20 20:48:40 +0000, Ryan C. Gordon wrote:

We're changing how we do SDL release versions; now releases will be even numbers (2.0.10, 2.0.12, etc), and as soon as we tag a release, we'll move the internal version number to an odd number (2.0.12 ships, we tag the latest in revision control as 2.0.13 immediately, which will become 2.0.14 on release, etc).

As such, I'm moving the bugs tagged with target-2.0.11 to target 2.0.12. Sorry if you get a lot of email from this change!

Thanks,
--ryan.

On 2019-10-10 16:37:58 +0000, Michael Olbrich wrote:

Created attachment 3981
Patch to fix the problem

This should fix the build error. The same thing happend with ES2 years ago. It just accidentally worked for ES1 until Mesa changed the header guard.

On 2019-10-11 03:53:27 +0000, wrote:

I forgot to update the thread a while back. I actually guessed, and gave something very similar a try and it worked, but I assumed I only cobbled up a workaround, instead of an actual fix. I guess it's good to merge

On 2019-10-12 16:50:28 +0000, Sylvain wrote:

Thanks! fixed in https://hg.libsdl.org/SDL/rev/369b01006eb2

For Open GLES 2, that was fixed in https://hg.libsdl.org/SDL/rev/a60b3c292f0f

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

No branches or pull requests

1 participant