Summary: | media-sound/pulseaudio-13 undefined symbol: pa_alsa_ucm_device_update_available | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | crabbed halo ablution <crabbedhaloablution> |
Component: | Current packages | Assignee: | Gentoo ALSA team [DISABLED] <alsa-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | chewi, gnome, jstein, polynomial-c |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://gitlab.freedesktop.org/pulseaudio/pulseaudio/merge_requests/212 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | emerge --info pulseaudio |
Description
crabbed halo ablution
2019-11-21 11:02:39 UTC
This has hit me on an arm and amd64 system. It's pretty bad and it's going to affect a lot of users. It seems to be a change in alsa-lib. 1.9.1 is okay. 1.2.1 and 1.2.1.1 are not. The aforementioned PulseAudio patches were already applied in 13.0 so it's not that. This is what I get from PulseAudio's config.log: configure:20948: checking alsa/use-case.h usability configure:20948: armv7a-unknown-linux-gnueabihf-gcc -c -mcpu=cortex-a9 -mfpu=neon -O3 -pipe -Wall -W -Wextra -Wno-long-long -Wno-overlength-strings -Wunsafe-loop-optimizations -Wundef -Wformat=2 -Wlogical-op -Wsign-compare -Wformat-security -Wmissing-include-dirs -Wformat-nonliteral -Wpointer-arith -Winit-self -Wdeclaration-after-statement -Wfloat-equal -Wmissing-prototypes -Wredundant-decls -Wmissing-declarations -Wmissing-noreturn -Wshadow -Wendif-labels -Wcast-align -Wstrict-aliasing -Wwrite-strings -Wno-unused-parameter -fno-common -fdiagnostics-show-option -fdiagnostics-color=auto -DFASTPATH -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 conftest.c >&5 In file included from conftest.c:185: /usr/include/alsa/use-case.h:461:36: error: unknown type name 'snd_ctl_elem_id_t' 461 | int snd_use_case_parse_ctl_elem_id(snd_ctl_elem_id_t *dst, | ^~~~~~~~~~~~~~~~~ /usr/include/alsa/use-case.h:472:33: error: unknown type name 'snd_mixer_selem_id_t' 472 | int snd_use_case_parse_selem_id(snd_mixer_selem_id_t *dst, | ^~~~~~~~~~~~~~~~~~~~ Okay, I think it's these. New includes are needed. https://git.alsa-project.org/?p=alsa-lib.git;a=commitdiff;h=5c88b183718ce3df11bc4753c76d95f3e9de6c7e https://git.alsa-project.org/?p=alsa-lib.git;a=commitdiff;h=cd4b07ffbdd4612989e63800528b18efe0039ad9 CC'ing alsa-bugs@g.o as this is serious and it's probably going to affect more than just PulseAudio. I gather the main alsa-lib maintainer, Polynomial-C, is somewhat unavailable right now. In the mean time, I tried adding some includes to use-case.h but the list kept getting longer. I'm not sure if that's the right approach in any case. Upstream may say that you're expected to include other headers first/instead. PulseAudio could therefore add alsa/asoundlib.h to its AC_CHECK_HEADERS call so that all the headers are included before use-case.h is checked. I imagine they're going to stumble across this very soon if they haven't already and they'll decide what the right course of action is. A short term work around for us is passing ac_cv_header_alsa_use_case_h=yes to PulseAudio's econf call. I suggest we just do that for now. Technically it should be a revbump as the problem doesn't appear until runtime. I went ahead and submitted a fix to PulseAudio. We'll see what they say. ALSA has now fixed this instead so I closed the above merge request. I don't know when it'll be released though so maybe we should apply the patch now. https://git.alsa-project.org/?p=alsa-lib.git;a=commitdiff;h=1a4b362115b5e1bb4f5b888919ffd11736adbf0c Patch confirmed working. First apply to alsa-lib, then recompile pulseaudio before it works. When it is applied to the tree, we should revbump pulseaudio also. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=e5883f1964b837ad24addc4c24b7bea731012979 commit e5883f1964b837ad24addc4c24b7bea731012979 Author: Lars Wendler <polynomial-c@gentoo.org> AuthorDate: 2019-11-25 21:13:32 +0000 Commit: Lars Wendler <polynomial-c@gentoo.org> CommitDate: 2019-11-25 21:14:48 +0000 media-libs/alsa-lib: Revbump to fix pulseaudio build Reported-by: crabbed halo ablution <crabbedhaloablution@icloud.com> Tested-by: crabbed halo ablution <crabbedhaloablution@icloud.com> Thanks-to: James Le Cuirot <chewi@gentoo.org> Closes: https://bugs.gentoo.org/700870 Package-Manager: Portage-2.3.79, Repoman-2.3.18 Signed-off-by: Lars Wendler <polynomial-c@gentoo.org> media-libs/alsa-lib/alsa-lib-1.2.1.1-r1.ebuild | 86 ++++++++++++++++++++++ .../alsa-lib-1.2.1.1-ucm_add_asoundlib_h.patch | 25 +++++++ 2 files changed, 111 insertions(+) |