[Bug]: Q3VR doesn't work with Wayland on Linux

[RippeR37]

Operating System

Linux

VR Runtime

Other (please specify below)

Other VR Runtime

N/A

GPU

N/A

VR Headset

N/A

VR Controllers

N/A

Problem description

Game doesn't start when using Wayland on Linux.

Note

This is reported based on feedback in Discord.

Steps to reproduce

1. Set up your Linux distro to use Wayland
2. Launch the game
3. Observe fail to launch (fail to create XR session)

Note

It is possible to workaround this issue by forcing SDL to use X11 instead of Wayland and launching the game with with SDL_VIDEODRIVER=x11 ./q3vr.

Game version / build

v1.0

[RippeR37]

This is related to the fact that currently VR sessions support only Win32 and X11. The implementation for other would have to be added in:

• code/vr/vr_types.h
• code/vr/vr_session.c

It's open question whether it is possible to build a single binary with support for both or if it would need to be built separately - most likely doable in a single binary but may need a bit of refactoring to ensure that X11/Wayland/... symbols don't escape too widely outside of the code that's intended to use them.

On OpenXR side there is of course XR_USE_PLATFORM_WAYLAND and XrGraphicsBindingOpenGLWaylandKHR to be used there.

---

Edit: detection of current underlying display system (x11/wayland) should be doable with SDL_GetCurrentVideoDriver(), or even better yet something like:

#include <SDL2/SDL_syswm.h>

SDL_SysWMinfo info;
SDL_VERSION(&info.version);

if (SDL_GetWindowWMInfo(window, &info)) {
    if (info.subsystem == SDL_SYSWM_X11) {
        // X11
    } else if (info.subsystem == SDL_SYSWM_WAYLAND) {
        // Wayland
    }
}

which directly provides needed wl_display* and is already used there. For clarity I think we should move the platform-specific logic to separate translation units (.c files) and include platform headers only in those.

---

We should also update documentation and state clearly what's supported.