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" \