configure using deprecated alsa header.. ?

[Tom Spear]

I just downloaded and configured wine 0.9.34 and looking thru the
config.log (didnt do this on prior versions, so I'm pretty sure it's
not a regression), there is a warning about needing to use
alsa/asoundlib.h instead of sys/asoundlib.h ..  I made bug 7997 about
it: http://bugs.winehq.org/show_bug.cgi?id=7997

Could someone look into this and make a patch?  I honestly dont know
if it is causing any problems the way it is currently, but the warning
is a little unsettling.

In case it's needed: Slackware x86 11.0 with the alsa that is included
with the distro.

[Tom Spear]

As a side I looked thru config.log a little further up and saw the
test for asla/asoundlib.h which succeeded.  So in the case that both
headers exist, which one are we using?

On 4/9/07, Tom Spear  wrote:
> I just downloaded and configured wine 0.9.34 and looking thru the
> config.log (didnt do this on prior versions, so I'm pretty sure it's
> not a regression), there is a warning about needing to use
> alsa/asoundlib.h instead of sys/asoundlib.h ..  I made bug 7997 about
> it: http://bugs.winehq.org/show_bug.cgi?id=7997
>
> Could someone look into this and make a patch?  I honestly dont know
> if it is causing any problems the way it is currently, but the warning
> is a little unsettling.
>
> In case it's needed: Slackware x86 11.0 with the alsa that is included
> with the distro.
>
> --
> Thanks
>
> Tom
>
> Check out this new 3D Instant Messenger called IMVU.  It's the best I
> have seen yet!
>
> http://imvu.com/catalog/web_invitation.php?userId=1547373&from=power-ema...

[Tom Spear]

And another update, just ignore the bug and this whole thread.  I
looked thru winealsa code and alsa/asoundlib.h is used if it exists,
then sys/asoundlib.h if it doesnt...

On 4/9/07, Tom Spear  wrote:
> As a side I looked thru config.log a little further up and saw the
> test for asla/asoundlib.h which succeeded.  So in the case that both
> headers exist, which one are we using?
>
> On 4/9/07, Tom Spear  wrote:
> > I just downloaded and configured wine 0.9.34 and looking thru the
> > config.log (didnt do this on prior versions, so I'm pretty sure it's
> > not a regression), there is a warning about needing to use
> > alsa/asoundlib.h instead of sys/asoundlib.h ..  I made bug 7997 about
> > it: http://bugs.winehq.org/show_bug.cgi?id=7997
> >
> > Could someone look into this and make a patch?  I honestly dont know
> > if it is causing any problems the way it is currently, but the warning
> > is a little unsettling.
> >
> > In case it's needed: Slackware x86 11.0 with the alsa that is included
> > with the distro.
> >
> > --
> > Thanks
> >
> > Tom
> >
> > Check out this new 3D Instant Messenger called IMVU.  It's the best I
> > have seen yet!
> >
> > http://imvu.com/catalog/web_invitation.php?userId=1547373&from=power-ema...
>
> --
> Thanks
>
> Tom
>
> Check out this new 3D Instant Messenger called IMVU.  It's the best I
> have seen yet!
>
> http://imvu.com/catalog/web_invitation.php?userId=1547373&from=power-ema...

[Maarten Lankhorst]

Tom Spear schreef:
> I just downloaded and configured wine 0.9.34 and looking thru the
> config.log (didnt do this on prior versions, so I'm pretty sure it's
> not a regression), there is a warning about needing to use
> alsa/asoundlib.h instead of sys/asoundlib.h ..  I made bug 7997 about
> it: http://bugs.winehq.org/show_bug.cgi?id=7997

Looking through it it seems configure.ac first tries to include
alsa/asoundlib in its test and if that fails sys/asoundlib.h, so it
doesn't look like a bug to me.