[Patch] SDL_mixer 2.0: Add load_mp3.c to the build list

SDL Development
1

Well, this has been broken for a year and a half by now. Always fixed it
by hand every time I had to build.

I think it’s time to actually get it fixed on the repo though. It’s a
simple one-liner.

HG changeset patch

User @Nikos_Chantziaras

Date 1425906486 -7200

Mon Mar 09 15:08:06 2015 +0200

Node ID f7037deec7a6854b4ae6f086095fe858f460767b

Parent b28b41b93ba75fd7330b7cb82050cbbe9fb6ae56

Add load_mp3.c to the build list

diff -r b28b41b93ba7 -r f7037deec7a6 configure.in
— a/configure.in Sat Nov 29 14:42:02 2014 -0800
+++ b/configure.in Mon Mar 09 15:08:06 2015 +0200
@@ -141,6 +141,7 @@
$srcdir/effect_stereoreverse.c
$srcdir/effects_internal.c
$srcdir/load_aiff.c
+$srcdir/load_mp3.c
$srcdir/load_voc.c
$srcdir/mixer.c
$srcdir/music.c"

2

Hasn’t it been left out due to the “interesting” licensing issues around MP3?

3

Nope. MP3 support is controlled through configure options. The omission
of that file results in a build error due to an undefined reference to
the function that file is providing, even if MP3 is enabled in ./configure.On 09/03/15 16:01, MrTAToad wrote:

Hasn’t it been left out due to the “interesting” licensing issues around
MP3?

4

Fair enough :slight_smile: