Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 06 Mar 2009 15:47:08 -0500
From:      Joe Marcus Clarke <marcus@freebsd.org>
To:        Andre Heider <a.heider@gmail.com>
Cc:        Marcus von Appen <mva@freebsd.org>, freebsd-ports@freebsd.org
Subject:   Re: SDL / Pulseaudio
Message-ID:  <49B18BCC.6090005@freebsd.org>
In-Reply-To: <efea63230903061231m31a72919u37dbe10c16446e25@mail.gmail.com>
References:  <efea63230903031324x545f15e7l44022e4472f7888b@mail.gmail.com>	<20090303232459.GB1091@medusa.sysfault.org> <efea63230903061231m31a72919u37dbe10c16446e25@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Andre Heider wrote:
> 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 :)

This only works with procfs mounted on /proc, but given that many GNOME
components already require this, it's probably not a big deal here.

One thing I've done in console-kit, for example, is to print a message
when procfs is not mounted.  Perhaps something could be added in SDL as
well if the fopen() fails.

Joe

> 
> 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
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> freebsd-ports@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-ports
> To unsubscribe, send any mail to "freebsd-ports-unsubscribe@freebsd.org"


-- 
Joe Marcus Clarke
FreeBSD GNOME Team	::	gnome@FreeBSD.org
FreeNode / #freebsd-gnome
http://www.FreeBSD.org/gnome



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