From ed1040e5dca5686ba989c626f8139dc6db7d31c8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pablo=20Correa=20G=C3=B3mez?= Date: Mon, 20 Nov 2023 20:43:21 +0100 Subject: [PATCH] temp/pulseaudio: disable lto, might be broken in some armv7 devices (MR 4563) Something seems to have gone wrong in our rebuild and sync of pulseaudio with alpine upstream. And user feedback points that building armv7 devices with LTO is actually broken in our builders. In alpine, they enabled lto in pulseaudio very long ago (f46174365b219fe431890df3df7e7dc488c9451e 08-09-22) and nobody as complained so far. Since it's not really a phone-specific package, one would hope that more people upstream would have noticed if it were broken there. Since nobody has, we assume this is a downstream problem, and just unconditionally disable lto in our build. We hope this will be just a temporary workaround, until the patches that prompted the fork are no longer needed, and we can simply use upstream pulseaudio, that shouldn't be broken. Fixes #2368 [ci:skip-build] already built successfully in CI --- temp/pulseaudio/APKBUILD | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/temp/pulseaudio/APKBUILD b/temp/pulseaudio/APKBUILD index c95ed27da..15136677f 100644 --- a/temp/pulseaudio/APKBUILD +++ b/temp/pulseaudio/APKBUILD @@ -3,7 +3,7 @@ pkgname=pulseaudio pkgver=9999_git20220621 _pkgver=16.1 -pkgrel=1 +pkgrel=2 pkgdesc="featureful, general-purpose sound server" provider_priority=10 url="https://www.freedesktop.org/wiki/Software/PulseAudio/" @@ -115,8 +115,10 @@ prepare() { } build() { + # See pma#2368 + _lto_enable="false" abuild-meson \ - -Db_lto=true \ + -Db_lto="$_lto_enable" \ -Dgcov=false \ -Dman=true \ -Dtests="$(want_check && echo true || echo false)" \