Arch Linux compilation fails with sdl3_mixer-git

[WoolieWool]

Using the sdl3_mixer-git package from the AUR (r2074.f9566f4-1), attempting to build QrustyQuake with cdaudio yields the following errors:

cdaudio.c:67:10: error: ‘MIX_INIT_OGG’ undeclared here (not in a function) 67 | {MIX_INIT_OGG, "OGG", 1, (const char *[]){".ogg"}, true}, | ^~~~~~~~~~~~ cdaudio.c:68:10: error: ‘MIX_INIT_OPUS’ undeclared here (not in a function) 68 | {MIX_INIT_OPUS, "OPUS", 2, (const char *[]){".ogg", ".opus"}, true}, | ^~~~~~~~~~~~~ cdaudio.c:69:10: error: ‘MIX_INIT_MP3’ undeclared here (not in a function) 69 | {MIX_INIT_MP3, "MP3", 1, (const char *[]){".mp3"}, true}, | ^~~~~~~~~~~~ cdaudio.c:70:10: error: ‘MIX_INIT_FLAC’ undeclared here (not in a function) 70 | {MIX_INIT_FLAC, "FLAC", 1, (const char *[]){".flac"}, false}, | ^~~~~~~~~~~~~ cdaudio.c:71:10: error: ‘MIX_INIT_MID’ undeclared here (not in a function) 71 | {MIX_INIT_MID, "MID", 2, (const char *[]){".mid", ".midi"}, false}, | ^~~~~~~~~~~~ cdaudio.c:72:10: error: ‘MIX_INIT_MOD’ undeclared here (not in a function) 72 | {MIX_INIT_MOD, "MOD", 1, (const char *[]){".mod"}, false}, | ^~~~~~~~~~~~ cdaudio.c:73:10: error: ‘MIX_INIT_WAVPACK’ undeclared here (not in a function) 73 | {MIX_INIT_WAVPACK, "WAVPACK", 2, (const char *[]){".wav", ".wv"}, false} | ^~~~~~~~~~~~~~~~ cdaudio.c:76:8: error: unknown type name ‘Mix_Music’ 76 | static Mix_Music *current_music = NULL; | ^~~~~~~~~ cdaudio.c: In function ‘BGM_Play’: cdaudio.c:86:9: error: unknown type name ‘Mix_Music’ 86 | Mix_Music *music = NULL; | ^~~~~~~~~ cdaudio.c:123:17: error: implicit declaration of function ‘Mix_LoadMUS_IO’ [-Wimplicit-function-declaration] 123 | music = Mix_LoadMUS_IO(io, true); | ^~~~~~~~~~~~~~ cdaudio.c:123:15: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion] 123 | music = Mix_LoadMUS_IO(io, true); | ^ cdaudio.c:134:14: error: implicit declaration of function ‘Mix_PlayMusic’; did you mean ‘MIX_PlayAudio’? [-Wimplicit-function-declaration] 134 | if (!Mix_PlayMusic(current_music, looping ? -1 : 0)) | ^~~~~~~~~~~~~ | MIX_PlayAudio cdaudio.c: In function ‘CDAudio_Play’: cdaudio.c:159:9: error: unknown type name ‘Mix_Music’ 159 | Mix_Music *music = NULL; | ^~~~~~~~~ cdaudio.c:190:15: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion] 190 | music = Mix_LoadMUS_IO(io, true); | ^ cdaudio.c: In function ‘CDAudio_Stop’: cdaudio.c:210:9: error: implicit declaration of function ‘Mix_HaltMusic’ [-Wimplicit-function-declaration] 210 | Mix_HaltMusic(); | ^~~~~~~~~~~~~ gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g d_polyse.c -o d_polyse.o cdaudio.c: In function ‘CDAudio_Pause’: cdaudio.c:219:9: error: implicit declaration of function ‘Mix_PauseMusic’ [-Wimplicit-function-declaration] 219 | Mix_PauseMusic(); | ^~~~~~~~~~~~~~ cdaudio.c: In function ‘CDAudio_Resume’: cdaudio.c:224:9: error: implicit declaration of function ‘Mix_ResumeMusic’ [-Wimplicit-function-declaration] 224 | Mix_ResumeMusic(); | ^~~~~~~~~~~~~~~ gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g d_scan.c -o d_scan.o cdaudio.c: In function ‘CDAudio_Update’: cdaudio.c:235:17: error: implicit declaration of function ‘Mix_VolumeMusic’ [-Wimplicit-function-declaration] 235 | Mix_VolumeMusic(bgmvolume.value * MIX_MAX_VOLUME); | ^~~~~~~~~~~~~~~ gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g d_sky.c -o d_sky.o gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g d_sprite.c -o d_sprite.o gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g d_surf.c -o d_surf.o cdaudio.c:235:51: error: ‘MIX_MAX_VOLUME’ undeclared (first use in this function) 235 | Mix_VolumeMusic(bgmvolume.value * MIX_MAX_VOLUME); | ^~~~~~~~~~~~~~ cdaudio.c:235:51: note: each undeclared identifier is reported only once for each function it appears in gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g d_fog.c -o d_fog.o cdaudio.c: In function ‘CDAudio_Init’: cdaudio.c:244:17: error: unknown type name ‘MIX_InitFlags’; did you mean ‘SDL_InitFlags’? 244 | MIX_InitFlags flags = Mix_Init(music_formats[i].flag); | ^~~~~~~~~~~~~ | SDL_InitFlags gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g d_common.c -o d_common.o cdaudio.c:244:39: error: implicit declaration of function ‘Mix_Init’; did you mean ‘MIX_Init’? [-Wimplicit-function-declaration] 244 | MIX_InitFlags flags = Mix_Init(music_formats[i].flag); | ^~~~~~~~ | MIX_Init gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g host.c -o host.o gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g host_cmd.c -o host_cmd.o cdaudio.c:254:14: error: implicit declaration of function ‘Mix_OpenAudio’ [-Wimplicit-function-declaration] 254 | if (!Mix_OpenAudio(SDL_AUDIO_DEVICE_DEFAULT_PLAYBACK, NULL)) | ^~~~~~~~~~~~~ cdaudio.c: In function ‘CDAudio_Shutdown’: cdaudio.c:272:9: error: implicit declaration of function ‘Mix_Quit’; did you mean ‘MIX_Quit’? [-Wimplicit-function-declaration] 272 | Mix_Quit(); | ^~~~~~~~ | MIX_Quit gcc -c -Wall -Wextra -D_DEFAULT_SOURCE -DAVAIL_SDL3MIXER -O3 -flto -g in_sdl.c -o in_sdl.o make: *** [Makefile:60: cdaudio.o] Error 1 make: *** Waiting for unfinished jobs....

[WoolieWool]

Upon further investigation, the sdl3_mixer-git package in the AUR is out of date, but even compiling both SDL3 and SDL3_mixer from scratch, I have not been able to successfully build with music enabled. Would it be possible to make an AppImage with SDL3_mixer statically linked until SDL3_mixer gets proper packaging?

[cyanbun96]

In the least helpful turn of events, my system with sdl3_mixer-git from AUR still works just fine.
Appimage with static libs is something I've considered, but didn't bother with yet. I guess that'd be a good goal for v6.1.
Taking a look at your logs, it's mostly full of MIX_Quit and Mix_Quit function name mismatches - one's from SDL2, the other from SDL3... They're supposed to be backwards compatible though? And they work on my and the original author's @erysdren systems? What a mess. I'll try and poke around and have it make more sense, updating this issue if anything productive occurs.

[Blzut3]

Not sure what you mean by "They're supposed to be backwards compatible though?" SDL3 mixer hasn't been officially released and, as the major version difference implies, it's an API/ABI break from SDL2 mixer so anything goes for a least a little bit longer. (I believe the stated plan is for mixer to release with SDL 3.4.)

The -git AUR package fetches the latest version so contrary to what WoolieWool said, it's not out of date as evidenced by the version being stated f9566f4 in the original post. From what I understand the version number listed on the AUR website for -git packages isn't correct since it's determined dynamically on build.

[cyanbun96]

Not sure what you mean by "They're supposed to be backwards compatible though?"

The function names, I mean.
SDL2 quit function
SDL3 quit function
The only difference appears to be capitalization, which WoolieWool's compiler is complaining about. It could be "fixed" quite easily with an explicit #define Mix_ MIX_, but why? Why doesn't it work for WoolieWool in the first place?

[Blzut3]

SDL3_mixer is not backwards compatible with SDL2_mixer. It looks like the change in function names (whole library overhaul is more like it) got merged about a month ago: libsdl-org/SDL_mixer@05393fb So if you're not having the same issue as WoolieWool that means you need to update your SDL3_mixer. Since SDL3_mixer hasn't had an official release yet that the API shifted under you shouldn't be a surprise.

That is assuming QrustyQuake doesn't support being compiled with SDL2 and SDL2_mixer of course.

[cyanbun96]

You're right, and it's even messier than I thought. Attempting to forcefully update the AUR package on my system gets me

Could not find a configuration file for package "SDL3" that is compatible
  with requested version "3.3.0".

The official Arch repo's got SDL3.2, and it indeed doesn't support MIX_ functions, only Mix_.
WoolieWool's system is the most up-to-date one here, and I suppose all of us are gonna have the same error when the Arch repos update.
Oh well. Should be a simple search and replace when it happens.

[WoolieWool]

Error output from trying to compile again, but with saner formatting:

[ 1%] Building C object CMakeFiles/qrustyquake.dir/src/cmd.c.o
[ 2%] Building C object CMakeFiles/qrustyquake.dir/src/cl_main.c.o
[ 4%] Building C object CMakeFiles/qrustyquake.dir/src/chase.c.o
[ 5%] Building C object CMakeFiles/qrustyquake.dir/src/cl_input.c.o
[ 7%] Building C object CMakeFiles/qrustyquake.dir/src/cl_demo.c.o
[ 9%] Building C object CMakeFiles/qrustyquake.dir/src/cdaudio.c.o
[ 9%] Building C object CMakeFiles/qrustyquake.dir/src/cl_parse.c.o
[ 12%] Building C object CMakeFiles/qrustyquake.dir/src/common.c.o
[ 12%] Building C object CMakeFiles/qrustyquake.dir/src/cl_tent.c.o
[ 14%] Building C object CMakeFiles/qrustyquake.dir/src/console.c.o
[ 16%] Building C object CMakeFiles/qrustyquake.dir/src/crc.c.o
[ 16%] Building C object CMakeFiles/qrustyquake.dir/src/cvarlist.c.o
[ 18%] Building C object CMakeFiles/qrustyquake.dir/src/cvar.c.o
[ 19%] Building C object CMakeFiles/qrustyquake.dir/src/d_common.c.o
/home/hem/sources/qrustyquake/src/cdaudio.c:67:10: error: ‘MIX_INIT_OGG’ undeclared here (not in a function)
67 | {MIX_INIT_OGG, "OGG", 1, (const char *[]){".ogg"}, true},
| ^~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:68:10: error: ‘MIX_INIT_OPUS’ undeclared here (not in a function)
68 | {MIX_INIT_OPUS, "OPUS", 2, (const char *[]){".ogg", ".opus"}, true},
| ^~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:69:10: error: ‘MIX_INIT_MP3’ undeclared here (not in a function)
69 | {MIX_INIT_MP3, "MP3", 1, (const char *[]){".mp3"}, true},
| ^~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:70:10: error: ‘MIX_INIT_FLAC’ undeclared here (not in a function)
70 | {MIX_INIT_FLAC, "FLAC", 1, (const char *[]){".flac"}, false},
| ^~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:71:10: error: ‘MIX_INIT_MID’ undeclared here (not in a function)
71 | {MIX_INIT_MID, "MID", 2, (const char *[]){".mid", ".midi"}, false},
| ^~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:72:10: error: ‘MIX_INIT_MOD’ undeclared here (not in a function)
72 | {MIX_INIT_MOD, "MOD", 1, (const char *[]){".mod"}, false},
| ^~~~~~~~~~~~
[ 21%] Building C object CMakeFiles/qrustyquake.dir/src/d_fog.c.o
[ 22%] Building C object CMakeFiles/qrustyquake.dir/src/d_edge.c.o
[ 23%] Building C object CMakeFiles/qrustyquake.dir/src/d_part.c.o
/home/hem/sources/qrustyquake/src/cdaudio.c:73:10: error: ‘MIX_INIT_WAVPACK’ undeclared here (not in a function)
73 | {MIX_INIT_WAVPACK, "WAVPACK", 2, (const char *[]){".wav", ".wv"}, false}
| ^~~~~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:76:8: error: unknown type name ‘Mix_Music’
76 | static Mix_Music *current_music = NULL;
| ^~~~~~~~~
[ 25%] Building C object CMakeFiles/qrustyquake.dir/src/d_polyse.c.o
/home/hem/sources/qrustyquake/src/cdaudio.c: In function ‘BGM_Play’:
/home/hem/sources/qrustyquake/src/cdaudio.c:86:9: error: unknown type name ‘Mix_Music’
86 | Mix_Music *music = NULL;
| ^~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:123:17: error: implicit declaration of function ‘Mix_LoadMUS_IO’ [-Wimplicit-function-declaration]
123 | music = Mix_LoadMUS_IO(io, true);
| ^~~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:123:15: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
123 | music = Mix_LoadMUS_IO(io, true);
| ^
[ 26%] Building C object CMakeFiles/qrustyquake.dir/src/d_scan.c.o
/home/hem/sources/qrustyquake/src/cdaudio.c:134:14: error: implicit declaration of function ‘Mix_PlayMusic’; did you mean ‘MIX_PlayAudio’? [-Wimplicit-function-declaration]
134 | if (!Mix_PlayMusic(current_music, looping ? -1 : 0))
| ^~~~~~~~~~~~~
| MIX_PlayAudio
[ 29%] Building C object CMakeFiles/qrustyquake.dir/src/d_sky.c.o
[ 28%] Building C object CMakeFiles/qrustyquake.dir/src/d_sprite.c.o
[ 30%] Building C object CMakeFiles/qrustyquake.dir/src/globals.c.o
[ 32%] Building C object CMakeFiles/qrustyquake.dir/src/d_surf.c.o
/home/hem/sources/qrustyquake/src/cdaudio.c: In function ‘CDAudio_Play’:
/home/hem/sources/qrustyquake/src/cdaudio.c:159:9: error: unknown type name ‘Mix_Music’
159 | Mix_Music *music = NULL;
| ^~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:190:15: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
190 | music = Mix_LoadMUS_IO(io, true);
| ^
[ 33%] Building C object CMakeFiles/qrustyquake.dir/src/host.c.o
[ 35%] Building C object CMakeFiles/qrustyquake.dir/src/host_cmd.c.o
/home/hem/sources/qrustyquake/src/cdaudio.c: In function ‘CDAudio_Stop’:
/home/hem/sources/qrustyquake/src/cdaudio.c:210:9: error: implicit declaration of function ‘Mix_HaltMusic’ [-Wimplicit-function-declaration]
210 | Mix_HaltMusic();
| ^~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c: In function ‘CDAudio_Pause’:
/home/hem/sources/qrustyquake/src/cdaudio.c:219:9: error: implicit declaration of function ‘Mix_PauseMusic’ [-Wimplicit-function-declaration]
219 | Mix_PauseMusic();
| ^~~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c: In function ‘CDAudio_Resume’:
/home/hem/sources/qrustyquake/src/cdaudio.c:224:9: error: implicit declaration of function ‘Mix_ResumeMusic’ [-Wimplicit-function-declaration]
224 | Mix_ResumeMusic();
| ^~~~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c: In function ‘CDAudio_Update’:
/home/hem/sources/qrustyquake/src/cdaudio.c:235:17: error: implicit declaration of function ‘Mix_VolumeMusic’ [-Wimplicit-function-declaration]
235 | Mix_VolumeMusic(bgmvolume.value * MIX_MAX_VOLUME);
| ^~~~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:235:51: error: ‘MIX_MAX_VOLUME’ undeclared (first use in this function)
235 | Mix_VolumeMusic(bgmvolume.value * MIX_MAX_VOLUME);
| ^~~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c:235:51: note: each undeclared identifier is reported only once for each function it appears in
[ 36%] Building C object CMakeFiles/qrustyquake.dir/src/image.c.o
/home/hem/sources/qrustyquake/src/cdaudio.c: In function ‘CDAudio_Init’:
/home/hem/sources/qrustyquake/src/cdaudio.c:244:17: error: unknown type name ‘MIX_InitFlags’; did you mean ‘SDL_InitFlags’?
244 | MIX_InitFlags flags = Mix_Init(music_formats[i].flag);
| ^~~~~~~~~~~~~
| SDL_InitFlags
/home/hem/sources/qrustyquake/src/cdaudio.c:244:39: error: implicit declaration of function ‘Mix_Init’; did you mean ‘MIX_Init’? [-Wimplicit-function-declaration]
244 | MIX_InitFlags flags = Mix_Init(music_formats[i].flag);
| ^~~~~~~~
| MIX_Init
/home/hem/sources/qrustyquake/src/cdaudio.c:254:14: error: implicit declaration of function ‘Mix_OpenAudio’ [-Wimplicit-function-declaration]
254 | if (!Mix_OpenAudio(SDL_AUDIO_DEVICE_DEFAULT_PLAYBACK, NULL))
| ^~~~~~~~~~~~~
/home/hem/sources/qrustyquake/src/cdaudio.c: In function ‘CDAudio_Shutdown’:
/home/hem/sources/qrustyquake/src/cdaudio.c:272:9: error: implicit declaration of function ‘Mix_Quit’; did you mean ‘MIX_Quit’? [-Wimplicit-function-declaration]
272 | Mix_Quit();
| ^~~~~~~~
| MIX_Quit
make[2]: *** [CMakeFiles/qrustyquake.dir/build.make:79: CMakeFiles/qrustyquake.dir/src/cdaudio.c.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:87: CMakeFiles/qrustyquake.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

[cyanbun96]

Yup, seems broken. After a short consultation with @erysdren, the original implementer, a decision was made to postpone the fix until the SDL3_Mixer gets a stable release.

For reference, the working commit of SDL3_Mixer is c9178dd3 and the version of SDL from Arch repos is sdl3 3.2.20-1

[WoolieWool]

Would it be possible to make an AppImage with the relevant libraries statically linked as a stopgap?

[cyanbun96]

Would it be possible to make an AppImage with the relevant libraries statically linked as a stopgap?

That's the plan for v6.1. I only have, like, two-ish minor features in mind that I'd like to add for it, so that should come in the near future.