diff options
author | Piotr Tworek <tworaz@tworaz.net> | 2019-07-24 22:33:50 +0200 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-07-26 08:41:39 +0100 |
commit | 70a93af71b372af2c16036c835ca327216bd6fae (patch) | |
tree | 8d566e5993d091e7ef29510ffba8f606e1791938 /meta | |
parent | 768696d9c97cfb22a03a31c64e1b86db12ac5d88 (diff) | |
download | poky-70a93af71b372af2c16036c835ca327216bd6fae.tar.gz |
pulseaudio: Backport upstream fix new alsa compatibility.
Recent alsa upgrade stripped /usr/include/alsa directory from include
path reported by pkgconfig. Due to this pulseaudio 12.2 configure script
can find alsa's use-case.h header which in turn results in HAVE_ALSA_UCM
being undefined. This turn results in pa_alsa_ucm_device_update_available
symbol missing even though libalsa-util.so needs it. Once could argue
pulseaudio should not allow undefined symmbols in its shared modules.
Unfortunately it does and due to this current OE builds of pulseaudio
crash when the server tries to dlopen any module using libalsa-util.so.
Fix this by backporting ustream alsa header include fix.
(From OE-Core rev: 58319f6285de41f7b99f5fd97c42d836fb5544f3)
Signed-off-by: Piotr Tworek <tworaz@tworaz.net>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
-rw-r--r-- | meta/recipes-multimedia/pulseaudio/pulseaudio/0001-alsa-Fix-inclusion-of-use-case.h.patch | 46 | ||||
-rw-r--r-- | meta/recipes-multimedia/pulseaudio/pulseaudio_12.2.bb | 1 |
2 files changed, 47 insertions, 0 deletions
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio/0001-alsa-Fix-inclusion-of-use-case.h.patch b/meta/recipes-multimedia/pulseaudio/pulseaudio/0001-alsa-Fix-inclusion-of-use-case.h.patch new file mode 100644 index 0000000000..15026a2f83 --- /dev/null +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio/0001-alsa-Fix-inclusion-of-use-case.h.patch | |||
@@ -0,0 +1,46 @@ | |||
1 | From b89d33bb182c42db5ad3987b0e91b7bf62f421e8 Mon Sep 17 00:00:00 2001 | ||
2 | From: Takashi Iwai <tiwai@suse.de> | ||
3 | Date: Sun, 21 Apr 2019 11:59:30 +0200 | ||
4 | Subject: [PATCH] alsa: Fix inclusion of use-case.h | ||
5 | |||
6 | The recent change in ALSA upstream stripped -I$include/alsa path from | ||
7 | pkgconfig. We already fixed for this change in some places but still | ||
8 | the code for UCM was overlooked, and this resulted in the unresolved | ||
9 | symbols in alsa card module. Fix them as well. | ||
10 | |||
11 | Signed-off-by: Takashi Iwai <tiwai@suse.de> | ||
12 | Upstream-Status: Backport | ||
13 | --- | ||
14 | configure.ac | 2 +- | ||
15 | src/modules/alsa/alsa-ucm.h | 2 +- | ||
16 | 2 files changed, 2 insertions(+), 2 deletions(-) | ||
17 | |||
18 | diff --git a/configure.ac b/configure.ac | ||
19 | index c004bd70d..b44ed1595 100644 | ||
20 | --- a/configure.ac | ||
21 | +++ b/configure.ac | ||
22 | @@ -826,7 +826,7 @@ AS_IF([test "x$enable_alsa" = "xyes" && test "x$HAVE_ALSA" = "x0"], | ||
23 | AS_IF([test "x$HAVE_ALSA" = "x1"], | ||
24 | [ | ||
25 | save_CPPFLAGS="$CPPFLAGS"; CPPFLAGS="$CPPFLAGS $ASOUNDLIB_CFLAGS" | ||
26 | - AC_CHECK_HEADERS([use-case.h], HAVE_ALSA_UCM=1, HAVE_ALSA_UCM=0) | ||
27 | + AC_CHECK_HEADERS([alsa/use-case.h], HAVE_ALSA_UCM=1, HAVE_ALSA_UCM=0) | ||
28 | CPPFLAGS="$save_CPPFLAGS" | ||
29 | ], | ||
30 | HAVE_ALSA_UCM=0) | ||
31 | diff --git a/src/modules/alsa/alsa-ucm.h b/src/modules/alsa/alsa-ucm.h | ||
32 | index 53abf3f90..c926f3cc3 100644 | ||
33 | --- a/src/modules/alsa/alsa-ucm.h | ||
34 | +++ b/src/modules/alsa/alsa-ucm.h | ||
35 | @@ -23,7 +23,7 @@ | ||
36 | ***/ | ||
37 | |||
38 | #ifdef HAVE_ALSA_UCM | ||
39 | -#include <use-case.h> | ||
40 | +#include <alsa/use-case.h> | ||
41 | #else | ||
42 | typedef void snd_use_case_mgr_t; | ||
43 | #endif | ||
44 | -- | ||
45 | 2.21.0 | ||
46 | |||
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_12.2.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_12.2.bb index c020fbd4f6..88b4975a66 100644 --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_12.2.bb +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_12.2.bb | |||
@@ -3,6 +3,7 @@ require pulseaudio.inc | |||
3 | SRC_URI = "http://freedesktop.org/software/pulseaudio/releases/${BP}.tar.xz \ | 3 | SRC_URI = "http://freedesktop.org/software/pulseaudio/releases/${BP}.tar.xz \ |
4 | file://0001-client-conf-Add-allow-autospawn-for-root.patch \ | 4 | file://0001-client-conf-Add-allow-autospawn-for-root.patch \ |
5 | file://0001-introduce-a-special-build-flag-to-explicitly-disable.patch \ | 5 | file://0001-introduce-a-special-build-flag-to-explicitly-disable.patch \ |
6 | file://0001-alsa-Fix-inclusion-of-use-case.h.patch \ | ||
6 | file://0002-do-not-display-CLFAGS-to-improve-reproducibility-bui.patch \ | 7 | file://0002-do-not-display-CLFAGS-to-improve-reproducibility-bui.patch \ |
7 | file://volatiles.04_pulse \ | 8 | file://volatiles.04_pulse \ |
8 | " | 9 | " |