Skip to content
This repository was archived by the owner on Dec 11, 2025. It is now read-only.

Conversation

@res2k
Copy link
Contributor

@res2k res2k commented Apr 9, 2025

In the previous PR (#461) I missed that newer openal-soft requires CMake >= 3.26, but the used container image only contains 3.25.
Remedy that by explicitly installing a newer CMake version.
(Also, this means that the CMake versions on the Linux and Windows runners are now the same, which can be quite convenient.)

@res2k res2k marked this pull request as draft April 9, 2025 06:48
@res2k res2k marked this pull request as ready for review April 9, 2025 07:28
@apanteleev
Copy link
Collaborator

Thanks, but I would prefer to avoid pulling third-party actions just to update cmake. Maybe there's a way to do it with like apt-get install...?

Also, if the errors are indeed caused by openal-soft requiring a newer cmake verison, perhaps it's worth filing a bug against openal-soft to update their cmake_minimum_required which currently states 3.13.

@res2k
Copy link
Contributor Author

res2k commented Apr 28, 2025

Thanks, but I would prefer to avoid pulling third-party actions just to update cmake. Maybe there's a way to do it with like apt-get install...?

Unfortunately, no.
The "sniper" image is based on Debian Bullseye (AFAICS). The newest CMake version there seems to be cmake 3.25.1 via bullseye-backports (https://packages.debian.org/bullseye-backports/devel/cmake) - this is the version already contained in the sniper image and just short of what we need.

Also, if the errors are indeed caused by openal-soft requiring a newer cmake verison, perhaps it's worth filing a bug against openal-soft to update their cmake_minimum_required which currently states 3.13.

It already was: kcat/openal-soft#1088
Apparently, since the issue only affects static builds, it seems the thinking is to leave the minimum required version at 3.13.

An alternative to using a 3rd party action would be to directly pull & use the Linux x86_64 installer from https://cmake.org/download/.

@res2k
Copy link
Contributor Author

res2k commented Apr 28, 2025

It already was: kcat/openal-soft#1088

And also: kcat/openal-soft#1099

@apanteleev
Copy link
Collaborator

OpenAL updated in a464d10.
Closing.

@apanteleev apanteleev closed this Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants