Bug 8570 - Incorrect version of ALSA used in builds
Summary: Incorrect version of ALSA used in builds
Status: RESOLVED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Sound (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.19.0
Assignee: Pierre Ossman
URL:
Keywords: prosaic
Depends on:
Blocks:
 
Reported: 2025-04-15 10:09 CEST by Pierre Ossman
Modified: 2025-04-23 10:55 CEST (History)
0 users

See Also:
Acceptance Criteria:
MUST: * ThinLinc should be built targeting the latest ALSA that is still supported by all major distributions SHOULD: * Related audio libraries should also be upgraded


Attachments

Description Pierre Ossman cendio 2025-04-15 10:09:38 CEST
The upgrade of gcc (bug 8361) has exposed a latent issue with our usage of ALSA; we're actually requiring a newer version than we think.

Since r27650 for bug 4569, we use the function snd_use_case_set() which was added in alsa-lib 1.0.24. What we have in the build system is 1.0.21.

The only reason it works is that this function is used in a module, where the linker has to tolerate that some symbols cannot be resolved until runtime.
Comment 1 Pierre Ossman cendio 2025-04-15 16:46:17 CEST
We might as well upgrade to the latest possible version.

This is the version used on the oldest current systems:

RHEL 8: 1.2.10
SLES 15: 1.2.10
Ubuntu 22.04: 1.2.6
Debian 11: 1.2.4
Comment 5 Pierre Ossman cendio 2025-04-23 10:55:15 CEST
Seems to work well after an upgrade. Tested the client on Fedora 42, Windows 11, and macOS 15.4.

> MUST:
> 
>  * ThinLinc should be built targeting the latest ALSA that is still supported by all major distributions

Yup. Debian 11 was unfortunately a bit old, so that set the bar. Should hopefully mean better support for minor distributions, though.

> SHOULD:
> 
>  * Related audio libraries should also be upgraded

Upgraded all the libraries in the audio/ folder.

Note You need to log in before you can comment on or make changes to this bug.