Re: Feedback requested on an OpenAL audio driver patch

[Leon Freitag]

> AC_CHECK_HEADERS(\
> - OpenAL/al.h \
> - al/al.h \
> AudioUnit/AudioUnit.h \
> CoreAudio/CoreAudio.h \
> IOKit/IOKitLib.h \

Your patch checks for OpenAL headers only in these places. However my distro 
(Suse 10.1) puts openal headers to AL/ instead of al/ and so configure with 
the patch as-is doesn't find files on my system.

Also patching dlls/makefile.in fails. Had to patch it manually.

Leon

Feedback requested on an OpenAL audio driver patch

[Nick Burns]

> From: Leon Freitag 
> Date: Wed, 7 Jun 2006 12:07:32 +0200
> > AC_CHECK_HEADERS(\
> > - OpenAL/al.h \
> > - al/al.h \
> > AudioUnit/AudioUnit.h \
> > CoreAudio/CoreAudio.h \
> > IOKit/IOKitLib.h \
>
> Your patch checks for OpenAL headers only in these places. However my 
> distro
> (Suse 10.1) puts openal headers to AL/ instead of al/ and so configure with
> the patch as-is doesn't find files on my system.

Whoops typo -- al/al.h is wrong -- it should be AL/al.h
(fixed in next patch)
> Also patching dlls/makefile.in fails. Had to patch it manually.

This is probably cause I hand edited the patch diff...
I will redo this in the next patch.
(fixed in next patch)
> Leon

- Nick