Re: [Iaxclient-devel] speex echo cancellation tested?

[Matthew C. <mc...@no...>]

On Thu, 2008-04-24 at 15:20 -0400, Peter Grayson wrote:
> Hi Matt
> 
> On Thu, Apr 24, 2008 at 12:24 PM, Matthew Crane <mc...@no...> wrote:
> > On Thu, 2008-04-24 at 10:35 -0400, Peter Grayson wrote:
> >
> >  > >  This is my first time compiling code checked out from the svn
> >  > >  repository.  When trying to run the autogen.sh script in your echocan
> >  > >  branch, autoconf is generating an error when it comes across
> >  > >  "PKG_REQUIRES":
> >  > >
> >  > >  configure.ac:213: error: possibly undefined macro: PKG_REQUIRES
> >  > >       If this token and others are legitimate, please use
> >  > >  m4_pattern_allow.
> >  > >       See the Autoconf documentation.
> 
> [snip]
> 
> >  I found the problem.  The version of pkgconfig in SuSE Linux Enterprise
> >  Desktop 10 SP1 has the following entry in /usr/share/aclocal/pkg.m4:
> >
> >  m4_pattern_forbid([^_?PKG_[A-Z_]+$])
> >
> >  Basically, trying to define any variables of the form
> >  "PKG_BLAH_BLAH_BLAH" in a configure.ac macro file is a no-no.  I've
> >  changed all instances of PKG_REQUIRES in my source tree to
> >  PACKAGE_REQUIRES.  If you want to include this change in the trunk so
> >  this issue doesn't hit any other Linux users that happen to have this
> >  pkgconfig setup, here's the diff:
> 
> Good find. I've committed a fix to trunk.
> 
> Also, I have committed the new echo cancellation code to iaxclient
> trunk. This effectively obsoletes Mihai's echocan branch. As Mihai
> mentioned, the results on Windows have not been spectacular thus far.
> Using a different API besides wmme might yield good results. Note that
> I have not yet given up on getting aec working with wmme.
> 

Unfortunately, it looks like some sort of audio problem has crept in
between 2.1beta3 and now (at least on SLED 10).  Incoming audio sounds
very choppy.  I turned echo cancellation off -- problem still there.  I
compiled using --without-echo-can -- problem still there.

I'm going to checkout a version of the trunk from a few days ago to see
if the echo-cancellation merge introduced these problems or if it was
something earlier.

-Matt