Your message dated Fri, 26 Dec 2025 05:52:51 +0000
with message-id <[email protected]>
and subject line Bug#1123731: fixed in wesnoth-1.18 1:1.18.6-1
has caused the Debian Bug report #1123731,
regarding wesnoth-1.18: autopkgtest fails with sdl2-compat
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
1123731: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1123731
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Source: wesnoth-1.18
Severity: normal
Tags: patch
User: [email protected]
Usertags: regression

After discussion with SDL upstream, I'm looking into what needs to 
happen for Debian (and indirectly Ubuntu) to replace "classic" SDL2 with 
sdl2-compat, following in the footsteps of other distros like Arch and 
Fedora. sdl2-compat is a reimplementation of the SDL 2 API using SDL 3.

One issue that I've noticed is that the autopkgtest for wesnoth-1.18 has 
regressed. This seems to be because "classic" SDL2 will automatically 
use the `offscreen` video driver if neither X11 nor Wayland works, but 
there was an upstream change in SDL3 to make the `offscreen` video 
driver only work if explicitly requested, similar to the behaviour that 
the `dummy` video driver already had. The result is that on a CI system 
with no X11 display or Wayland compositor, this test fails to initialize 
SDL video when SDL is really sdl2-compat.

The upstream change seems reasonable to me - by default, games should 
only use video drivers that actually display some pixels to the user! - 
but I've opened https://github.com/libsdl-org/sdl2-compat/issues/561 to 
query whether sdl2-compat should be bug-for-bug compatible with 
"classic" SDL2 on this point. So I think this is really an issue in 
wesnoth-1.18, more than an issue in sdl2-compat.

A workaround that I have confirmed does work is to run the test with 
SDL_VIDEODRIVER="offscreen" (or "dummy" or "offscreen,dummy"), so in the 
short term please consider applying the attached patch.

The xauth and xvfb dependencies can probably also be dropped, although I 
haven't verified that.

In the longer term, for consideration upstream:

wesnoth --nogui still calls SDL_InitSubSystem(SDL_INIT_VIDEO), even 
though it has been asked not to have a GUI. Perhaps ideally it wouldn't? 
(But perhaps there's some feature of the video subsystem that it is 
relying on?)

Or perhaps wesnoth could call SDL_SetHint(SDL_HINT_VIDEODRIVER, "offscreen")
(or "dummy", or "offscreen,dummy") when invoked with --nogui or equivalent?

Thanks,
    smcv
>From 2292879cf54cb8b9ef33f4fd0fd85f105faffebc Mon Sep 17 00:00:00 2001
From: Simon McVittie <[email protected]>
Date: Sat, 20 Dec 2025 16:17:18 +0000
Subject: [PATCH] d/tests/control.in: Explicitly use offscreen or dummy video
 driver

In a CI environment we can't expect to have a working X11 display or
Wayland compositor unless we explicitly run one, which this test
does not. In "classic" SDL2 this was masked by the offscreen video
driver being enabled by default, but in SDL3 + sdl2-compat the offscreen
video driver is only used if it is specifically requested, with the
reasonable justification that video drivers that don't show any pixels
to the user are only rarely what is wanted.

Closes: #-1
---
 debian/tests/control    | 2 +-
 debian/tests/control.in | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/debian/tests/control b/debian/tests/control
index 12a7dad6ad5..64879f04f16 100644
--- a/debian/tests/control
+++ b/debian/tests/control
@@ -1,6 +1,6 @@
 # Inspired by upstream CI script utils/CI/play_test_executor.sh
 # This runs wesnoth with a multiplayer AI-only matchup.
-Test-Command: /usr/games/wesnoth-1.18 -m --controller 1:ai --controller 2:ai --nogui
+Test-Command: SDL_VIDEODRIVER=offscreen,dummy /usr/games/wesnoth-1.18 -m --controller 1:ai --controller 2:ai --nogui
 Depends:
  wesnoth-1.18,
  xauth,
diff --git a/debian/tests/control.in b/debian/tests/control.in
index 582ad1ce727..3929ef24c5f 100644
--- a/debian/tests/control.in
+++ b/debian/tests/control.in
@@ -1,6 +1,6 @@
 # Inspired by upstream CI script utils/CI/play_test_executor.sh
 # This runs wesnoth with a multiplayer AI-only matchup.
-Test-Command: /usr/games/wesnoth-BRANCH -m --controller 1:ai --controller 2:ai --nogui
+Test-Command: SDL_VIDEODRIVER=offscreen,dummy /usr/games/wesnoth-BRANCH -m --controller 1:ai --controller 2:ai --nogui
 Depends:
  wesnoth-BRANCH,
  xauth,
-- 
2.51.0


--- End Message ---
--- Begin Message ---
Source: wesnoth-1.18
Source-Version: 1:1.18.6-1
Done: Vincent Cheng <[email protected]>

We believe that the bug you reported is fixed in the latest version of
wesnoth-1.18, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Vincent Cheng <[email protected]> (supplier of updated wesnoth-1.18 package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Thu, 25 Dec 2025 21:05:52 -0800
Source: wesnoth-1.18
Built-For-Profiles: noudeb
Architecture: source
Version: 1:1.18.6-1
Distribution: unstable
Urgency: medium
Maintainer: Debian Games Team <[email protected]>
Changed-By: Vincent Cheng <[email protected]>
Closes: 1123731
Changes:
 wesnoth-1.18 (1:1.18.6-1) unstable; urgency=medium
 .
   [ Vincent Cheng ]
   * New upstream stable release.
   * Bump Standards-Version to 4.7.3, no changes needed.
 .
   [ Simon McVittie ]
   * d/tests/control.in: Explicitly use offscreen video driver
     (Closes: #1123731)
Checksums-Sha1:
 2156a28d8cdc4ea6755ef8eff8147c36761f1bb8 4333 wesnoth-1.18_1.18.6-1.dsc
 8795f627447daded51f6a1316b9923396308ceb1 467190064 
wesnoth-1.18_1.18.6.orig.tar.bz2
 914070e4a10d520310673c91c33dcfe33fa9c6ab 179852 
wesnoth-1.18_1.18.6-1.debian.tar.xz
Checksums-Sha256:
 334feddcef3522838cbf2d69d3186e551ef4007ab5647d0cd799e5c906201c10 4333 
wesnoth-1.18_1.18.6-1.dsc
 6bb8b17854c974bc66cb7a2574a53fe9efb4d2c138bb1373032e57788204985e 467190064 
wesnoth-1.18_1.18.6.orig.tar.bz2
 c1822a06969c8d6c115a764a14d2e0a576377ea7c191302c575739afe88564b8 179852 
wesnoth-1.18_1.18.6-1.debian.tar.xz
Files:
 24d4edd23d5be473c9984c8a485d2445 4333 games optional wesnoth-1.18_1.18.6-1.dsc
 6df331cefb56598eb8cdd032a897cee5 467190064 games optional 
wesnoth-1.18_1.18.6.orig.tar.bz2
 398ada58c39b389f388b57d4ed96620d 179852 games optional 
wesnoth-1.18_1.18.6-1.debian.tar.xz

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEE1TqBWjy3ZZr4guOVju3MG6ofMv8FAmlOHAAACgkQju3MG6of
Mv8AzxAAxTGn6708tunGk17WEL/ImG8m1GfHRRMZN8gY6UvO1JEAUwpRuvNeEdPE
5omC99vYpQu4Liemu9BdoTP84Uv0huwOURBZAs2w9QYGESy4fY3yRrou0ycxGs0/
STb+S2MR1V3eDP8D3kGHXMwVqb/PLikaIwHM1T4a0PifwZChLxe+5vJOhIJNbO9V
hmujdBO3kl9IJNUQQ392p9Vs8OgkgZGTv3Ni+AtlytJqPyEza5u945fZbHmb9IMu
eZ3T/6Vdb5loWvfBDUmGJJrQKYUwPiC/dFMeC96ZOGgqHmQvNZbuzKI1DyjIVZLk
vpzSEMPgDLTYBTcso4spGxbnoQbAv3UaFJrf356oR7V0qU5kTIYsIvM2Yu15Hil7
9jopnvzQSb2i+lSLulXHnITyYOrXr27PfpPh+aE15D6pRGSTgGMOTPXEi71ETbYv
AW1ty9d6r4HrLnq/Jh6R2zKIdPVky3FjftHVGf3vC6QF036h3WuSNp+xZHeDLHfY
ZPv0IpBAPDC4ANhTBzj8NmnBOxiIIAZXlvjtXv2m444mefkmFjj09bZIavXyvsXQ
fsy+7gFegBpD0LVsfO7ElG8HfwVnqduhxEinzLd0aWUEyg4Q2vBQjL+PJTgPnnni
rGPM6kmjr8F0FCffpp3Sv2m6I7Z2/6Dq/VgFPifRaWPeyUX4VBk=
=ZcCJ
-----END PGP SIGNATURE-----

Attachment: pgp1KlksWMyV5.pgp
Description: PGP signature


--- End Message ---

Reply via email to