blob: cbf96a66320e58a913d5758be00bc885f06856d0 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
|
https://github.com/icculus/SDL_sound/pull/80
From 83f33e04125517197520e26452d06b81c2d7b429 Mon Sep 17 00:00:00 2001
From: Sam James <sam@gentoo.org>
Date: Sat, 12 Nov 2022 02:49:21 +0000
Subject: [PATCH] Fix underlinking with lld (missing -lm)
We need to explicitly search for and link with -lm if needed by
libc for math libraries like `log` and `sin`:
```
ld.lld: error: undefined reference due to --no-allow-shlib-undefined: sin
>>> referenced by ../.libs/libSDL_sound.so
ld.lld: error: undefined reference due to --no-allow-shlib-undefined: log
>>> referenced by ../.libs/libSDL_sound.so
```
It turns out Gentoo has actually had a patch for this since 2012 (mea
culpa!) but it never made its way upstream.
We're already using libtool, so just use the macro it provides for
this purpose.
Signed-off-by: Sam James <sam@gentoo.org>
--- a/Makefile.am
+++ b/Makefile.am
@@ -12,7 +12,7 @@ endif
SDLSOUND_HDRS = SDL_sound.h
SDLSOUND_SRCS = SDL_sound.c SDL_sound_internal.h alt_audio_convert.c alt_audio_convert.h audio_convert.c
SDLSOUND_LDFLAGS = -no-undefined -release $(LT_RELEASE) -version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE)
-SDLSOUND_LIBADD = decoders/libdecoders.la $(TIMIDITY_LIB)
+SDLSOUND_LIBADD = decoders/libdecoders.la $(TIMIDITY_LIB) $(LIBM)
if USE_SDL2
lib_LTLIBRARIES = libSDL2_sound.la
--- a/configure.ac
+++ b/configure.ac
@@ -54,6 +54,7 @@ AC_PROG_CC
AC_PROG_INSTALL
AC_PROG_LN_S
LT_INIT([win32-dll])
+LT_LIB_M
PKG_PROG_PKG_CONFIG
|