Re: SDL audio saga continues

[Peter Valchev <pvalchev@openbsd.org>]

vedge <vedge2@vedge.com.ar> wrote:

> ah. the troublemaker is patch-src_audio_SDL_audio_c
> (#if defined(unix) || defined(__OpenBSD__)) =)
> 
> $OpenBSD: patch-src_audio_SDL_audio_c,v 1.1.1.1 2001/04/19 18:55:40 obecian Exp $
> --- src/audio/SDL_audio.c.orig	Wed Mar 21 10:19:56 2001
> +++ src/audio/SDL_audio.c	Sat May 19 12:01:58 2001
> @@ -185,7 +185,7 @@ int SDL_AudioInit(const char *driver_nam
>  	/* Select the proper audio driver */
>  	audio = NULL;
>  	idx = 0;
> -#ifdef unix
> +#if defined(unix) || defined (__OpenBSD__)
>  	if ( (driver_name == NULL) && (getenv("ESPEAKER") != NULL) ) {
>  		/* Ahem, we know that if ESPEAKER is set, user probably wants
>  		   to use ESD, but don't start it if it's not already running.

Man, as I said -Dunix is passed when the "sun" flavor is selected.
Otherwise, it isn't, and if the OSS flavor is, then exactly because of
that line the sound actually works.  And I don't see how this line is
problematic at all.  Theoretically, it does not matter.  Practically,
try it and see for yourself before posting to ports@.