On Sat, Jun 22, 2019 at 10:45:16AM +0200, Frederic Cambus wrote:
> Hi ports@,
> 
> Here is a new port: audio/libopenmpt.
> 
> From DESCR:
> 
> libopenmpt is a cross-platform C++ and C library to decode tracked music
> files (modules) into a raw PCM audio stream.
> 
> openmpt123 is a cross-platform command-line or terminal based module file
> player.
> 
> libopenmpt is based on the player code of the OpenMPT project (Open ModPlug
> Tracker). In order to avoid code base fragmentation, libopenmpt is developed
> in the same source code repository as OpenMPT.
> 
> Comments? OK?

It looks good and works on amd64.
I also tried enabling it in ffmpeg (--enable-libopenmpt) which works fine too.

On macppc however, linking fails with the following errors:
bin/.libs/libopenmpt.so.0.0: undefined reference to `mpg123_open_handle_64'
bin/.libs/libopenmpt.so.0.0: undefined reference to 
`mpg123_replace_reader_handle_64'
bin/.libs/libopenmpt.so.0.0: undefined reference to `mpg123_length_64'
collect2: error: ld returned 1 exit status

Does anyone know why the large file support is disabled in mpg123?
With the diffs below (one is for mpg123), it seems to work.. 


diff -uNpr /tmp/libopenmpt/Makefile /usr/ports/audio/libopenmpt/Makefile
--- /tmp/libopenmpt/Makefile    Sat Jun 22 10:42:41 2019
+++ /usr/ports/audio/libopenmpt/Makefile        Tue Jun 25 12:58:44 2019
@@ -21,6 +21,8 @@ WANTLIB +=    sndio vorbis vorbisenc vorbisfile z
 
 MASTER_SITES = https://lib.openmpt.org/files/libopenmpt/src/
 
+COMPILER =     base-clang ports-gcc
+
 LIB_DEPENDS =  audio/flac \
                audio/libsndfile \
                audio/libvorbis \
Index: Makefile
===================================================================
RCS file: /cvs/ports/audio/mpg123/Makefile,v
retrieving revision 1.97
diff -u -p -u -p -r1.97 Makefile
--- Makefile    10 Jul 2018 15:28:51 -0000      1.97
+++ Makefile    25 Jun 2019 11:53:37 -0000
@@ -22,7 +22,6 @@ WANTLIB=      c m sndio
 
 CONFIGURE_STYLE=gnu
 CONFIGURE_ARGS=        --enable-static \
-               --disable-lfs-alias \
                --with-audio=sndio \
                --with-optimization=0
 CONFIGURE_ENV= CPPFLAGS="-I${LOCALBASE}/include" \

Reply via email to