Re: [ITA] SDL2 (2.26.2)

[Takashi Yano <takashi.yano@nifty.ne.jp>]

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

On Sat, 28 Jan 2023 16:24:35 +0000
Jon Turney wrote:
> On 22/01/2023 12:05, Takashi Yano via Cygwin-apps wrote:
> > On Sat, 21 Jan 2023 14:58:55 +0000
> > Jon Turney wrote:
> >> You've made some changes in the configuration here (switching from
> >> enabling pulseaudio to oss)
> >>
> >> I think you've also dropped the winmm patch.
> >>
> >> I don't understand enough to evaluate the correctness those changes, so
> >> it would be nice if you could explain that to me.
> > 
> > Thanks for reviewing. The reason why I turnd off pulseaudio
> > is that the latency was unacceptably large, despite the fact
> > that pulseaudio is preferred over other sound drivers.
> > Due to this, ffplay missynchronized sound with video.
> > 
> > Also, I guess OSS was previously disabled because it does not
> > work as expected due to a bug of cygwin:
> > https://cygwin.com/pipermail/cygwin-patches/2023q1/012134.html
> > and winmm was enabled instead.
> > 
> > Now, the bug has been fixed and SDL2 gets working with OSS.
> > Therefore, we don't need winmm support anymore. I think
> > it is better not to call winmm directly from cygwin apps.
> > 
> > BTW, I tried to modify SDL2 code to minimize the latency of
> > pulseaudio and it succeeded. So, I re-enabled the pulseaudio
> > and updated the package.
> > 
> > Could you please review the package again?
> 
> Thanks for this explanation, it all makes sense now.
> 
> Please go ahead.

Version updated to 2.26.3.

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

[-- Attachment #2: SDL2.cygport --]
[-- Type: text/plain, Size: 1547 bytes --]

NAME="SDL2"
VERSION=2.26.3
RELEASE=1
LICENSE="Zlib"
CATEGORY="Libs"
SUMMARY="Simple DirectMedia Layer core library"
DESCRIPTION="This is the Simple DirectMedia Layer, a general API that provides
low level access to audio, keyboard, mouse, joystick, 3D hardware via OpenGL,
and 2D framebuffer across multiple platforms."
HOMEPAGE="http://www.libsdl.org/"
SRC_URI="http://www.libsdl.org/release/SDL2-${VERSION}.tar.gz"

PKG_NAMES="libSDL2_2.0_0 libSDL2-devel"
libSDL2_2_0_0_REQUIRES="libdbus1_3 libGL1 libsamplerate0 libX11_6 libXcursor1 libXext6 libXi6 libXinerama1 libXrandr2 libXss1" # libaudio2 libpulse-simple0
libSDL2_2_0_0_CONTENTS="usr/bin/cygSDL2-2-0-0.dll usr/share/doc/"
libSDL2_devel_CONTENTS="usr/bin/sdl2-config usr/include/ usr/lib/ usr/share/aclocal/"

DISTCLEANFILES="acinclude/l*.m4"
DIFF_EXCLUDES="SDL_config.h"

ACLOCAL_FLAGS="-I acinclude"
NO_AUTOHEADER=1
CYGCONF_ARGS="
	--enable-pthreads
	--enable-pthread-sem

	--disable-arts
	--disable-arts-shared
	--enable-esd
	--enable-esd-shared
	--enable-nas
	--enable-nas-shared
	--enable-oss
	--enable-pulseaudio
	--enable-pulseaudio-shared
	--enable-libsamplerate
	--enable-libsamplerate-shared

	--enable-video-dummy
	--enable-video-opengl
	--enable-video-opengles
	--enable-video-x11
	--enable-video-x11-scrnsaver
	--enable-video-x11-xcursor
	--enable-video-x11-xinput
	--enable-video-x11-xrandr
	--enable-video-x11-xshape
	--enable-x11-shared
"

DOCS="README-SDL.txt README-porting.txt"

BUILD_REQUIRES="libX11-devel libGL-devel libEGL-devel libsamplerate-devel libesd-devel nas"