From 2bfa848a41335869354a67578a5c4bbb21a3b9aa Mon Sep 17 00:00:00 2001
From: André Batista <nandre@riseup.net>
Date: Tue, 27 Aug 2024 15:46:26 -0300
Subject: gnu: ffmpeg: Fix build on i686 with binutils 2.41.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* gnu/packages/video.scm (ffmpeg): Add 'bypass-openal-check phase
which breaks configure phase on i686. Unconditionally set openal
configure check to true.

Co-authored-by: Ludovic Courtès <ludo@gnu.org>
Change-Id: Iddba489690a42a5e9f9a40f9a238c658ddac6e20
---
 gnu/packages/video.scm | 12 ++++++++++++
 1 file changed, 12 insertions(+)

(limited to 'gnu/packages')

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 462daf2867..ac9fabeb38 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -1836,6 +1836,18 @@ operate properly.")
          "--disable-mipsfpu")
       #:phases
       #~(modify-phases %standard-phases
+          #$@(if (target-x86-32?)
+                 #~((add-before 'configure 'bypass-openal-check
+                      ;; configure fails linking to openal when using binutils
+                      ;; >= 2.38 due to openal's usage of protected visibility
+                      ;; for its dynamic symbols. Bypass this configure time
+                      ;; check for now. See:
+                      ;; https://lists.gnu.org/archive/html/guix-devel/2024-08/msg00159.html
+                      (lambda _
+                        (substitute* "configure"
+                          (("alGetError \\|\\|")
+                           "alGetError \|\| true \|\|")))))
+                 #~())
           (replace 'configure
             ;; configure does not work followed by "SHELL=..." and
             ;; "CONFIG_SHELL=..."; set environment variables instead
-- 
cgit v1.2.3