Skip to content

Commit

Permalink
- Fix build after SDL2_Mixer update
Browse files Browse the repository at this point in the history
Don't try to check for SDL2_Mixer at configure phase by trying to
run program which tries Mix_LoadMUS.

First, the program is incorrect as per SDL2_Mixer documentation,
one is not allowed to call any SDL2_Mixer functions without first
calling Mix_Init() (which is not called).

Next, calling any mixer functions may involve opening audio device,
which may not be present on the machine where the package is built.

Fix by forcing the broken check to pass.

PR:		235557
Reported by:	[email protected]
Approved by:	portmgr blanket (build fix)
Upstream issue:	sekaiproject/ponscripter-fork#92
  • Loading branch information
AMDmi3 committed Feb 8, 2019
1 parent 636c641 commit 65106fc
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions games/ponscripter-sekai/files/patch-configure
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
--- configure.orig 2019-02-08 19:59:55 UTC
+++ configure
@@ -606,7 +606,7 @@ then
_EOF
INTERNAL_SDL_MIXER=true
$CXX `$SDL_CONFIG --cflags` test.cc `$SDL_CONFIG --libs` -lSDL2_mixer -o mtest >/dev/null 2>&1
- ./mtest 2>/dev/null
+ true 2>/dev/null
case $? in
0) echo "yes"; INTERNAL_SDL_MIXER=false ;;
1) echo "no MP3" ;; 2) echo "no Ogg" ;; 3) echo "no Ogg or MP3" ;;

0 comments on commit 65106fc

Please sign in to comment.