Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 6 Mar 2009 21:31:57 +0100
From:      Andre Heider <a.heider@gmail.com>
To:        Marcus von Appen <mva@freebsd.org>, freebsd-ports@freebsd.org
Subject:   Re: SDL / Pulseaudio
Message-ID:  <efea63230903061231m31a72919u37dbe10c16446e25@mail.gmail.com>
In-Reply-To: <20090303232459.GB1091@medusa.sysfault.org>
References:  <efea63230903031324x545f15e7l44022e4472f7888b@mail.gmail.com> <20090303232459.GB1091@medusa.sysfault.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--0016e6570b861bc2580464792d21
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

On Wed, Mar 4, 2009 at 12:24 AM, Marcus von Appen <mva@freebsd.org> wrote:
> On, Tue Mar 03, 2009, Andre Heider wrote:
>
>> Hi list,
>>
>> devel/sdl12 doesn't pick up pulseaudio, no matter if I select it in
>> "make config" or not. I didn't find the time to dig further, but I
>> noticed 2 bits:
>> - the dependency to libpulse-simple.so.0 isn't set in the Makefile
>
> I missed that one, sorry. It will be fixed soon.
>
>> - if I run ./configure without args, it correctly finds the lib and
>> compiles the SDL audio driver for it
>>
>> Is this a known issue or am I missing some other port?
>
> For some weird reason the --enable-pulseaudio switch will disable
> PulseAudio support in 1.2.13. That'll be fixed as well.
>
> Thanks for noticing and notifying :-).
>
> Regards
> Marcus
>

Hi Marcus,

thanks for the compile fix, it builds fine now with pulseaudio support :)

Unfortunately there're some runtime problems with this SDL driver:

1) setting the envvar PULSE_SERVER to forward the stream to another
box results in 'E: proplist.c: Assertion 'p' failed at
pulse/proplist.c:295, function pa_proplist_contains(). Aborting.'
(thats from the SDL app using pulseaudio). paplay works just fine.
2) inefficency, using not used envvars like PASERVER, etc

I fixed 1) with

 static char *get_progname(void)
 {
        char *progname = NULL;
-#ifdef __LINUX__
+#if defined(__LINUX__) || defined (__FreeBSD__)
        FILE *fp;
        static char temp[BUFSIZ];

This obviously doesn't work for other platforms, but at least its fine
on freebsd for me. Streaming to a Linux pulseaudio server works now :)

for 2) see http://bugzilla.libsdl.org/show_bug.cgi?id=698

I applied 2) (without the additional compatibility path, can't test
that and there's at least an #include missing) and combined it with my
oneliner change. Attached you'll find the results, ready to throw at
files/

Regards,
Andre

--0016e6570b861bc2580464792d21--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?efea63230903061231m31a72919u37dbe10c16446e25>