[ITA] libopenmpt

[ITA] libopenmpt

[Takashi Yano]

[-- Attachment #1: Type: text/plain, Size: 106 bytes --]

I'd like to adopt the libopenmpt package.
Thanks in advance.

-- 
Takashi Yano <takashi.yano@nifty.ne.jp>

[-- Attachment #2: 0.3.7-cygwin-pulse.patch --]
[-- Type: text/plain, Size: 605 bytes --]

--- origsrc/libopenmpt-0.3.7+release.autotools/configure.ac	2018-03-11 08:03:36.000000000 -0500
+++ src/libopenmpt-0.3.7+release.autotools/configure.ac	2018-03-16 02:04:32.918676500 -0500
@@ -153,7 +153,7 @@ AM_CONDITIONAL([ENABLE_TESTS], [test "x$
 AC_ARG_WITH([pulseaudio], AS_HELP_STRING([--with-pulseaudio], [Enable use of libpulse and libpulse-simple (enabled by default on Linux).]))
 AS_IF([test "x$enable_openmpt123" != "xno"],[
 case $host_os in
- linux*)
+ linux*|cygwin*)
 AS_IF([test "x$with_pulseaudio" != "xno"],
  [
   PKG_CHECK_MODULES([PULSEAUDIO], [libpulse libpulse-simple],

[-- Attachment #3: libopenmpt.cygport --]
[-- Type: text/plain, Size: 1073 bytes --]

NAME="libopenmpt"
VERSION=0.7.3
RELEASE=1
LICENSE="BSD-3-clause"
CATEGORY="Libs"
SUMMARY="Tracked music file decoder library"
DESCRIPTION="libopenmpt is a cross-platform C++ and C library to decode tracked
music files (modules) into a raw PCM audio stream."
HOMEPAGE="https://lib.openmpt.org/libopenmpt/"
SRC_URI="https://lib.openmpt.org/files/libopenmpt/src/libopenmpt-${VERSION}+release.autotools.tar.gz"
SRC_DIR="libopenmpt-${VERSION}+release.autotools"
PATCH_URI="0.3.7-cygwin-pulse.patch"

PKG_NAMES="libopenmpt0 libopenmpt-devel openmpt123"
libopenmpt0_CONTENTS="
	usr/bin/cygopenmpt-0.dll
	usr/share/doc/${NAME}
"
libopenmpt_devel_CONTENTS="
	usr/include/libopenmpt/
	usr/lib/libopenmpt.dll.a
	usr/lib/pkgconfig/libopenmpt.pc
"
openmpt123_CATEGORY="Audio"
openmpt123_SUMMARY=${SUMMARY/ library/}
openmpt123_CONTENTS="
	usr/bin/openmpt123.exe
	usr/share/man/man1/openmpt123.1*
"

DIFF_EXCLUDES="ax_*.m4 pkg.m4"

CPPFLAGS+=" -DMPG123_NO_LARGENAME"

BUILD_REQUIRES="libportaudiocpp-devel libmpg123-devel libogg-devel libvorbis-devel"

[-- Attachment #4: libopenmpt-0.7.3-1.src.patch --]
[-- Type: text/plain, Size: 1014 bytes --]

--- origsrc/libopenmpt-0.7.3+release.autotools/src/mpt/string_transcode/transcode.hpp	2023-04-11 18:17:44.000000000 +0900
+++ src/libopenmpt-0.7.3+release.autotools/src/mpt/string_transcode/transcode.hpp	2023-09-22 01:18:13.564555400 +0900
@@ -1663,7 +1663,7 @@ inline Tdststring encode(logical_encodin
 #elif !defined(MPT_COMPILER_QUIRK_NO_WCHAR)
 	switch (encoding) {
 		case logical_encoding::locale:
-			return encode_locale<Tdststring>(std::locale(""), src);
+			return encode_locale<Tdststring>(std::locale("C"), src);
 			break;
 		case logical_encoding::active_locale:
 			return encode_locale<Tdststring>(std::locale(), src);
@@ -1834,7 +1834,7 @@ inline mpt::widestring decode(logical_en
 #elif !defined(MPT_COMPILER_QUIRK_NO_WCHAR)
 	switch (encoding) {
 		case logical_encoding::locale:
-			return decode_locale(std::locale(""), src);
+			return decode_locale(std::locale("C"), src);
 			break;
 		case logical_encoding::active_locale:
 			return decode_locale(std::locale(), src);

Re: [ITA] libopenmpt

[Jon Turney]

On 22/09/2023 05:22, Takashi Yano via Cygwin-apps wrote:
> libopenmpt.cygport
> 
> 
> BUILD_REQUIRES="libportaudiocpp-devel libmpg123-devel libogg-devel libvorbis-devel"

It seems like this also needs libpulse-devel.

This looks like a historical mistake, but maybe the files in 
/usr/share/doc/libopenmpt/examples/ should be in the devel package?

Otherwise, looks good.

I added this to your packages.

Thanks.

Re: [ITA] libopenmpt

[Takashi Yano]

On Fri, 22 Sep 2023 15:54:30 +0100
Jon Turney wrote:
> On 22/09/2023 05:22, Takashi Yano via Cygwin-apps wrote:
> > libopenmpt.cygport
> > 
> > 
> > BUILD_REQUIRES="libportaudiocpp-devel libmpg123-devel libogg-devel libvorbis-devel"
> 
> It seems like this also needs libpulse-devel.

Added.

> This looks like a historical mistake, but maybe the files in 
> /usr/share/doc/libopenmpt/examples/ should be in the devel package?

Fixed.

Thanks for the advice.

-- 
Takashi Yano <takashi.yano@nifty.ne.jp>